From ef23baed3dc9b1c4e09aa7d9d013ca726fcaf767 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.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
---
 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.20.1