From ff2b9b63b46e035caa7b96dbf1b7a4f1435e7c6f Mon Sep 17 00:00:00 2001 From: Jose Martin Date: Wed, 7 Mar 2018 15:21:39 +0300 Subject: [PATCH] Bug 20348: SIP2 patron identification fails to use userid Replaces "or" with "||" in variable assignment second attempt to retrieve borrower was not being executed due to changed precedence Signed-off-by: Colin Campbell Signed-off-by: Josef Moravec Signed-off-by: Jonathan Druart Signed-off-by: Nick Clemens --- C4/SIP/ILS/Patron.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/SIP/ILS/Patron.pm b/C4/SIP/ILS/Patron.pm index 6cd3300eee..c3ea43e210 100644 --- a/C4/SIP/ILS/Patron.pm +++ b/C4/SIP/ILS/Patron.pm @@ -33,7 +33,7 @@ sub new { my $type = ref($class) || $class; my $self; $kp = Koha::Patrons->find( { cardnumber => $patron_id } ) - or Koha::Patrons->find( { userid => $patron_id } ); + || Koha::Patrons->find( { userid => $patron_id } ); $debug and warn "new Patron: " . Dumper($kp->unblessed) if $kp; unless ($kp) { syslog("LOG_DEBUG", "new ILS::Patron(%s): no such patron", $patron_id); -- 2.39.5