nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-01 07:37:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(bom): drop redundant secondary item qty None check

Browse Source 
Float fields default to 0, so qty is never None. Per review feedback,
remove the validation entirely.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
(cherry picked from commit c1bef53f92)

# Conflicts:
#	erpnext/manufacturing/doctype/bom/bom.py
This commit is contained in:
Mohammad Umair Sayed
2026-06-15 15:06:24 +05:30
committed by Mergify
parent 532c343b57
commit 57a7bcae8e
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
erpnext/manufacturing/doctype/bom/bom.py
View File

@@ -341,6 +341,7 @@ class BOM(WebsiteGenerator):
).format(item.idx, get_link_to_form("Item", item.item_code)) ).format(item.idx, get_link_to_form("Item", item.item_code))
) )
<<<<<<< HEAD
if item.qty is None: if item.qty is None:
frappe.throw( frappe.throw(
<<<<<<< HEAD <<<<<<< HEAD
@@ -353,6 +354,8 @@ class BOM(WebsiteGenerator):
) )
) )
=======
>>>>>>> c1bef53f92 (refactor(bom): drop redundant secondary item qty None check)
if item.process_loss_per >= 100: if item.process_loss_per >= 100:
frappe.throw( frappe.throw(
_("Row #{0}: Process Loss Percentage should be less than 100% for {1} Item {2}").format( _("Row #{0}: Process Loss Percentage should be less than 100% for {1} Item {2}").format(