From 7153f6f66dc8ee2a79aee4acf6e295c994828879 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. Signed-off-by: Kyle M Hall --- acqui/basketgroup.pl | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/acqui/basketgroup.pl b/acqui/basketgroup.pl index fa89f79d4f..69cf5b0378 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.24.3 (Apple Git-128)