From 28554c1a70bab65950960e1596a610772df8f064 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Mon, 26 Jun 2017 15:25:19 -0300 Subject: [PATCH] Bug 17554: (followup) Shibboleth check should use ->find too There was a remaining use of C4::Members::GetBorrowersWithEmail in Auth.pm. Signed-off-by: Tomas Cohen Arazi --- C4/Auth.pm | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/C4/Auth.pm b/C4/Auth.pm index 549ece9..e7b0895 100644 --- a/C4/Auth.pm +++ b/C4/Auth.pm @@ -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; } -- 2.7.4