Attachment 156295 Details for Bug 34886 – Bug 34886: Move patron check onto C4/Reserves.pm

[patch] Bug 34886: Move patron check onto C4/Reserves.pm

Bug-34886-Move-patron-check-onto-C4Reservespm.patch (text/plain), 2.60 KB, created by Pedro Amorim on 2023-09-27 14:14:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pedro Amorim

Created: 2023-09-27 14:14:22 UTC

Size: 2.60 KB

patch

obsolete

>From aefe604c2f907c28874435479b104d3891baa486 Mon Sep 17 00:00:00 2001
>From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
>Date: Mon, 25 Sep 2023 12:30:16 +0000
>Subject: [PATCH] Bug 34886: Move patron check onto C4/Reserves.pm
>
>Rollback changes in opac-detail.pl
>
>Run tests:
>prove t/db_dependent/Holds/DisallowHoldIfItemsAvailable.t
>prove t/db_dependent/Reserves.t
>
>Bug 15534 changed onshelfholds going from 0/1 circ rule to 0/1/2
>But the check in opac-detail.pl still remained checking for just a positive value.
>Currently, onshelfholds is as follows:
>1 - Yes
>0 - If any unavailable
>2 - If all unavailable
>
>I think the check needs to be updated to only when onshelfholds = 1 (Yes), which is effectively done in IsAvailableForItemLevelRequest in C4/Reserves.pm, with the caveat that when onshelfholds = 2, it requires the $patron to exist to do the full check.
>
>This patch fixes the issue. But also makes it so that the "Place hold" link only shows for the unauthenticated user if onshelfholds = "Yes", whereas before it showed for the unauthenticated user if onshelfholds = "Yes" OR if onshelfholds = "If all unavailable"
>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> C4/Reserves.pm      | 2 +-
> opac/opac-detail.pl | 7 +------
> 2 files changed, 2 insertions(+), 7 deletions(-)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index 3625737885..e07a46b500 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -1369,7 +1369,7 @@ sub IsAvailableForItemLevelRequest {
> 
>     if ( $on_shelf_holds == 1 ) {
>         return 1;
>-    } elsif ( $on_shelf_holds == 2 ) {
>+    } elsif ( $on_shelf_holds == 2 && $patron) {
> 
>         # These calculations work at the biblio level, and can be expensive
>         # we use the in-memory cache to avoid calling once per item when looping items on a biblio
>diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
>index 59eeedcfe1..7191625a31 100755
>--- a/opac/opac-detail.pl
>+++ b/opac/opac-detail.pl
>@@ -682,12 +682,7 @@ else {
>         $item_info->{holding_library_info} = $opac_info_holding->content if $opac_info_holding;
>         $item_info->{home_library_info} = $opac_info_home->content if $opac_info_home;
> 
>-        if ( $patron ) {
>-            $has_reservable_items = IsAvailableForItemLevelRequest($item, $patron, undef);
>-        } else {
>-            $has_reservable_items =
>-                Koha::CirculationRules->get_onshelfholds_policy( { item => $item, patron => undef } );
>-        }
>+        $has_reservable_items = $has_reservable_items || IsAvailableForItemLevelRequest ( $item, $patron, undef );
> 
>         # get collection code description, too
>         my $ccode = $item->ccode;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34886: 156107 | 156114 | 156150 | 156151 | 156152 | 156153 | 156277 | 156278 | 156279 | 156293 | 156294 | 156295 | 156326 | 156327 | 156980 | 156981 | 159678 | 159679 | 161633 | 161634 | 161981 | 161982 | 163674 | 164106 | 164107 | 164108 | 164109 | 164110 | 164159 | 164160 | 164161