@@ -, +, @@ basket.pl --- acqui/basket.pl | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) --- a/acqui/basket.pl +++ a/acqui/basket.pl @@ -207,8 +207,12 @@ if ( $op eq 'delete_confirm' ) { } #if the basket is closed,and the user has the permission to edit basketgroups, display a list of basketgroups my ($basketgroup, $basketgroups); - my $member = GetMember(borrowernumber => $loggedinuser); - if ($basket->{closedate} && haspermission({ acquisition => 'group_manage'} )) { + # my $member = GetMember(borrowernumber => $loggedinuser); + # FIXME $member is not used in current code (BZ 10258) + # FIXME The following code should check the permission { acquisition => 'group_manage'} + # Removed an invalid call to haspermission for now. No behavior change. + # Only adding $member->{userid} is not the solution.. + if ($basket->{closedate}) { $basketgroups = GetBasketgroups($basket->{booksellerid}); for my $bg ( @{$basketgroups} ) { if ($basket->{basketgroupid} && $basket->{basketgroupid} == $bg->{id}){ --