https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42404 --- Comment #19 from Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> --- Created attachment 200838 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200838&action=edit Bug 42404: (follow-up) Wrap per-image replace+store in a transaction Without a transaction, the delete-then-store sequence in replace mode is non-atomic: if ->store() fails the old images are already gone and cannot be recovered. Wrap each image's delete (optional) and store in a txn_do() block so both either succeed together or roll back together. Each image is processed independently so a failure on one does not affect others. -- You are receiving this mail because: You are watching all bug changes.