From 868b2467320b9c1cbb76e27eab5428133ff9e1f6 Mon Sep 17 00:00:00 2001 From: senator Date: Thu, 17 Feb 2011 20:58:28 +0000 Subject: [PATCH] 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_1_6_1@19474 dcc99617-32d9-48b4-a31d-7c20da2025e4 --- .../src/perlmods/OpenILS/Application/Storage/Publisher/action.pm | 8 ++++++++ 1 file changed, 8 insertions(+) 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 1574d2790c..432e90e486 100644 --- a/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm +++ b/Open-ILS/src/perlmods/OpenILS/Application/Storage/Publisher/action.pm @@ -1351,6 +1351,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; -- 2.11.0