https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20086
Jonathan Druart <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #94377|0 |1 is obsolete| | Attachment #94378|0 |1 is obsolete| | --- Comment #7 from Jonathan Druart <[email protected]> --- Created attachment 94421 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94421&action=edit Bug 20086: Execute AddRenewal in a transaction to avoid partial success and doubled fines This patch starts a transaction and only commits if renewal and fines updates and charges are successful (partial in any cna be problematic) There is no feedback (as currently there is none either) but if part fails, all fails. I didn't include stats and notifications in the transaction, but we could. (Edit JD: not true, they are included) To test: 1 - Apply patch 2 - prove t/db_dependent/Circulation.t 3 - Attempt circs and renewals should be no difference 4 - If possible make part of transaction fail and ensure all fails Signed-off-by: Martin Renvoize <[email protected]> Signed-off-by: Jonathan Druart <[email protected]> Amended: commit title and one indendation (return statement) -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
