From 57a7bcae8e9ccde46e770ff9edf899e383bcba98 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 (cherry picked from commit c1bef53f92ded95d80abd255f94102f372d44b70) # Conflicts: # erpnext/manufacturing/doctype/bom/bom.py --- erpnext/manufacturing/doctype/bom/bom.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/erpnext/manufacturing/doctype/bom/bom.py b/erpnext/manufacturing/doctype/bom/bom.py index aabfe3c4b7b..b92e920dd76 100644 --- a/erpnext/manufacturing/doctype/bom/bom.py +++ b/erpnext/manufacturing/doctype/bom/bom.py @@ -341,6 +341,7 @@ class BOM(WebsiteGenerator): ).format(item.idx, get_link_to_form("Item", item.item_code)) ) +<<<<<<< HEAD if item.qty is None: frappe.throw( <<<<<<< HEAD @@ -353,6 +354,8 @@ class BOM(WebsiteGenerator): ) ) +======= +>>>>>>> c1bef53f92 (refactor(bom): drop redundant secondary item qty None check) if item.process_loss_per >= 100: frappe.throw( _("Row #{0}: Process Loss Percentage should be less than 100% for {1} Item {2}").format(