fix: ignore Non-Stock Item while calculating % Picked in Sales Order
(cherry picked from commit 0305a925fe)
This commit is contained in:
@@ -398,9 +398,16 @@ class SalesOrder(SellingController):
|
|||||||
def update_picking_status(self):
|
def update_picking_status(self):
|
||||||
total_picked_qty = 0.0
|
total_picked_qty = 0.0
|
||||||
total_qty = 0.0
|
total_qty = 0.0
|
||||||
|
per_picked = 0.0
|
||||||
|
|
||||||
for so_item in self.items:
|
for so_item in self.items:
|
||||||
|
if cint(
|
||||||
|
frappe.get_cached_value("Item", so_item.item_code, "is_stock_item")
|
||||||
|
) or self.has_product_bundle(so_item.item_code):
|
||||||
total_picked_qty += flt(so_item.picked_qty)
|
total_picked_qty += flt(so_item.picked_qty)
|
||||||
total_qty += flt(so_item.stock_qty)
|
total_qty += flt(so_item.stock_qty)
|
||||||
|
|
||||||
|
if total_picked_qty and total_qty:
|
||||||
per_picked = total_picked_qty / total_qty * 100
|
per_picked = total_picked_qty / total_qty * 100
|
||||||
|
|
||||||
self.db_set("per_picked", flt(per_picked), update_modified=False)
|
self.db_set("per_picked", flt(per_picked), update_modified=False)
|
||||||
|
|||||||
Reference in New Issue
Block a user