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

This commit is contained in:
rohitwaghchaure
2024-12-09 20:22:51 +05:30
committed by GitHub
parent 616bb383c5
commit f5c038cd1b
2 changed files with 24 additions and 9 deletions

View File

@@ -303,15 +303,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",
@@ -339,6 +330,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) || [];

View File

@@ -217,6 +217,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 "-"