adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-15 03:18:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fixed bug in Model::__save validation logic, closes #4554

Browse source 
git-svn-id: https://svn.cakephp.org/repo/branches/1.2.x.x@6824 3807eeeb-6ff5-0310-8944-8be069107fe0
This commit is contained in:
the_undefined 2008-05-13 01:03:11 +00:00
parent 232e3f7ec1
commit 823eb81dc7
2 changed files with 18 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
cake/libs/model/model.php
View file

@ -1428,7 +1428,7 @@ class Model extends Overloadable {
*/
function __save(&$model, $data, $options) {
if ($options['validate'] === 'first' || $options['validate'] === 'only') {
if (!($model->create($data) && $this->validates())) {
if (!($model->create($data) && $model->validates())) {
return false;
}
} elseif (!($model->create(null) !== null && $model->save($data, $options))) {

17
cake/tests/cases/libs/model/model.test.php
View file

@ -2321,6 +2321,23 @@ class ModelTest extends CakeTestCase {
$this->model->validate['body'] = VALID_NOT_EMPTY;
}
function testSaveAllValidationOnly() {
$this->model =& new Comment();
$this->model->Attachment->validate = array('attachment' => VALID_NOT_EMPTY);
$data = array(
'Comment' => array(
'comment' => 'This is the comment'
),
'Attachment' => array(
'attachment' => ''
)
);
$result = $this->model->saveAll($data, array('validate' => 'only'));
$this->assertFalse($result);
}
function testSaveWithCounterCache() {
$this->loadFixtures('Syfile', 'Item');
$this->model =& new Syfile();