mirror of
https://github.com/frappe/erpnext.git
synced 2026-09-14 01:20:41 +00:00
Merge pull request #58050 from mihir-kandoi/codex/fix-bom-creator-item-selection
fix(manufacturing): preserve BOM Creator item details
This commit is contained in:
@@ -208,13 +208,6 @@ frappe.ui.form.on("BOM Creator", {
|
|||||||
});
|
});
|
||||||
|
|
||||||
frappe.ui.form.on("BOM Creator Item", {
|
frappe.ui.form.on("BOM Creator Item", {
|
||||||
item_code(frm, cdt, cdn) {
|
|
||||||
let item = frappe.get_doc(cdt, cdn);
|
|
||||||
if (item.item_code && item.is_root) {
|
|
||||||
frappe.model.set_value(cdt, cdn, "fg_item", item.item_code);
|
|
||||||
}
|
|
||||||
},
|
|
||||||
|
|
||||||
do_not_explode(frm, cdt, cdn) {
|
do_not_explode(frm, cdt, cdn) {
|
||||||
let item = frappe.get_doc(cdt, cdn);
|
let item = frappe.get_doc(cdt, cdn);
|
||||||
if (!item.do_not_explode) {
|
if (!item.do_not_explode) {
|
||||||
@@ -237,6 +230,20 @@ frappe.ui.form.on("BOM Creator Item", {
|
|||||||
});
|
});
|
||||||
|
|
||||||
erpnext.bom.BomConfigurator = class BomConfigurator extends erpnext.TransactionController {
|
erpnext.bom.BomConfigurator = class BomConfigurator extends erpnext.TransactionController {
|
||||||
|
item_code(doc, cdt, cdn) {
|
||||||
|
if (cdt !== "BOM Creator Item") {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
let item = frappe.get_doc(cdt, cdn);
|
||||||
|
if (item.item_code && item.is_root) {
|
||||||
|
frappe.model.set_value(cdt, cdn, "fg_item", item.item_code);
|
||||||
|
}
|
||||||
|
|
||||||
|
// BOM Creator does not support TransactionController's server-side item selection.
|
||||||
|
return this.process_item_selection(doc, cdt, cdn);
|
||||||
|
}
|
||||||
|
|
||||||
conversion_rate(doc) {
|
conversion_rate(doc) {
|
||||||
if (this.frm.doc.currency === this.get_company_currency()) {
|
if (this.frm.doc.currency === this.get_company_currency()) {
|
||||||
this.frm.set_value("conversion_rate", 1.0);
|
this.frm.set_value("conversion_rate", 1.0);
|
||||||
|
|||||||
@@ -79,7 +79,7 @@ def _preprocess_ctx(ctx):
|
|||||||
@erpnext.normalize_ctx_input(ItemDetailsCtx)
|
@erpnext.normalize_ctx_input(ItemDetailsCtx)
|
||||||
def get_item_details(
|
def get_item_details(
|
||||||
ctx: ItemDetailsCtx,
|
ctx: ItemDetailsCtx,
|
||||||
doc: Document | str | None = None,
|
doc: Document | str | dict | None = None,
|
||||||
for_validate: bool | None = False,
|
for_validate: bool | None = False,
|
||||||
overwrite_warehouse: bool = True,
|
overwrite_warehouse: bool = True,
|
||||||
):
|
):
|
||||||
|
|||||||
Reference in New Issue
Block a user