Bug 35322: Fix the logic for checking for holds in CanBookBeIssued
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 13 Nov 2023 15:37:43 +0000 (10:37 -0500)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Tue, 16 Jan 2024 11:06:13 +0000 (12:06 +0100)
commit9b10f69ee318c6fe3fa1ffb6de82c0f9888cac71
treeb617ccdb614ff7fd5deae8226b37bed2245e755b
parent8f13f74e39a93314abc814735b6fcd1504e49373
Bug 35322: Fix the logic for checking for holds in CanBookBeIssued

We are having reports that AllowItemsOnHoldCheckoutSCO and AllowItemsOnHoldCheckoutSIP no longer work. It appreas that in CanBookBeIssued, the ignore reserves check was changed from "check reserves unless the ignore reserves flag was passed" to "check reserves unless the ignore reserves flag was passed *and* we have a recall". I think this was a logic mistake and we want to check reserves unless we have an ignore flag *or* there is a recall.

Test Plan:
1) Enable AllowItemsOnHoldCheckoutSCO
2) Place a hold on an item
3) Attempt to check that item out to another patron
4) Note the checkout is blocked
5) Apply this patch
6) Restart all the things!
7) Attempt the checkout again
8) The checkout now succeeds!

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
C4/Circulation.pm