From a723e0c8a0684fce65b0b18fbfdcb4f5ad2f0e25 Mon Sep 17 00:00:00 2001 From: Andrew Isherwood Date: Tue, 5 Jan 2021 11:34:28 +0000 Subject: [PATCH] Bug 23971: (follow-up) Remove stale param As highlighted by Jonathan in comment #90, we were passing borrowernumber to ModBasket. This was a hangover from when we explicitly logged the borrower, which was later discovered to be unnecessary duplication, and was removed in commit "Remove data duplication". This commit removes this unnecessary parameter. --- acqui/basketgroup.pl | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/acqui/basketgroup.pl b/acqui/basketgroup.pl index 962296aa6e..3b53bd9602 100755 --- a/acqui/basketgroup.pl +++ b/acqui/basketgroup.pl @@ -307,8 +307,7 @@ if ( $op eq "add" ) { my $basketno=$input->param('basketno'); my $basketgroupid=$input->param('basketgroupid'); ModBasket( { basketno => $basketno, - basketgroupid => $basketgroupid, - borrowernumber => $loggedinuser } ); + basketgroupid => $basketgroupid } ); print $input->redirect("basket.pl?basketno=" . $basketno); } elsif ( $op eq 'closeandprint') { # -- 2.20.1