From ff2b9b63b46e035caa7b96dbf1b7a4f1435e7c6f Mon Sep 17 00:00:00 2001
From: Jose Martin <jose.i.martin.cuesta@gmail.com>
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 <colin.campbell@ptfs-europe.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
---
 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