nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-17 10:36:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(manufacturing): case-sensitive variant BOM lookup on Postgres (#56407)

Browse Source 
Reapply "fix(manufacturing): case-sensitive variant BOM lookup on Postgres"

This reverts commit 1f86b57f94.
This commit is contained in:
Mihir Kandoi
2026-06-24 10:21:29 +05:30
committed by GitHub
parent 21541e3ad3
commit ead694c9cb
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -1403,16 +1403,18 @@ def validate_bom_no(item, bom_no):
def _bom_contains_item(bom, item): def _bom_contains_item(bom, item):
item = item.lower() item_lower = item.lower()
for d in bom.items: for d in bom.items:
if d.item_code.lower() == item: if d.item_code.lower() == item_lower:
return True return True
for d in bom.secondary_items: for d in bom.secondary_items:
if d.item_code.lower() == item: if d.item_code.lower() == item_lower:
return True return True
# Use the original-cased `item` for the Item lookup: names are case-sensitive on Postgres,
# so a lowercased name would miss the record and drop the variant->template BOM match.
return ( return (
bom.item.lower() == item bom.item.lower() == item_lower
or bom.item.lower() == cstr(frappe.db.get_value("Item", item, "variant_of")).lower() or bom.item.lower() == cstr(frappe.db.get_value("Item", item, "variant_of")).lower()
) )