View | Details | Raw Unified | Return to bug 10258
Collapse All | Expand All

(-)a/acqui/basket.pl (-3 / +6 lines)
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
- 

Return to bug 10258