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

Bug 21771: (follow-up) Revert the shift statement

Browse source 
Obviously you should never shift an items from an array if you need that
item later on :)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tested with entering userid as well as entering email..

Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This commit is contained in:
Marcel de Rooy 2018-11-06 15:02:47 +01:00 committed by Nick Clemens
parent 143d738dab
commit 1885e6a214
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
opac/opac-password-recovery.pl
View file

@ -75,7 +75,8 @@ if ( $query->param('sendEmail') || $query->param('resendEmail') ) {
elsif ( $borrower = $search_results->next() ) { # One matching borrower
my @emails = grep { $_ } ( $borrower->email, $borrower->emailpro, $borrower->B_email );
my $firstNonEmptyEmail = shift @emails;
my $firstNonEmptyEmail;
$firstNonEmptyEmail = $emails[0] if @emails;
# Is the given email one of the borrower's ?
if ( $email && !( grep /^$email$/i, @emails ) ) {