From f9507ccc7492be9b22e04b099d226457fe0d8c3b Mon Sep 17 00:00:00 2001 From: Lebbeous Fogle-Weekley Date: Fri, 19 Apr 2013 15:23:05 -0400 Subject: [PATCH] Repair too-timid hold targeter (it misses copies at other org units) The patch to address bug 1162989 overreached. Whereas before that patch, the hold targeter was wont to target copies with unholdable statuses, after the patch the hold targeter got the statuses right but undesirably filtered out copies whose circ_lib doesn't match the hold pickup lib. This again probably stems from the differences between FulfILLment's hold targeter and Evergreen's in the context of merging the calculated proximity code. This patch should resolve the issue. Signed-off-by: Lebbeous Fogle-Weekley Signed-off-by: Kathy Lussier --- .../src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm index baa7098e45..24369062c6 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Storage/Publisher/action.pm @@ -1580,7 +1580,7 @@ sub new_hold_copy_targeter { # reset prox list after trimming good copies $prox_list = create_prox_list( $self, $pu_lib, - [ grep { ''.$_->circ_lib eq $pu_lib && ( $_->status == 0 || $_->status == 7 ) } @good_copies ], + [ grep { $_->status == 0 || $_->status == 7 } @good_copies ], $hold ); -- 2.11.0