From ef23baed3dc9b1c4e09aa7d9d013ca726fcaf767 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 Signed-off-by: Jonathan Druart --- 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.39.5