LP1312824 open-ils.circ.hold.change_title fix
authorJason Boyer <JBoyer1@library.in.gov>
Fri, 25 Apr 2014 16:45:34 +0000 (12:45 -0400)
committerGalen Charlton <gmc@equinoxinitiative.org>
Tue, 30 May 2017 13:54:37 +0000 (09:54 -0400)
Add a capture_time => undef limit to both change_title
APIs to prevent currently captured holds from being reset
when moving holds from one title to another.
More work is needed to prevent items currently in transit
from having their transits canceled.

Test plan
---------
1. Create a bib with multiple holdings
2. Add multiple holds to this title and capture one of them - Item
   will have a status of On Holds Shelf
3. Open another bib record, open Actions for this Record menu,
   Mark as Hold Transfer Destination
4. Return to the record that has a captured hold, open Actions for
   this Record, Transfer all Title Holds
5. Check Captured hold, Item status is Available, hold has been reset
   and now points to the new bib record.

With the patch applied, steps 2-5 above should be repeated, but this
time in step 5 the hold will still be on the original record and the
item's status will still be On Holds Shelf.

Signed-off-by: Jason Boyer <JBoyer1@library.in.gov>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm

index 1825c71..87e1baf 100644 (file)
@@ -4037,6 +4037,7 @@ sub change_hold_title {
     my $holds = $e->search_action_hold_request(
         [
             {
+               capture_time     => undef,
                 cancel_time      => undef,
                 fulfillment_time => undef,
                 hold_type        => 'T',
@@ -4073,6 +4074,7 @@ sub change_hold_title_for_specific_holds {
     my $holds = $e->search_action_hold_request(
         [
             {
+               capture_time     => undef,
                 cancel_time      => undef,
                 fulfillment_time => undef,
                 hold_type        => 'T',