From ce2db934b053b443b783cca06ab35282808ee1e2 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 18 Jun 2020 13:23:04 -0400 Subject: [PATCH] Bug 25808: Renewal via the SIP 'checkout' message gives incorrect message If a renewal via SIP cannot be made because the patron has reached the maximum number of renewals, the AF screen message should read "Item has reached maximum renewals!", instead we get "Item checked out to another patron". Test Plan: 1) Check out an item to a patron 2) Using the SIP CLI tool, run checkout messages until the checkout has reached the maximum number of renewals 3) Note the incorrect message in the AF field 4) Apply this patch 5) Restart the SIP server 6) Run another SIP checkout message 7) Note the message is now correct! --- C4/SIP/ILS.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm index 7cff9c0579..c10bb18901 100644 --- a/C4/SIP/ILS.pm +++ b/C4/SIP/ILS.pm @@ -148,7 +148,7 @@ sub checkout { $circ->screen_msg("Invalid Item"); } elsif ( $item->{borrowernumber} - && !_ci_cardnumber_cmp( $item->{borrowernumber}, $patron_id ) ) + && !_ci_cardnumber_cmp( $item->{borrowernumber}, $patron->borrowernumber ) ) { $circ->screen_msg("Item checked out to another patron"); } @@ -160,7 +160,7 @@ sub checkout { # If the item is already associated with this patron, then # we're renewing it. $circ->renew_ok( $item->{borrowernumber} - && _ci_cardnumber_cmp( $item->{borrowernumber}, $patron_id ) ); + && _ci_cardnumber_cmp( $item->{borrowernumber}, $patron->borrowernumber ) ); $item->{borrowernumber} = $patron_id; $item->{due_date} = $circ->{due}; -- 2.24.1 (Apple Git-126)