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

(-)a/C4/Reserves.pm (-7 / +7 lines)
Lines 393-399 sub CanItemBeReserved { Link Here
393
            biblionumber   => $item->biblionumber,
393
            biblionumber   => $item->biblionumber,
394
        }
394
        }
395
    );
395
    );
396
    if ( $holds->count() >= $holds_per_record ) {
396
    if (   defined $holds_per_record && $holds_per_record ne ''
397
        && $holds->count() >= $holds_per_record ) {
397
        return { status => "tooManyHoldsForThisRecord", limit => $holds_per_record };
398
        return { status => "tooManyHoldsForThisRecord", limit => $holds_per_record };
398
    }
399
    }
399
400
Lines 402-411 sub CanItemBeReserved { Link Here
402
        reservedate    => dt_from_string->date
403
        reservedate    => dt_from_string->date
403
    });
404
    });
404
405
405
    if ( defined $holds_per_day &&
406
    if (   defined $holds_per_day && $holds_per_day ne ''
406
          (   ( $holds_per_day > 0 && $today_holds->count() >= $holds_per_day )
407
        && $today_holds->count() >= $holds_per_day )
407
           or ( $holds_per_day == 0 && $holds_per_day ne '') )
408
    {
408
        )  {
409
        return { status => 'tooManyReservesToday', limit => $holds_per_day };
409
        return { status => 'tooManyReservesToday', limit => $holds_per_day };
410
    }
410
    }
411
411
Lines 436-442 sub CanItemBeReserved { Link Here
436
    }
436
    }
437
437
438
    # we check if it's ok or not
438
    # we check if it's ok or not
439
    if ( $reservecount >= $allowedreserves ) {
439
    if (   defined  $allowedreserves && $allowedreserves ne ''
440
        && $reservecount >= $allowedreserves ) {
440
        return { status => 'tooManyReserves', limit => $allowedreserves };
441
        return { status => 'tooManyReserves', limit => $allowedreserves };
441
    }
442
    }
442
443
443
- 

Return to bug 18936