Attachment 137181 Details for Bug 31105 – Bug 31105: Holds to pull counts items from other branches when independentbranches is active

[patch] Bug 31105: Holds to pull counts items from other branches when independentbranches is active

Bug-31105-Holds-to-pull-counts-items-from-other-br.patch (text/plain), 2.52 KB, created by Shi Yao Wang on 2022-07-05 17:46:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Shi Yao Wang

Created: 2022-07-05 17:46:57 UTC

Size: 2.52 KB

patch

obsolete

>From d705beebe9c51d7732d5ae681f29493aa1b347da Mon Sep 17 00:00:00 2001
>From: Shi Yao Wang <shi-yao.wang@inlibro.com>
>Date: Tue, 5 Jul 2022 12:03:52 -0400
>Subject: [PATCH] Bug 31105: Holds to pull counts items from other branches
> when independentbranches is active
>
>Holds to pull should not show items from other branches that shares a
>record when IndependentBranches is active. Except for superlibrarians.
>
>Test plan:
>1- Have IndependentBranches set to Yes
>2- Have a record with an item from the current branch and another item from a different branch. We will call them branch A and branch B respectively
>3- Have a patron from branch A place a hold on that record
>4- Have a staff account on branch A and B with the following permissions:
>    - circulate (all)
>    - catalogue
>5- Go to circulation > holds to pull
>6- Notice under "Libraries" it displays branch A and B. Also, it shows 2 under "Items available"
>7- Checkout the item from branch A to another patron
>8- Go back to circulation > holds to pull
>9- Notice under "Libraries" it displays branch B even though we are in branch A
>10- Apply the patch
>11- Return the checked out item from step 7
>12- Redo step 5 to 9 with superlibrarian privileges and notice nothing changed
>13- Redo step 11-12 but using accounts from step 4 and notice Holds to
>pull only shows information relevant to their branch
>---
> Koha/Items.pm           | 1 +
> circ/pendingreserves.pl | 4 ++--
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/Koha/Items.pm b/Koha/Items.pm
>index 90b178a1ce..a5dff5aff7 100644
>--- a/Koha/Items.pm
>+++ b/Koha/Items.pm
>@@ -76,6 +76,7 @@ sub filter_by_for_hold {
>         withdrawn  => 0,
>         notforloan => { '<=' => 0 },    # items with negative or zero notforloan value are holdable
>         ( C4::Context->preference('AllowHoldsOnDamagedItems')? (): ( damaged => 0 ) ),
>+        ( C4::Context->only_my_library() ? ( homebranch => C4::Context::mybranch() ) : () ),
>     };
> 
>     if ( C4::Context->preference("item-level_itypes") ) {
>diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl
>index 1f853554b5..0563fd550b 100755
>--- a/circ/pendingreserves.pl
>+++ b/circ/pendingreserves.pl
>@@ -181,8 +181,8 @@ if ( !C4::Context->preference('AllowHoldsOnDamagedItems') ){
>     $where{'itembib.damaged'} = 0;
> }
> 
>-if ( C4::Context->preference('IndependentBranches') ){
>-    $where{'itembib.holdingbranch'} = C4::Context->userenv->{'branch'};
>+if ( C4::Context->only_my_library() ){
>+    $where{'me.branchcode'} = C4::Context->userenv->{'branch'};
> }
> 
> # get all distinct unfulfilled reserves
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 31105: 137181 | 137859 | 139496 | 140902