From 746f6bd12298b0ceb510a307cce11f62fdeeff75 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 --- .../perlmods/lib/OpenILS/Application/Acq/Order.pm | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) 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 4865a30..585bca0 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; -- 1.7.2.5