Bug 15752: (QA follow-up) Remove unecessary redirect
authorJosef Moravec <josef.moravec@gmail.com>
Thu, 21 Dec 2017 19:17:20 +0000 (19:17 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 12 Feb 2018 20:38:58 +0000 (17:38 -0300)
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

index 3622aa6..aa98f9a 100755 (executable)
@@ -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;