From 2112a7e64db186e401cb3d419cf3251e8c410576 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 --- reserve/placerequest.pl | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl index bda0d81..8bd9c3f 100755 --- a/reserve/placerequest.pl +++ b/reserve/placerequest.pl @@ -68,13 +68,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