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

(-)a/C4/Reserves.pm (-3 / +2 lines)
Lines 1220-1227 sub IsAvailableForItemLevelRequest { Link Here
1220
        $item->withdrawn        ||
1220
        $item->withdrawn        ||
1221
        ($item->damaged && !C4::Context->preference('AllowHoldsOnDamagedItems'));
1221
        ($item->damaged && !C4::Context->preference('AllowHoldsOnDamagedItems'));
1222
1222
1223
    my $on_shelf_holds = Koha::IssuingRules->get_onshelfholds_policy( { item => $item, patron => $patron } );
1224
1225
    if ($pickup_branchcode) {
1223
    if ($pickup_branchcode) {
1226
        my $destination = Koha::Libraries->find($pickup_branchcode);
1224
        my $destination = Koha::Libraries->find($pickup_branchcode);
1227
        return 0 unless $destination;
1225
        return 0 unless $destination;
Lines 1235-1240 sub IsAvailableForItemLevelRequest { Link Here
1235
        return 0 unless $branchitemrule->{hold_fulfillment_policy} ne 'holdgroup' || $home_library->validate_hold_sibling( {branchcode => $pickup_branchcode} );
1233
        return 0 unless $branchitemrule->{hold_fulfillment_policy} ne 'holdgroup' || $home_library->validate_hold_sibling( {branchcode => $pickup_branchcode} );
1236
    }
1234
    }
1237
1235
1236
    my $on_shelf_holds = Koha::IssuingRules->get_onshelfholds_policy( { item => $item, patron => $patron } );
1237
1238
    if ( $on_shelf_holds == 1 ) {
1238
    if ( $on_shelf_holds == 1 ) {
1239
        return 1;
1239
        return 1;
1240
    } elsif ( $on_shelf_holds == 2 ) {
1240
    } elsif ( $on_shelf_holds == 2 ) {
1241
- 

Return to bug 24185