[open-ils-commits] r19474 - branches/rel_1_6_1/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher (senator)

[svn at svn.open-ils.org]

Author: senator
Date: 2011-02-17 15:58:28 -0500 (Thu, 17 Feb 2011)
New Revision: 19474

Modified:
   branches/rel_1_6_1/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
Log:
Backport r19470 from trunk

Booking: fix the targeter in the I'm-reserving-this-exact-resource-with-this-
exact-barcode case.

It should have always worked like this. This should better mirror the behavior
in the holds targeter when placing a copy-level hold.  Basically don't fail
to target a resource for a reservation later just because it's not in an
available status right at this moment.



Modified: branches/rel_1_6_1/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
===================================================================
--- branches/rel_1_6_1/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm	2011-02-17 20:58:04 UTC (rev 19473)
+++ branches/rel_1_6_1/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm	2011-02-17 20:58:28 UTC (rev 19474)
@@ -1351,6 +1351,14 @@
                     next;
                 }
 
+                # At this point, if we're just targeting one specific
+                # resource, just succeed. We don't care about its present
+                # copy status.
+                if ($bresv->target_resource) {
+                    push @good_resources, $res;
+                    next;
+                }
+
                 if ($copy->status->id == 0 || $copy->status->id == 7) {
                     push @good_resources, $res;
                     next;