From 91cc49ce928e5d70974b0d4e9bacb16cd121924b Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 15 Sep 2015 08:24:07 +0100 Subject: [PATCH] Bug 14821: Do not list not returned items On the checkin page, the table is always filled with info of the scanned barcode, even if the checkin has failed. When the checkin fails, there is already an error displayed, so there is no need to keep a row for fails. Test plan: Confirm that the failed checkins don't appear anymore in the checkin table. - checkin an item which is not issued; - BlockReturnOfWithdrawnItems is set and item is withdrawn; - Not returned at the right place; - privacy=2 and AnonymousPatron not correctly set; Before this patch this behavior existed only for bad barcode. --- circ/returns.pl | 6 ------ 1 file changed, 6 deletions(-) diff --git a/circ/returns.pl b/circ/returns.pl index ec0679f..19dab4d 100755 --- a/circ/returns.pl +++ b/circ/returns.pl @@ -345,12 +345,6 @@ if ($barcode) { } } } - elsif ( !$messages->{'BadBarcode'} ) { - $input{duedate} = 0; - $returneditems{0} = $barcode; - $riduedate{0} = 0; - push( @inputloop, \%input ); - } $template->param( privacy => $borrower->{privacy} ); } $template->param( inputloop => \@inputloop ); -- 2.1.0