Cakephp 3 table updateAll using array

Question

In Cakephp 2's model updateAll, we were using arrays as conditions. But this doesn't work :

$data=array(12, 22, 44);
TableRegistry::get('Foobar')->updateAll(
   array("checked" => 1),
   array("user_id" => $data) 
 );

Is this not possible in Cakephp 3's updateAll ?

This works in cakephp 3:

$data=12;
TableRegistry::get('Foobar')->updateAll(
   array("checked" => 1),
   array("user_id" => $data) 
 );

Regarding to API docs, $conditions defined like this:

updateAll( array $fields , mixed $conditions )

array $fields
A hash of field => new value.

mixed $conditions
Conditions to be used, accepts anything Query::where() can take.

Edit:
In case of cake works well and I'm making a mistake. How can I understand the cause ? Because when condition is an integer, code works well.

Edit2: This also works well:

 $data=array(12, 22, 44);
 ConnectionManager::get('default')->query('UPDATE foobars SET checked=1 
     WHERE user_id IN ('.implode(",",$data).')')->execute();

Sorry I couldn't understand. Do you mean updateAll function ? — trante
– trante, Commented Jul 16, 2015 at 20:44
stackoverflow.com/questions/26887511/where-in-clause I hope you are aware that your raw query is prone to SQL injections in case $data will be user input! — ndm
– ndm, Commented Jul 17, 2015 at 14:04
Thank you ndm. Yes, thats why I need to create a cakephp type query, rather than a raw query. I suppose the problem is missing of IN keyword. I couldnt see it in orm migration notes. book.cakephp.org/3.0/en/appendices/orm-migration.html Also I would be happy to accept your reply as an answer to this question. — trante
– trante, Commented Jul 17, 2015 at 14:22

José Lorenzo Rodríguez · Accepted Answer · 2015-07-16 20:43:39Z

1

It does work, as explained in the docs:

http://book.cakephp.org/3.0/en/orm/saving-data.html#bulk-updates

answered Jul 16, 2015 at 20:43

José Lorenzo Rodríguez

9,62437 silver badges32 bronze badges

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

3 Comments

trante Over a year ago

Is there a test case that checks for updateAll, with array conditions that has one key and multiple values ? github.com/cakephp/cakephp/…

José Lorenzo Rodríguez Over a year ago

It seems like there is not... What does you intuition says about it? How would you use multiple set and multiple condiitions?

trante Over a year ago

I updated question, you can see from edit2 that, Connection->query() works well for this situation. I suppose I make a mistake using updateAll method. But not yet found the cause.

Lucho · Accepted Answer · 2018-06-05 14:20:40Z

1

A little late to the discussion, but since I had a similar problem, using something like id in array condition, this is how is done now on Cakephp 3.

//using the same example
$data=array(12, 22, 44);

TableRegistry::get('Foobar')->updateAll(
   ["checked" => 1],
   ["user_id IN " => $data] //mind the IN on the condition
);

Here is another way from the docs, updated for the example

$articles = TableRegistry::get('Articles');
$data=array(12, 22, 44);

$query = $articles->query();
$query->update()
    ->set(['checked' => true])
    ->where(['user_id IN' => $data])
    ->execute();

Hope it helps someone else.

edited Jun 5, 2018 at 14:20

answered Sep 6, 2017 at 18:21

Lucho

1,0441 gold badge17 silver badges26 bronze badges

Collectives™ on Stack Overflow

Cakephp 3 table updateAll using array

2 Answers 2

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related