From d00f3d122ce1e7dce1cf12ca24740427e12deba2 Mon Sep 17 00:00:00 2001
From: Josef Moravec <josef.moravec@gmail.com>
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