Bug 25566: Add option to ignore found holds and use it when checking high holds
authorNick Clemens <nick@bywatersolutions.com>
Thu, 21 May 2020 13:41:23 +0000 (13:41 +0000)
committerVictor Grousset/tuxayo <victor@tuxayo.net>
Mon, 14 Sep 2020 19:59:08 +0000 (21:59 +0200)
commita4f7eaf9616613060e7766a82b6c398bf73839d7
treec72a064139f37e98fe3dd5e573d4a1db03462775
parent60f26dfaf5e2ee164bb1682a0aece62eed05e83c
Bug 25566: Add option to ignore found holds and use it when checking high holds

To test:
 1 - Find or create a record with 10 items
 2 - Set sysprefs:
     decreaseLoanHighHolds - enable
     decreaseLoanHighHoldsDuration - 2
     decreaseLoanHighHoldsValue - 2
     decreaseLoanHighHoldsControl  - 'over the number of holdable items'/dynamic
 3 - Set circ rules to allow 1 hold per record on the relevant record
 4 - Place 3 holds on the record
 5 - Check one item in and confirm hold to set to waiting
 6 - Issue to the patron with the waiting hold
 7 - Get a notice that loan period is decreased
 8 - Don't confirm the checkout
 9 - Apply patch
10 - Restart all the things
11 - Repeat checkout, no decrease this time!

Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 96a871035043c7ffd93c61e756ee1ff89e3da0f0)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit d59536e289692537289460a9225cc0c0bce80a57)

Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
(cherry picked from commit fe86de90479a4f6a6a52277490a83cf8e133dbba)
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
C4/Circulation.pm
C4/Reserves.pm
t/db_dependent/DecreaseLoanHighHolds.t
t/db_dependent/Reserves/MultiplePerRecord.t