}
# Remove any items that are not holdable for this patron
- @items = grep { CanItemBeReserved( $borrower->{borrowernumber}, $_->itemnumber ) eq 'OK' } @items;
+ @items = grep { CanItemBeReserved( $borrower->{borrowernumber}, $_->itemnumber )->{status} eq 'OK' } @items;
my $items_count = scalar @items;
{ status => ageRestricted }, if the Item is age restricted for this borrower.
{ status => damaged }, if the Item is damaged.
{ status => cannotReserveFromOtherBranches }, if syspref 'canreservefromotherbranches' is OK.
- { status => tooManyReserves, limit => $limit }, if the borrower has exceeded his maximum reserve amount.
+ { status => tooManyReserves, limit => $limit }, if the borrower has exceeded their maximum reserve amount.
{ status => notReservable }, if holds on this item are not allowed
=cut
unless ($can_reserve->{status} eq 'OK') {
return $c->render( status => 403, openapi => {
- error => "Reserve cannot be placed. Reason: $can_reserve"
+ error => "Reserve cannot be placed. Reason: ". $can_reserve->{status}
} );
}
<td>[% IF ( branch_cat_rule_loo.unlimited_max_holds ) %]
Unlimited
[% ELSE %]
- [% branch_cat_rule_loo.max_holds %]
+ [% branch_cat_rule_loo.max_holds | html %]
[% END %]
</td>
!$item->{cantreserve}
&& !$exceeded_maxreserves
&& IsAvailableForItemLevelRequest($item, $patron_unblessed)
- && $can_item_be_reserved eq 'OK'
+ && $can_item_be_reserved->{status} eq 'OK'
)
{
$item->{available} = 1;
push( @available_itemtypes, $item->{itype} );
}
- elsif ( $can_item_be_reserved eq 'tooManyReserves' && C4::Context->preference('AllowHoldPolicyOverride') ) {
+ elsif ( $can_item_be_reserved->{status} eq 'tooManyReserves' && C4::Context->preference('AllowHoldPolicyOverride') ) {
# If AllowHoldPolicyOverride is set, it should override EVERY restriction, not just branch item rules
$item->{override} = 1;
$num_override++;