From dfb7054c3a943e98f2c95d4e598e48c6e20f402e Mon Sep 17 00:00:00 2001 From: Thomas Berezansky Date: Wed, 9 May 2012 08:42:58 -0400 Subject: [PATCH] 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 --- Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 -- 2.11.0