From 5f62d8b337f0e6937967b10a8926377ad1a8c52a 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 8328134f0a..e5c7e5474e 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Acq/Order.pm @@ -3416,6 +3416,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