]> git.koha-community.org Git - koha.git/commit
Bug 34678: Allow new entries to overwrite hold_fill_targets
authorNick Clemens <nick@bywatersolutions.com>
Wed, 13 Sep 2023 17:03:32 +0000 (17:03 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 27 Oct 2023 19:44:24 +0000 (16:44 -0300)
commitbbeab36789d8dd020bc5395d76c54cc2910caf49
tree2853e8dd50b90d0db5bdf05dd64684c2ec5beceb
parentdfad12d0790e38e739c610dd2a5f2345a053ed52
Bug 34678: Allow new entries to overwrite hold_fill_targets

When using background jobs, there is a possibility of a race condition where two jobs will be updating the holds queue for the same biblio. We should try to minimize those cases (see bug 34596)

In the meantime though, we should prevent jobs possibly dying, and allow the most recent update to succeed.

There is a possibility two updates wil assign different items to the same reserve, and that a reserve could end up in the queue twice, however, whichever one is filled first will delete both entries. as filling the hold deletes by reserve id (see bug 24359)

This patch adds a transaction to delete and then inset the new row

To test:
1 - prove -v t/db_dependent/Reserves.t
2 - It fails
3 - Apply patch
4 - t/db_dependent/Reserves.t
5 - It succeeds!

Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
C4/HoldsQueue.pm