From b2d052129d19a9b386701f155968cf936efb5413 Mon Sep 17 00:00:00 2001 From: Kyle Hall Date: Wed, 13 Oct 2021 12:20:33 -0400 Subject: [PATCH] Bug 29196: Follow-up to Bug 27068 - Remove unnecessary check The introduction of _checkHoldPolicy has made this check superfluous. Test Plan: 1) Apply this patch 2) prove t/db_dependent/HoldsQueue.t --- C4/HoldsQueue.pm | 4 ---- 1 file changed, 4 deletions(-) diff --git a/C4/HoldsQueue.pm b/C4/HoldsQueue.pm index b4d2b026f28..e22d380747d 100644 --- a/C4/HoldsQueue.pm +++ b/C4/HoldsQueue.pm @@ -615,10 +615,6 @@ sub MapItemsToHoldRequests { next unless _checkHoldPolicy($item, $request); next unless $items_by_itemnumber{ $item->{itemnumber} }->{_object}->can_be_transferred( { to => $libraries->{ $request->{branchcode} } } ); - # Don't fill item level holds that contravene the hold pickup policy at this time - next unless $item->{hold_fulfillment_policy} eq 'any' - || $request->{branchcode} eq $item->{ $item->{hold_fulfillment_policy} }; - # If hold itemtype is set, item's itemtype must match next unless ( !$request->{itemtype} || $item->{itype} eq $request->{itemtype} ); -- 2.30.2