Bugzilla – Attachment 160892 Details for
Bug 35322
AllowItemsOnHoldCheckoutSCO and AllowItemsOnHoldCheckoutSIP do not work
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 35322: Fix the logic for checking for holds in CanBookBeIssued
Bug-35322-Fix-the-logic-for-checking-for-holds-in-.patch (text/plain), 1.56 KB, created by
Emily Lamancusa (emlam)
on 2024-01-11 21:14:17 UTC
(
hide
)
Description:
Bug 35322: Fix the logic for checking for holds in CanBookBeIssued
Filename:
MIME Type:
Creator:
Emily Lamancusa (emlam)
Created:
2024-01-11 21:14:17 UTC
Size:
1.56 KB
patch
obsolete
>From 28df0519184efa9ec0a12c2587d00cdd76670a44 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Mon, 13 Nov 2023 10:37:43 -0500 >Subject: [PATCH] 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> >--- > C4/Circulation.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index 29abc95ac7..109809d4b9 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -1185,7 +1185,7 @@ sub CanBookBeIssued { > } > } > >- unless ( $ignore_reserves and defined $recall ) { >+ unless ( $ignore_reserves || $recall ) { > # See if the item is on reserve. > my ( $restype, $res ) = CheckReserves( $item_object ); > if ($restype) { >-- >2.34.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 35322
:
158886
|
159341
|
159510
|
159511
| 160892 |
160893
|
160894