schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 'Use Multi-Level BOM' checkbox default value (backport #44618) (#44620)

Browse Source 
fix: 'Use Multi-Level BOM' checkbox default value (#44618)

(cherry picked from commit f5c038cd1b)

Co-authored-by: rohitwaghchaure <rohitw1991@gmail.com>
This commit is contained in:
mergify[bot]
2024-12-10 10:57:20 +05:30
committed by GitHub
parent eae5996c4c
commit 5cd39941ee
2 changed files with 24 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
erpnext/manufacturing/doctype/bom/bom.js
View File

@@ -255,15 +255,6 @@ frappe.ui.form.on("BOM", {
});
}
if (!skip_qty_field) {
fields.push({
fieldtype: "Check",
label: __("Use Multi-Level BOM"),
fieldname: "use_multi_level_bom",
default: 1,
});
}
if (!skip_qty_field) {
fields.push({
fieldtype: "Float",
@@ -291,6 +282,13 @@ frappe.ui.form.on("BOM", {
cur_dialog.refresh();
},
});
fields.push({
fieldtype: "Check",
label: __("Use Multi-Level BOM"),
fieldname: "use_multi_level_bom",
default: frm.doc?.__onload.use_multi_level_bom,
});
}
var has_template_rm = frm.doc.items.filter((d) => d.has_variants === 1) || [];

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

@@ -214,6 +214,23 @@ class BOM(WebsiteGenerator):
return index
def onload(self):
super().onload()
self.set_onload_for_muulti_level_bom()
def set_onload_for_muulti_level_bom(self):
use_multi_level_bom = frappe.db.get_value(
"Property Setter",
{"field_name": "use_multi_level_bom", "doc_type": "Work Order", "property": "default"},
"value",
)
if use_multi_level_bom is None:
use_multi_level_bom = 1
self.set_onload("use_multi_level_bom", cint(use_multi_level_bom))
@staticmethod
def get_next_version_index(existing_boms: list[str]) -> int:
# split by "/" and "-"