[open-ils-commits] [GIT] Evergreen ILS branch master updated. fa8bdbecb4ebb7662e17aa020ee46907ea76cb04

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, master has been updated
       via  fa8bdbecb4ebb7662e17aa020ee46907ea76cb04 (commit)
      from  d46585ed0e2816cd641242bfb292f0869f01950c (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit fa8bdbecb4ebb7662e17aa020ee46907ea76cb04
Author: Lebbeous Fogle-Weekley <lebbeous at esilibrary.com>
Date:   Fri Apr 19 15:23:05 2013 -0400

    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 <lebbeous at esilibrary.com>
    Signed-off-by: Kathy Lussier <klussier at masslnc.org>
    Signed-off-by: Mike Rylander <mrylander at gmail.com>

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 f18a254..ed5c4ad 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
@@ -1615,7 +1615,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
 			);
 

-----------------------------------------------------------------------

Summary of changes:
 .../Application/Storage/Publisher/action.pm        |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS