From: Lebbeous Fogle-Weekley Date: Tue, 28 Aug 2012 20:09:00 +0000 (-0400) Subject: Acq: New lineitems added to existing POs should inherit provider X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=5f62d8b337f0e6937967b10a8926377ad1a8c52a;p=Evergreen.git 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 --- 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;