# |
Jun 27th 2017, 15:39 |
jeremyharris |
yep, thanks for working on a test case :slightly_smiling_face: will help a lot of other people if it is indeed a bug |
# |
Jun 27th 2017, 15:38 |
elboletaire |
thanks for the time buddies :slightly_smiling_face: |
# |
Jun 27th 2017, 15:38 |
elboletaire |
so, I'll try to do that... create a project to reproduce this issue (and others I've found related to this) |
# |
Jun 27th 2017, 15:37 |
elboletaire |
it's a very huge project, but I've tried disabling all the behaviors from both tables without any change on the save process... |
# |
Jun 27th 2017, 15:37 |
jeremyharris |
that’s probably a good idea. if you can create a failing test case I’d be happy to work on a fix |
# |
Jun 27th 2017, 15:37 |
jeremyharris |
yeah |
# |
Jun 27th 2017, 15:37 |
jeremyharris |
is there anything unconventional about your table setup |
# |
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 |