https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20348
Bug ID: 20348
Summary: SIP2 patron identification fails to use userid
Change sponsored?: ---
Product: Koha
Version: 17.11
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5 - low
Component: SIP2
Assignee: koha-bugs@lists.koha-community.org
Reporter: jose.i.martin.cue...@gmail.com
QA Contact: testo...@bugs.koha-community.org
CC: colin.campb...@ptfs-europe.com
The following code in C4/SIP/ILS/Patron.pm:
$kp = Koha::Patrons->find( { cardnumber => $patron_id } )
or Koha::Patrons->find( { userid => $patron_id } );
should be (using || instead of or):
$kp = Koha::Patrons->find( { cardnumber => $patron_id } )
|| Koha::Patrons->find( { userid => $patron_id } );
It will never try to match a patron by userid.
This is a bug according to Perl documentation here:
https://perldoc.perl.org/perlop.html#Logical-or-and-Exclusive-Or
--
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
