Changing return of Model::deleteAll(). When no records are matched by…

… the delete conditions return is now (bool)true as no records matching those conditions exist.

Test case added.
Fixes #6453

git-svn-id: https://svn.cakephp.org/repo/branches/1.2.x.x@8216 3807eeeb-6ff5-0310-8944-8be069107fe0

cake/libs/model/model.php

@@ -1811,7 +1811,7 @@ function deleteAll($conditions, $cascade = true, $callbacks = false) {
 			);
 
 			if (empty($ids)) {
-				return false;
+				return true;
 			}
 
 			if ($callbacks) {

cake/tests/cases/libs/model/model.test.php

@@ -7487,6 +7487,9 @@ function testDeleteAll() {
 				'published' => 'Y'
 		)));
 		$this->assertEqual($result, $expected);
+
+		$result = $TestModel->deleteAll(array('Article.user_id' => 999));
+		$this->assertTrue($result, 'deleteAll returned false when all no records matched conditions. %s');
 	}
 /**
  * testRecursiveDel method