@@ -, +, @@ --- C4/Circulation.pm | 2 +- C4/Reserves.pm | 2 +- Koha/REST/V1/Hold.pm | 2 +- koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt | 2 +- reserve/request.pl | 4 ++-- 5 files changed, 6 insertions(+), 6 deletions(-) --- a/C4/Circulation.pm +++ a/C4/Circulation.pm @@ -1183,7 +1183,7 @@ sub checkHighHolds { } # 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; --- a/C4/Reserves.pm +++ a/C4/Reserves.pm @@ -299,7 +299,7 @@ sub CanBookBeReserved{ { 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 --- a/Koha/REST/V1/Hold.pm +++ a/Koha/REST/V1/Hold.pm @@ -91,7 +91,7 @@ sub add { 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} } ); } --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt @@ -515,7 +515,7 @@