|
Lines 207-214
if ( $op eq 'delete_confirm' ) {
Link Here
|
| 207 |
} |
207 |
} |
| 208 |
#if the basket is closed,and the user has the permission to edit basketgroups, display a list of basketgroups |
208 |
#if the basket is closed,and the user has the permission to edit basketgroups, display a list of basketgroups |
| 209 |
my ($basketgroup, $basketgroups); |
209 |
my ($basketgroup, $basketgroups); |
| 210 |
my $member = GetMember(borrowernumber => $loggedinuser); |
210 |
# my $member = GetMember(borrowernumber => $loggedinuser); |
| 211 |
if ($basket->{closedate} && haspermission({ acquisition => 'group_manage'} )) { |
211 |
# FIXME $member is not used in current code (BZ 10258) |
|
|
212 |
# FIXME The following code should check the permission { acquisition => 'group_manage'} |
| 213 |
# Removed an invalid call to haspermission for now. No behavior change. |
| 214 |
# Only adding $member->{userid} is not the solution.. |
| 215 |
if ($basket->{closedate}) { |
| 212 |
$basketgroups = GetBasketgroups($basket->{booksellerid}); |
216 |
$basketgroups = GetBasketgroups($basket->{booksellerid}); |
| 213 |
for my $bg ( @{$basketgroups} ) { |
217 |
for my $bg ( @{$basketgroups} ) { |
| 214 |
if ($basket->{basketgroupid} && $basket->{basketgroupid} == $bg->{id}){ |
218 |
if ($basket->{basketgroupid} && $basket->{basketgroupid} == $bg->{id}){ |
| 215 |
- |
|
|