From c611a1d5ed59c8c6acf51e31f3bc7ef91f9d1d21 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Thu, 8 Aug 2024 11:18:43 +0000 Subject: [PATCH] Bug 32702: (follow-up) Add CanItemBeReserved checks to opac scripts --- opac/opac-ISBDdetail.pl | 4 +++- opac/opac-MARCdetail.pl | 4 +++- opac/opac-detail.pl | 6 ++++-- 3 files changed, 10 insertions(+), 4 deletions(-) diff --git a/opac/opac-ISBDdetail.pl b/opac/opac-ISBDdetail.pl index 0e9b5787ce2..4bf0bc08cb0 100755 --- a/opac/opac-ISBDdetail.pl +++ b/opac/opac-ISBDdetail.pl @@ -216,7 +216,9 @@ my $can_holds_be_placed = $patron ? 0 : $holdable_items; if ($patron) { my $items = $biblio->items; while ( my $item = $items->next ) { - $can_holds_be_placed = $can_holds_be_placed || IsAvailableForItemLevelRequest( $item, $patron, undef ); + $can_holds_be_placed = $can_holds_be_placed + || ( IsAvailableForItemLevelRequest( $item, $patron, undef ) + && CanItemBeReserved( $patron, $item, undef ) ); } } diff --git a/opac/opac-MARCdetail.pl b/opac/opac-MARCdetail.pl index f4a4198459e..5af49becf3a 100755 --- a/opac/opac-MARCdetail.pl +++ b/opac/opac-MARCdetail.pl @@ -177,7 +177,9 @@ my $can_holds_be_placed = $patron ? 0 : $holdable_items; if ($patron) { $items->reset; while ( my $item = $items->next ) { - $can_holds_be_placed = $can_holds_be_placed || IsAvailableForItemLevelRequest( $item, $patron, undef ); + $can_holds_be_placed = $can_holds_be_placed + || ( IsAvailableForItemLevelRequest( $item, $patron, undef ) + && CanItemBeReserved( $patron, $item, undef ) ); } } diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl index 9cd2364f849..45c8f2ba75e 100755 --- a/opac/opac-detail.pl +++ b/opac/opac-detail.pl @@ -746,8 +746,10 @@ if ( not $viewallitems and $items->count > $max_items_to_display ) { # We only need to check if we haven't determined holds can be placed # and if we don't have patron, we have already decided - $can_holds_be_placed = - $can_holds_be_placed || $patron && IsAvailableForItemLevelRequest( $item, $patron, undef ); + $can_holds_be_placed = $can_holds_be_placed + || ( $patron + && IsAvailableForItemLevelRequest( $item, $patron, undef ) + && CanItemBeReserved( $patron, $item, undef ) ); # get collection code description, too my $ccode = $item->ccode; -- 2.39.5