From: senator <senator@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Thu, 17 Feb 2011 20:57:31 +0000 (+0000)
Subject: Backport r19470 from trunk
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=a44c99a0b6b1e5b24edab9a071b035a19c983b1d;p=evergreen%2Fequinox.git

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.



git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_2_0@19472 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 d978f06cdd..d7061f005f 100644
--- a/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
+++ b/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm
@@ -1586,6 +1586,14 @@ sub reservation_targeter {
                     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;