Debugging save() returning false CakePHP 3.0

Question

I have some code which has broken somewhere along the way and I'm having trouble debugging it.

This is a simplified version of it.

$data = $this->request->data;

$form = $this->Forms->get($data['id'], [
    'contain' => ['FieldsForms' => ['data']
    ]
]);

$form = $this->Forms->patchEntity($form, $data,
    ['associated' => [
        'FieldsForms.Data',

    ]
]);

if ($this->Forms->save($form)) {
    // sunshine and rainbows
} else {
    // wailing and gnashing of teeth
}

I'm left wailing and gnashing teeth without any errors, as far as I can see if I debug the $data it looks like it's ok (though since it's fairly long and contains a bunch of UUIDs it's possible I'm missing something).

Validation errors is empty.

The save is returning false - any suggestions on how to debug this might save what sanity I have left.

Thanks!

When you are saying "Validation errors is empty", how do you check that? After saving so that possible table rule errors are included? — ndm
– ndm, Commented Feb 20, 2015 at 15:16
The view is rendered and using debug kit it shows validation for 'form' as empty — Rich Hobbs
– Rich Hobbs, Commented Feb 20, 2015 at 15:25
Then I'd suggest digging into the CakePHP core source to debug the control flow that originates from the Table::save() call. — ndm
– ndm, Commented Feb 20, 2015 at 16:24

space97 · Accepted Answer · 2020-05-23 22:38:35Z

19

The problem turned out to be the data, as expected but couldn't see immediately because the save was returning false and the data was quite large.

I first made a subset of the problem data which displayed the same behaviour then, following ndm's suggestion, changed the ORM/Table.php code for the save function as follows to be able to see where the problem was:

// $entity->errors() is deprecated as of CakePHP 3.7
// $entity->getErrors() should be used in later versions
$x = $entity->errors();
if ($x) {
    debug($entity);
    debug($x);
    return false;
}

So that I could see what was going on and went on to fix the data.

edited May 23, 2020 at 22:38

space97

3031 gold badge5 silver badges23 bronze badges

answered Feb 23, 2015 at 13:46

Rich Hobbs

3031 gold badge2 silver badges10 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

miCRoSCoPiC_eaRthLinG Over a year ago

Rich - this is brilliant. Works like a charm. Is there any way to implement this by overriding the Table class in your app code, instead of modifying the core? Thank you.

space97 Over a year ago

$entity->getErrors() should be used instead of $entity->errors() in CakePHP 3.7 and higher.

Annabel · Accepted Answer · 2018-03-02 13:08:54Z

5

Not sure if the earlier answer is based on an older version, but in the latest cakephp version (3.4) you can retrieve the errors directly from the $entity within the controller. The errors array contains each entity field that failed, with a child array of failed validations.

<?php
// In Articles Controller
...

public function add(){
...
  if ($this->Articles->save($article)) {
    $this->Flash->success(__('The Article has been saved.'));
    return $this->redirect(['action' => 'index']);
  } else {
    Log::Debug($article->errors());
  }

edited Mar 2, 2018 at 13:08

Annabel

1,42416 silver badges23 bronze badges

answered Sep 7, 2017 at 2:32

Brad

1361 silver badge3 bronze badges

2 Comments

jokermt235 Over a year ago

Using cakephp 3.3.* helped a lot thanks one up arrow from me!

levolutionniste Over a year ago

For me, the following worked well: }else{debug($item->errors());} Though I wonder why debug the $item value doesn't display the errors part of the $item object by default. It would be easier if by simply '(trivial debugging the manipulated value ($item in my case, or $article here up) one could directly see those eventual errors .

Dino Moreira · Accepted Answer · 2018-02-16 22:28:23Z

0

Instead modify the core Cake code you can do this:

if ($this->Forms->save($form)) {
    // sunshine and rainbows
} else {
    //you know now what fail
    $andTheErrorsAre = $entity->getErrors();
}

answered Feb 16, 2018 at 22:28

Dino Moreira

611 silver badge3 bronze badges

Comments

Marwan Salim · Accepted Answer · 2019-07-04 10:51:31Z

0

For CakePHP 3.4.*

Try this code:

if (!$this->Forms->save($form)) {
    $this->log($form->getErrors(),'error');
}

Read more about logs here: https://book.cakephp.org/3.0/en/core-libraries/logging.html#using-the-filelog-adapter

answered Jul 4, 2019 at 10:51

Marwan Salim

7327 silver badges15 bronze badges

Collectives™ on Stack Overflow

Debugging save() returning false CakePHP 3.0

4 Answers 4

2 Comments

2 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

2 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related