From 5699c825ccd89bc4166c5e8abfc4b3c39f8ad859 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Wed, 3 Feb 2021 10:03:31 +0000 Subject: [PATCH] Bug 27600: Remove password check from SIP2 `renew_all` This patch removes the password check code from the renew_all method for SIP2. This is required, as many units send an empty AD field in the SIP message which can cause the transaction to fail.. but the authentication should have already taken place earlier in the session. Signed-off-by: Nick Clemens Signed-off-by: Jonathan Druart --- C4/SIP/ILS.pm | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/C4/SIP/ILS.pm b/C4/SIP/ILS.pm index 3e5b6a4824..a80052e033 100644 --- a/C4/SIP/ILS.pm +++ b/C4/SIP/ILS.pm @@ -511,15 +511,12 @@ sub renew_all { siplog("LOG_DEBUG", "ILS::renew_all: Invalid patron id: '%s'", $patron_id); } - if (!defined($patron)) { + if (!$patron) { $trans->screen_msg("Invalid patron barcode."); return $trans; } elsif (!$patron->renew_ok) { $trans->screen_msg("Renewals not allowed."); return $trans; - } elsif (defined($patron_pwd) && !$patron->check_password($patron_pwd)) { - $trans->screen_msg("Invalid patron password."); - return $trans; } $trans->do_renew_all; -- 2.39.5