| # |
Jun 27th 2017, 15:37 |
 elboletaire |
I'll try to create a really simple project with all these entities, and try to reproduce the steps there, so I can open an issue |
| # |
Jun 27th 2017, 15:36 |
jeremyharris |
oy |
| # |
Jun 27th 2017, 15:36 |
elboletaire |
reverse doesn't work either... |
| # |
Jun 27th 2017, 15:36 |
jeremyharris |
either way, it shouldn’t return true if it’s not saving, so there might be a bug somwehere |
| # |
Jun 27th 2017, 15:36 |
elboletaire |
I'm trying.. |
| # |
Jun 27th 2017, 15:35 |
jeremyharris |
while this shouldn’t matter, have you tried the reverse? `$this->Users->link($role, [$users])` |
| # |
Jun 27th 2017, 15:34 |
elboletaire |
nope, same result :( |
| # |
Jun 27th 2017, 15:33 |
elboletaire |
nope, let me try |
| # |
Jun 27th 2017, 15:33 |
jeremyharris |
have you tried saving a single user instead of looping? I wonder if there’s some state bug |
| # |
Jun 27th 2017, 15:32 |
jeremyharris |
(I’m reading through the code atm) |
| # |
Jun 27th 2017, 15:32 |
jeremyharris |
I feel ya |
| # |
Jun 27th 2017, 15:32 |
elboletaire |
exactly that's how I've been the latest two ours... |
| # |
Jun 27th 2017, 15:31 |
jeremyharris |
hrm |
| # |
Jun 27th 2017, 15:31 |
elboletaire |
`replace` |
| # |
Jun 27th 2017, 15:30 |
jeremyharris |
what’s the saveStrategy? |
| # |
Jun 27th 2017, 15:30 |
elboletaire |
no relations are touched... the original relations remain the same |
| # |
Jun 27th 2017, 15:30 |
elboletaire |
in fact |
| # |
Jun 27th 2017, 15:30 |
elboletaire |
so both apparently are saving, but no relations are stored |
| # |
Jun 27th 2017, 15:30 |
elboletaire |
the link returns true, the `save` (using `set`) returns the entity |
| # |
Jun 27th 2017, 15:29 |
jeremyharris |
wait you said it’s returning true and fails to save? |
| # |
Jun 27th 2017, 15:29 |
jeremyharris |
also check the table rules |
| # |
Jun 27th 2017, 15:29 |
jeremyharris |
perhaps pepper the core with some debug statements, that’d be my next step. specifically where it returns false. one might be failing for some strange reason |
| # |
Jun 27th 2017, 15:28 |
elboletaire |
ofc |
| # |
Jun 27th 2017, 15:27 |
jeremyharris |
I’m assuming you debugged roles and users, and they have content |
| # |
Jun 27th 2017, 15:26 |
jeremyharris |
yeah actually; scratch that idea |
| # |
Jun 27th 2017, 15:26 |
elboletaire |
but, setting it to `dirty` is only needed if I'm trying to save new contained entities (roles in my case), right? |
| # |
Jun 27th 2017, 15:25 |
jeremyharris |
elboletaire they might be returning true if there is nothing to save, eg, no dirty properties |
| # |
Jun 27th 2017, 15:21 |
elboletaire |
btw, I'm using Cake 3.3.16, I don't know if there's any known issue related to this (I wasn't able to found it in github issues) |
| # |
Jun 27th 2017, 15:21 |
elboletaire |
but I don't feel very confortable using that approach, as both entities are already saved (using `_ids` seems like I'm adding much logic that can be ignored if I use `link`) |
| # |
Jun 27th 2017, 15:20 |
elboletaire |
yes |
| # |
Jun 27th 2017, 15:20 |
neon1024 |
Does it work if you build an array with _ids and use patchEntity() ? |
| # |
Jun 27th 2017, 15:19 |
elboletaire |
maybe I could debug it somehow? some protected property in the entity maybe? |
| # |
Jun 27th 2017, 15:19 |
neon1024 |
save() returns mixed if I recall, false or the data |
| # |
Jun 27th 2017, 15:19 |
neon1024 |
I would imagine that set() and link() return boolean as they’re just assigning things, so the boolean would be a ‘Yes Ive done this’ type return |
| # |
Jun 27th 2017, 15:18 |
elboletaire |
btw, either case works for me :( any ideas on why this apparently works (both `save` using `set` and `link` always return true), but it really doesn't? |
| # |
Jun 27th 2017, 15:17 |
elboletaire |
it's a bit more complicated than that... but it does internally `set` the related entities, and then runs `saveLinks` |
| # |
Jun 27th 2017, 15:17 |
neon1024 |
Oh right, okay :slightly_smiling_face: |
| # |
Jun 27th 2017, 15:17 |
elboletaire |
in fact, `link` does that internally afaik |
| # |
Jun 27th 2017, 15:16 |
elboletaire |
doesn't work either using `set` |
| # |
Jun 27th 2017, 15:16 |
neon1024 |
It’s the default too I think |
| # |
Jun 27th 2017, 15:16 |
neon1024 |
Cool, that’s the one I prefer. I find it simpler to deal with |