From 06ea76c3c2c9afcc324166c12b0510fdad0d77f1 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Wed, 9 Oct 2013 14:02:23 -0400 Subject: [PATCH] Bug 10243: (follow-up) prevent TransportCostMatrix from overriding library holds policy 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. Signed-off-by: Galen Charlton --- C4/HoldsQueue.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/C4/HoldsQueue.pm b/C4/HoldsQueue.pm index fa6678fcf9..7062cbe6af 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"; -- 2.39.5