Merge pull request #7 from dieend/patch-1

petrabarus · petrabarus · commit 6f4ad3ca63a7 · 2015-11-03T09:40:00.000+07:00
Return false when fails to insert new record
diff --git a/src/ActiveRecord.php b/src/ActiveRecord.php
@@ -115,13 +115,13 @@ public function insert($runValidation = true, $attributes = null) {
         }
         $values = $this->getDirtyAttributes($attributes);
 
-        $this->getDb()->createCommand()->insert($this->tableName(), $values);
+        $ret = $this->getDb()->createCommand()->insert($this->tableName(), $values);
 
         $changedAttributes = array_fill_keys(array_keys($values), null);
         $this->setOldAttributes($values);
         $this->afterSave(true, $changedAttributes);
         
-        return true;
+        return $ret;
     }
     /**
      * Returns the primary key **name(s)** for this AR class.

Original file line number	Diff line number	Diff line change
`@@ -115,13 +115,13 @@ public function insert($runValidation = true, $attributes = null) {`
`115`	`115`	`}`
`116`	`116`	`$values = $this->getDirtyAttributes($attributes);`
`117`	`117`
`118`		`- $this->getDb()->createCommand()->insert($this->tableName(), $values);`
	`118`	`+ $ret = $this->getDb()->createCommand()->insert($this->tableName(), $values);`
`119`	`119`
`120`	`120`	`$changedAttributes = array_fill_keys(array_keys($values), null);`
`121`	`121`	`$this->setOldAttributes($values);`
`122`	`122`	`$this->afterSave(true, $changedAttributes);`
`123`	`123`
`124`		`- return true;`
	`124`	`+ return $ret;`
`125`	`125`	`}`
`126`	`126`	`/**`
`127`	`127`	`* Returns the primary key name(s) for this AR class.`