biblionumber => $item->biblionumber,
}
);
- if ( $holds->count() >= $holds_per_record ) {
+ if ( defined $holds_per_record && $holds_per_record ne ''
+ && $holds->count() >= $holds_per_record ) {
return { status => "tooManyHoldsForThisRecord", limit => $holds_per_record };
}
reservedate => dt_from_string->date
});
- if ( defined $holds_per_day &&
- ( ( $holds_per_day > 0 && $today_holds->count() >= $holds_per_day )
- or ( $holds_per_day == 0 ) )
- ) {
+ if ( defined $holds_per_day && $holds_per_day ne ''
+ && $today_holds->count() >= $holds_per_day )
+ {
return { status => 'tooManyReservesToday', limit => $holds_per_day };
}
}
# we check if it's ok or not
- if ( $reservecount >= $allowedreserves ) {
+ if ( defined $allowedreserves && $allowedreserves ne ''
+ && $reservecount >= $allowedreserves ) {
return { status => 'tooManyReserves', limit => $allowedreserves };
}
my $holds_per_record = $input->param('holds_per_record');
my $holds_per_day = $input->param('holds_per_day');
$holds_per_day =~ s/\s//g;
- $holds_per_day = undef if $holds_per_day !~ /^\d+/;
+ $holds_per_day = '' if $holds_per_day !~ /^\d+/;
my $onshelfholds = $input->param('onshelfholds') || 0;
$maxissueqty =~ s/\s//g;
$maxissueqty = '' if $maxissueqty !~ /^\d+/;