From abc446ae53f9461629394f68fa7710fcdd397fd9 Mon Sep 17 00:00:00 2001 From: Julian Maurice Date: Tue, 19 Apr 2016 12:15:46 +0200 Subject: [PATCH] Bug 15565: Set 'found' parameter correctly when ReservesNeedReturns is off Signed-off-by: Danielle Elder --- reserve/placerequest.pl | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl index 298d0a3..3a454d8 100755 --- a/reserve/placerequest.pl +++ b/reserve/placerequest.pl @@ -70,13 +70,13 @@ foreach my $bibnum (@biblionumbers) { my @found; -foreach my $checkitem (@checkitems) { - # if we have an item selectionned, and the pickup branch is the same as the holdingbranch - # of the document, we force the value $rank and $found . - $rank[0] = '0' unless C4::Context->preference('ReservesNeedReturns'); - my $item = GetItem($checkitem); - if ( $item->{'holdingbranch'} eq $branch ){ - push @found, 'W' unless C4::Context->preference('ReservesNeedReturns'); +unless (C4::Context->preference('ReservesNeedReturns')) { + foreach my $checkitem (@checkitems) { + # if we have an item selectionned, and the pickup branch is the same as + # the holdingbranch of the document, we force the value $rank and $found + $rank[0] = 0; + my $item = GetItem($checkitem); + push @found, $item->{holdingbranch} eq $branch ? 'W' : undef; } } -- 2.1.4