From 81ea0af7f13d115754109861d4b0e3e1e7c056c3 Mon Sep 17 00:00:00 2001
From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
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