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