From: miker <miker@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Mon, 29 Nov 2010 19:47:20 +0000 (+0000)
Subject: logic error (inversion) in retargetting of previous copy when no other holds are... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2523cc41056ea0f53810adf317ce1a742b921108;p=working%2FEvergreen.git

logic error (inversion) in retargetting of previous copy when no other holds are looking at it

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_0@18859 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm b/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
index 85c726c991..47e743c092 100644
--- a/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
+++ b/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
@@ -1442,7 +1442,7 @@ sub new_hold_copy_targeter {
 				$log->debug("\tUpdating hold [".$hold->id."] with new 'current_copy' [".$best->id."] for hold fulfillment.");
 			} elsif (
 				$old_best &&
-				action::hold_request
+				!action::hold_request
 					->search_where(
 						{ current_copy => $old_best->id,
 						  fulfillment_time => undef,