Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 17554: (followup) Shibboleth check should use ->find too

Browse source 
There was a remaining use of C4::Members::GetBorrowersWithEmail in Auth.pm.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Tomás Cohen Arazi 2017-06-26 15:25:19 -03:00 committed by Jonathan Druart
parent 69ddc09434
commit 60e55c857c
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
C4/Auth.pm
View file

@ -965,12 +965,12 @@ sub checkauth {
# doesn't have a userid. So if there is none, we pass along the
# borrower number, and the bits of code that need to know the user
# ID will have to be smart enough to handle that.
require C4::Members;
my @users_info = C4::Members::GetBorrowersWithEmail($value);
if (@users_info) {
my $patrons = Koha::Patrons->search({ email => $value });
if ($patrons->count) {
# First the userid, then the borrowernum
$value = $users_info[0][1] || $users_info[0][0];
my $patron = $patrons->next;
$value = $patron->userid || $patron->borrowernumber;
} else {
undef $value;
}