From d00f3d122ce1e7dce1cf12ca24740427e12deba2 Mon Sep 17 00:00:00 2001 From: Josef Moravec Date: Thu, 21 Dec 2017 19:17:20 +0000 Subject: [PATCH] Bug 15752: Remove unecessary redirect (follow-up) If the borrowernumber is defined in query, it starts to get information about previous borrower and the new one is taken into account fully on second page reload --- circ/circulation.pl | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/circ/circulation.pl b/circ/circulation.pl index df9590b..7127612 100755 --- a/circ/circulation.pl +++ b/circ/circulation.pl @@ -86,11 +86,13 @@ my $barcodes = []; my $barcode = $query->param('barcode'); my $findborrower; my $autoswitched; +my $borrowernumber = $query->param('borrowernumber'); if (C4::Context->preference("AutoSwitchPatron") && $barcode) { if (Koha::Patrons->search( { cardnumber => $barcode} )->count() > 0) { $findborrower = $barcode; undef $barcode; + undef $borrowernumber; $autoswitched = 1; } } @@ -118,7 +120,6 @@ if ( $barcode || ( defined($barcode) && $barcode eq '0' ) ) { $barcodes = [ uniq @$barcodes ]; my $template_name = q|circ/circulation.tt|; -my $borrowernumber = $query->param('borrowernumber'); my $patron = $borrowernumber ? Koha::Patrons->find( $borrowernumber ) : undef; my $batch = $query->param('batch'); my $batch_allowed = 0; -- 2.1.4