From 18c6f99baf04ff4d2933e6558492e72ccbb432c0 Mon Sep 17 00:00:00 2001 From: Lebbeous Fogle-Weekley Date: Tue, 28 Aug 2012 16:09:00 -0400 Subject: [PATCH] Acq: New lineitems added to existing POs should inherit provider Jeremy Miller of the Albany Public Library in Albany, OR, reports that through the interface at Actions For This Record -> View/Place Orders in the staff client, when you add a lineitem to an existing PO, the new lineitem has a null value in the "provider" field, instead of inheriting the expected value from the PO. This causes problems for some Acquisitions workflows. The feature containing the bug appeared in 2.2, and this fix for it can be backported that far. Signed-off-by: Lebbeous Fogle-Weekley Signed-off-by: Bill Erickson --- Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm index 4865a30234..585bca0867 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm @@ -3467,6 +3467,7 @@ sub add_li_to_po { return {success => 0, li => $li, error => 'bad-li-state'}; } + $li->provider($po->provider); $li->purchase_order($po_id); $li->state('pending-order'); update_lineitem($mgr, $li) or return $e->die_event; -- 2.11.0