Making saveAll() behave like plain save() when suplied empty data arr…

…ay, closes #277

cake/libs/model/model.php

@@ -1584,6 +1584,11 @@ function saveAll($data = null, $options = array()) {
 		$validates = true;
 		$return = array();
 
+		if (empty($data) && $options['validate'] !== false) {
+			$result = $this->save($data, $options);
+			return !empty($result);
+		}
+
 		if ($options['atomic'] && $options['validate'] !== 'only') {
 			$db->begin($this);
 		}

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

@@ -3798,6 +3798,24 @@ function testProductUpdateAllWithoutForeignKey() {
         $this->assertEqual($resultsFkFalse, $expected);
     }
 
+/**
+ * test that saveAll behaves like plain save() when suplied empty data
+ *
+ * @link http://cakephp.lighthouseapp.com/projects/42648/tickets/277-test-saveall-with-validation-returns-incorrect-boolean-when-saving-empty-data
+ * @access public
+ * @return void
+ */
+	function testSaveAllEmptyData() {
+		$this->loadFixtures('Article', 'ProductUpdateAll');
+		$model =& new Article();
+		$result = $model->saveAll(array(), array('validate' => 'first'));
+		$this->assertTrue($result);
+
+		$model =& new ProductUpdateAll();
+		$result = $model->saveAll(array());
+		$this->assertFalse($result);
+	}
+
 }
 
 ?>