From c1bef53f92ded95d80abd255f94102f372d44b70 Mon Sep 17 00:00:00 2001 From: Mohammad Umair Sayed Date: Mon, 15 Jun 2026 15:06:24 +0530 Subject: [PATCH] refactor(bom): drop redundant secondary item qty None check Float fields default to 0, so qty is never None. Per review feedback, remove the validation entirely. Co-Authored-By: Claude Opus 4.8 --- erpnext/manufacturing/doctype/bom/bom.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/erpnext/manufacturing/doctype/bom/bom.py b/erpnext/manufacturing/doctype/bom/bom.py index c20e8f3cd47..ebe6e04ac03 100644 --- a/erpnext/manufacturing/doctype/bom/bom.py +++ b/erpnext/manufacturing/doctype/bom/bom.py @@ -373,13 +373,6 @@ class BOM(WebsiteGenerator): ).format(item.idx, get_link_to_form("Item", item.item_code)) ) - if item.qty is None: - frappe.throw( - _("Row #{0}: Quantity is required for {1} Item {2}").format( - item.idx, item.secondary_item_type, get_link_to_form("Item", item.item_code) - ) - ) - if item.process_loss_per >= 100: frappe.throw( _("Row #{0}: Process Loss Percentage should be less than 100% for {1} Item {2}").format(