From: Thomas Berezansky Date: Wed, 9 May 2012 12:42:58 +0000 (-0400) Subject: Retarget Local Holds: Use part ID properly X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=dbee9ecee0a9833e8fa2df384ab456da9622d0c1;p=evergreen%2Fpines.git Retarget Local Holds: Use part ID properly Because the mapping ID is meaningless unless you only create one part at a time, assign them only once in the order you created them, etc. Which I think pretty much describes most of my original test cycle. Signed-off-by: Thomas Berezansky Signed-off-by: Lebbeous Fogle-Weekley --- diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm index 731131dec3..5bdb1c349c 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm @@ -2365,7 +2365,7 @@ sub checkin_retarget { # Check for parts on this copy my $parts = $self->editor->search_asset_copy_part_map({ target_copy => $self->copy->id }); my %parts_hash = (); - %parts_hash = map {$_->id, 1} @$parts if @$parts; + %parts_hash = map {$_->part, 1} @$parts if @$parts; # Loop over holds in request-ish order # Stage 1: Get them into request-ish order