Attachment 60542 Details for Bug 18150 – Bug 18150 - CanItemBeReserved doesn't work with (IndependentBranches AND ! canreservefromotherbranches)

[patch] Bug 18150 - CanItemBeReserved doesn't work with (IndependentBranches AND ! canreservefromotherbranches)

Bug-18150---CanItemBeReserved-doesnt-work-with-Ind.patch (text/plain), 1.47 KB, created by Nicolas Legrand on 2017-02-22 09:33:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicolas Legrand

Created: 2017-02-22 09:33:12 UTC

Size: 1.47 KB

patch

obsolete

>From 10d8ccce587f14fd84c029d297413f0b55f9dbaa Mon Sep 17 00:00:00 2001
>From: Nicolas Legrand <nicolas.legrand@bulac.fr>
>Date: Wed, 22 Feb 2017 10:11:18 +0100
>Subject: [PATCH] Bug 18150 - CanItemBeReserved doesn't work with
> (IndependentBranches AND ! canreservefromotherbranches)
>
>The variable $item used to be a hash, but at the end of the function,
>it's a Koha object. As $item->{homebranch} doesn't yield anything and
>should be $item->homebranch. It prevents people using different
>branches without holds between branches from placing a hold on an item
>they should be able to place hold on.
>
>Test plan:
>
>1. Before patch
>
>  a. with IndependantBranches off
>  b. try to place hold on an item you should be able to place hold on
>  c. it should work
>  d. put IndependantBranches on and canreservefromotherbranches off
>  e. shouldn't work
>
>2. after patches redo steps from (1) and everything should be working
>fine.
>---
> C4/Reserves.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index c078af1..8c4175d 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -581,7 +581,7 @@ sub CanItemBeReserved {
>     if ( C4::Context->preference('IndependentBranches')
>         and !C4::Context->preference('canreservefromotherbranches') )
>     {
>-        my $itembranch = $item->{homebranch};
>+        my $itembranch = $item->homebranch;
>         if ( $itembranch ne $borrower->{branchcode} ) {
>             return 'cannotReserveFromOtherBranches';
>         }
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 18150: 60542 | 60853 | 60854 | 60901