From 09a72046f4c6abc4826db3df5170be72cbb66c4e 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: (QA follow-up) Remove unecessary redirect

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 circ/circulation.pl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/circ/circulation.pl b/circ/circulation.pl
index 3622aa6763..aa98f9af1f 100755
--- a/circ/circulation.pl
+++ b/circ/circulation.pl
@@ -85,11 +85,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;
     }
 }
@@ -117,7 +119,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.39.5