[open-ils-commits] r9002 - trunk/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher

[svn at svn.open-ils.org]

Author: miker
Date: 2008-03-13 12:38:43 -0400 (Thu, 13 Mar 2008)
New Revision: 9002

Modified:
   trunk/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
Log:
making old_best tests more explicit

Modified: trunk/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
===================================================================
--- trunk/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm	2008-03-13 15:57:51 UTC (rev 9001)
+++ trunk/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm	2008-03-13 16:38:43 UTC (rev 9002)
@@ -951,11 +951,7 @@
 	
 			if (!scalar(@good_copies)) {
 				$log->info("\tNo (non-current) copies eligible to fill the hold.");
-				if (
-				  $old_best &&
-				  grep { $old_best eq $_ } @$all_copies &&
-				  !action::hold_request->search_where({ current_copy => $old_best->id, capture_time => undef, cancel_time => undef })
-				) {
+				if ( $old_best && grep { ''.$old_best->id eq ''.$_->id } @$all_copies ) {
 					# the old copy is still available
 					$log->debug("\tPushing current_copy back onto the targeting list");
 					push @good_copies, $old_best;