| # |
Jun 29th 2017, 06:11 |
Sebollson |
but afterSave does not work like I am expecting they would |
| # |
Jun 29th 2017, 06:11 |
 savant |
yes they should |
| # |
Jun 29th 2017, 06:11 |
Sebollson |
so in general as I understand correctly indeed events in behavior works the same as hooks in table |
| # |
Jun 29th 2017, 06:10 |
Sebollson |
yes i think that I will |
| # |
Jun 29th 2017, 06:10 |
savant |
Maybe you can request that we handle it for transactions? |
| # |
Jun 29th 2017, 06:09 |
savant |
https://github.com/cakephp/cakephp/blob/master/src/ORM/Table.php#L1850 |
| # |
Jun 29th 2017, 06:09 |
savant |
but dont handle transactions like that in `Model.afterSave` |
| # |
Jun 29th 2017, 06:09 |
savant |
looks like we handle that case for `Model.beforeSave` |
| # |
Jun 29th 2017, 06:09 |
savant |
https://github.com/cakephp/cakephp/blob/master/src/ORM/Table.php#L1789-L1793 |
| # |
Jun 29th 2017, 06:06 |
Sebollson |
you want to say that afterSave ->false in table will not rollback either? |
| # |
Jun 29th 2017, 06:06 |
Sebollson |
wait |
| # |
Jun 29th 2017, 06:05 |
Sebollson |
returning false in afterSave in behavior does not rollback the transaction |
| # |
Jun 29th 2017, 06:05 |
savant |
what makes you say that |
| # |
Jun 29th 2017, 06:05 |
savant |
why isnt it |
| # |
Jun 29th 2017, 06:05 |
Sebollson |
not* |
| # |
Jun 29th 2017, 06:05 |
Sebollson |
but it now ;( |
| # |
Jun 29th 2017, 06:05 |
savant |
it should be the same |
| # |
Jun 29th 2017, 06:04 |
Sebollson |
i think that behavior events should be threated the same |
| # |
Jun 29th 2017, 06:04 |
Sebollson |
and it would work |
| # |
Jun 29th 2017, 06:04 |
Sebollson |
normally I would reutrn false in afterSave method of table |
| # |
Jun 29th 2017, 06:04 |
savant |
if you want to screw with its connection |
| # |
Jun 29th 2017, 06:03 |
savant |
but you may need to get a reference to the table |
| # |
Jun 29th 2017, 06:03 |
Sebollson |
normally |
| # |
Jun 29th 2017, 06:03 |
Sebollson |
as special case of ORM event listener |
| # |
Jun 29th 2017, 06:03 |
savant |
it should |
| # |
Jun 29th 2017, 06:03 |
Sebollson |
shouldnt it work the same ? |
| # |
Jun 29th 2017, 06:03 |
Sebollson |
but in case of behaviors |
| # |
Jun 29th 2017, 06:03 |
Sebollson |
i know that beforeSave in table is not the same as beforeSave event |
| # |
Jun 29th 2017, 06:03 |
Sebollson |
but now |
| # |
Jun 29th 2017, 06:03 |
savant |
sad panda |
| # |
Jun 29th 2017, 06:03 |
savant |
:( |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
in both cases I have to rollback |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
and target entity |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
during saving of audit entity |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
and I have 2 possible points of failure |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
its about audits |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
i have to |
| # |
Jun 29th 2017, 06:02 |
savant |
but that should work |
| # |
Jun 29th 2017, 06:02 |
Sebollson |
:) |
| # |
Jun 29th 2017, 06:02 |
savant |
YOLODB |
| # |
Jun 29th 2017, 06:02 |
savant |
i dont use transactions usually |