From 3fd0378caacc4689bd65c5b4969e734695aa9ead Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 9 Oct 2013 14:02:23 -0400 Subject: [PATCH] Bug 10243 - Holds queue is making transfer requests that contradict the branch holds policy - QA Followup Using the TransportCostMatrix can cause the same issue. Removing the last ditch use of the first item causes the the subroutine to continue with the traditional matching, which will respect the hold policies. --- C4/HoldsQueue.pm | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/C4/HoldsQueue.pm b/C4/HoldsQueue.pm index fa6678f..7062cbe 100755 --- a/C4/HoldsQueue.pm +++ b/C4/HoldsQueue.pm @@ -448,7 +448,6 @@ sub MapItemsToHoldRequests { $itemnumber = $item->{itemnumber}; last; } - $itemnumber ||= $holding_branch_items->[0]->{itemnumber}; } else { warn "No transport costs for $pickup_branch"; -- 1.7.2.5