Changed Orm\Query::update() to return true when affected rows = 0.

fuel · May 21, 2011 · 42e17df · 42e17df
1 parent e322780
commit 42e17df
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/classes/model.php b/classes/model.php
@@ -870,7 +870,6 @@ protected function update()
 		}
 
 		// update the original property on success
-
 		$this->observe('after_update');
 
 		return true;

diff --git a/classes/query.php b/classes/query.php
@@ -1026,7 +1026,8 @@ public function update()
 		$this->relations = $tmp_relations;
 		$this->group_by  = $tmp_group_by;
 
-		return $res > 0;
+		// Update can affect 0 rows when input types are different but outcome stays the same
+		return $res >= 0;
 	}
 
 	/**