gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 8178 - circ/circulation.pl under plack duplicates checkout rows 

Bug 7851 introduced our scoping for vairables, unfortunatly it has
side-effect that checkout rows accumulate on page reloads instead
of being initialized to empty array (which this patch fixes)

This also fixes %renew_failed initialization on each request.

Test scenario:
1. start intranet under plack
2. open /cgi-bin/koha/circ/circulation.pl and reload page few time
   confirming that rows gets duplicated
3. apply patch and reload page to verify that it works

Signed-off-by: Alex Arnaud <alex.arnaud@biblibre.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
This commit is contained in:
Dobrica Pavlinusic 2012-06-01 15:22:30 +02:00 committed by Chris Cormack
parent 4e10171129
commit 19c7e9905a
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
circ/circulation.pl
View File

@ -94,7 +94,7 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user (
my $branches = GetBranches();
my @failedrenews = $query->param('failedrenew'); # expected to be itemnumbers
our %renew_failed;
our %renew_failed = {};
for (@failedrenews) { $renew_failed{$_} = 1; }
my $findborrower = $query->param('findborrower');
@ -417,10 +417,10 @@ if ($borrowernumber) {
# make the issued books table.
my $todaysissues = '';
my $previssues = '';
our @todaysissues;
our @previousissues;
our @relissues;
our @relprevissues;
our @todaysissues = ();
our @previousissues = ();
our @relissues = ();
our @relprevissues = ();
my $displayrelissues;
our $totalprice = 0;