From ad5859cfeb212cc952a2321307eec5da70017a68 Mon Sep 17 00:00:00 2001
From: Julian Maurice <julian.maurice@biblibre.com>
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 <danielle@bywatersolution.com>
---
 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.7.4