From 0bf436a0eadf0e89f0513409b6a03e05e7254f18 Mon Sep 17 00:00:00 2001 From: paul Date: Tue, 1 Apr 2008 16:01:36 +0200 Subject: [PATCH] use HomeOrHoldingBranch to determine if a return is valid or not in fact, i'm not 100% sure of this bugfix : I don't see the case where a return should be done directly to homebranch, when the check out has been done at holdingbranch so maybe we could hardcode 'holdingbranch' instead of using HomeOrHoldingBranch the patch as written won't break anything existing. Signed-off-by: Joshua Ferraro --- C4/Circulation.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 7ff88a3856..9586fc53d9 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -1230,7 +1230,7 @@ sub AddReturn { } # check if the book is in a permanent collection.... - my $hbr = $iteminformation->{'homebranch'}; + my $hbr = $iteminformation->{C4::Context->preference("HomeOrHoldingBranch")}; my $branches = GetBranches(); if ( $hbr && $branches->{$hbr}->{'PE'} ) { $messages->{'IsPermanent'} = $hbr; -- 2.39.2