summaryrefslogtreecommitdiff
path: root/framework/Data/ActiveRecord
diff options
context:
space:
mode:
authorxue <>2007-10-03 14:22:01 +0000
committerxue <>2007-10-03 14:22:01 +0000
commitc81e28244ac07075473a1ce1bb8fec4719a9a3ba (patch)
tree3f3294438c63e37f2e18b1f2f0ffe12ac09dfda8 /framework/Data/ActiveRecord
parent33bfc4e18d51ba7cb459efa6c3d2902cc34d6514 (diff)
removed redundant code.
Diffstat (limited to 'framework/Data/ActiveRecord')
-rw-r--r--framework/Data/ActiveRecord/TActiveRecord.php19
1 files changed, 9 insertions, 10 deletions
diff --git a/framework/Data/ActiveRecord/TActiveRecord.php b/framework/Data/ActiveRecord/TActiveRecord.php
index bf7b68d9..b53cdf45 100644
--- a/framework/Data/ActiveRecord/TActiveRecord.php
+++ b/framework/Data/ActiveRecord/TActiveRecord.php
@@ -220,8 +220,7 @@ abstract class TActiveRecord extends TComponent
*/
public function equals(TActiveRecord $record, $strict=false)
{
- $thisClass=__CLASS__;
- if(!($record instanceof $thisClass))
+ if($record===null)
return false;
$tableInfo = $this->getRecordGateway()->getRecordTableInfo($this);
$pks = $tableInfo->getPrimaryKeys();
@@ -251,7 +250,7 @@ abstract class TActiveRecord extends TComponent
*/
public static function finder($className=__CLASS__)
{
- if($className==='TActiveRecord')
+ if($className===__CLASS__)
throw new TActiveRecordException('ar_invalid_finder_class_name');
static $finders = array();
@@ -425,7 +424,7 @@ abstract class TActiveRecord extends TComponent
*
* @param string|TActiveRecordCriteria SQL condition or criteria object.
* @param mixed parameter values.
- * @return TActiveRecord matching record object.
+ * @return TActiveRecord matching record object. Null if no result is found.
*/
public function find($criteria,$parameters=array())
{
@@ -440,7 +439,7 @@ abstract class TActiveRecord extends TComponent
*
* @param string|TActiveRecordCriteria SQL condition or criteria object.
* @param mixed parameter values.
- * @return array matching record objects
+ * @return array matching record objects. Empty array if no result is found.
*/
public function findAll($criteria=null,$parameters=array())
{
@@ -461,7 +460,7 @@ abstract class TActiveRecord extends TComponent
* </code>
*
* @param mixed primary keys
- * @return TActiveRecord
+ * @return TActiveRecord. Null if no result is found.
*/
public function findByPk($keys)
{
@@ -486,7 +485,7 @@ abstract class TActiveRecord extends TComponent
* $finder->findAllByPk(array(array($key1, $key2), array($key3, $key4), ...));
* </code>
* @param mixed primary keys
- * @return array matching ActiveRecords
+ * @return array matching ActiveRecords. Empty array is returned if no result is found.
*/
public function findAllByPks($keys)
{
@@ -502,7 +501,7 @@ abstract class TActiveRecord extends TComponent
* class.
* @param string select SQL
* @param array $parameters
- * @return TActiveRecord
+ * @return TActiveRecord, null if no result is returned.
*/
public function findBySql($sql,$parameters=array())
{
@@ -518,7 +517,7 @@ abstract class TActiveRecord extends TComponent
* class.
* @param string select SQL
* @param array $parameters
- * @return array matching active records.
+ * @return array matching active records. Empty array is returned if no result is found.
*/
public function findAllBySql($sql,$parameters=array())
{
@@ -538,7 +537,7 @@ abstract class TActiveRecord extends TComponent
* @param TActiveRecordCriteria additional criteria
* @param array field names to match with "(fields) IN (values)" sql clause.
* @param array matching field values.
- * @return array matching active records.
+ * @return array matching active records. Empty array is returned if no result is found.
*/
public function findAllByIndex($criteria,$fields,$values)
{