From 46bff320f361e51a5a1143186b32ea22667d19a0 Mon Sep 17 00:00:00 2001 From: Mihir Kandoi Date: Mon, 3 Aug 2026 13:28:09 +0530 Subject: [PATCH] fix(stock): cost a BOM-less secondary item out of the finished good The legacy scrap checkbox deducted the scrap row's value from the finished good, so a Manufacture entry balanced. Its replacement, the Secondary Item Type dropdown, only balances when the row carries a BOM Secondary Item link, because the cost allocation percentage lives there. A row typed as Scrap in the UI has no such link, so its value was added on top of a finished good that already absorbed the whole raw material cost, and the entry closed with a non-zero difference. Treat a secondary row with no BOM link the way the legacy scrap item was treated: deduct its value from the finished good. The finished good's rate is derived from the other incoming rows, so those rows must be rated first. Previously the finished good was rated in row order, ahead of the secondary rows, and picked up their amounts only on a later validate pass. Rate the finished goods last so a single pass is correct. (cherry picked from commit 5e81cd1540a370dcdc00c7d0637302ca2efaccbb) # Conflicts: # erpnext/stock/doctype/stock_entry/stock_entry.py --- .../stock/doctype/stock_entry/stock_entry.py | 24 ++++++++++++++++++- 1 file changed, 23 insertions(+), 1 deletion(-) diff --git a/erpnext/stock/doctype/stock_entry/stock_entry.py b/erpnext/stock/doctype/stock_entry/stock_entry.py index ad22e5c3c08..59b75c925a2 100644 --- a/erpnext/stock/doctype/stock_entry/stock_entry.py +++ b/erpnext/stock/doctype/stock_entry/stock_entry.py @@ -83,10 +83,20 @@ from erpnext.controllers.subcontracting_inward_controller import SubcontractingI form_grid_templates = {"items": "templates/form_grid/stock_entry_grid.html"} +<<<<<<< HEAD def _qty_tolerance(precision: int) -> float: """One unit at the column's precision -- absorbs float rounding without letting a real (whole-unit) quantity divergence slip through.""" return 1.0 / (10**precision) +======= +def is_costed_out_of_finished_item(row) -> bool: + """Whether the row takes its value out of the finished good instead of adding to it. + + A secondary item that is not linked to a BOM has no cost allocation of its own, so it is + valued the way the legacy scrap item was: its cost is deducted from the finished good. + """ + return bool(row.is_legacy_scrap_item or (row.secondary_item_type and not row.bom_secondary_item)) +>>>>>>> 5e81cd1540 (fix(stock): cost a BOM-less secondary item out of the finished good) class StockEntry(StockController, SubcontractingInwardController): @@ -1447,9 +1457,19 @@ class StockEntry(StockController, SubcontractingInwardController): outgoing_items_cost = self.set_rate_for_outgoing_items(reset_outgoing_rate, raise_error_if_no_rate) has_consumption_basis = self.has_consumption_basis() +<<<<<<< HEAD items = [] # Set basic rate for incoming items for d in self.get("items"): +======= + bom_cost_allocation_per = ( + frappe.get_cached_value("BOM", self.bom_no, "cost_allocation_per") if self.bom_no else None + ) + + zero_valuation_items = [] + finished_items_last = sorted(self.get("items"), key=lambda row: cint(row.is_finished_item)) + for d in finished_items_last: +>>>>>>> 5e81cd1540 (fix(stock): cost a BOM-less secondary item out of the finished good) if d.s_warehouse or d.set_basic_rate_manually: continue @@ -1605,7 +1625,9 @@ class StockEntry(StockController, SubcontractingInwardController): self, finished_item_qty, outgoing_items_cost=0, has_consumption_basis=False ) -> float: settings = frappe.get_single("Manufacturing Settings") - scrap_items_cost = sum([flt(d.basic_amount) for d in self.get("items") if d.is_legacy_scrap_item]) + scrap_items_cost = sum( + [flt(d.basic_amount) for d in self.get("items") if is_costed_out_of_finished_item(d)] + ) if settings.material_consumption: if settings.get_rm_cost_from_consumption_entry and self.work_order: