Merge pull request #54343 from frappe/mergify/bp/version-16-hotfix/pr-54342

fix: make Target Warehouse mandatory on UI for WO (backport #54342)
This commit is contained in:
rohitwaghchaure
2026-04-17 13:27:35 +05:30
committed by GitHub

View File

@@ -241,6 +241,11 @@ frappe.ui.form.on("Work Order", {
frm.trigger("allow_alternative_item");
frm.trigger("hide_reserve_stock_button");
frm.trigger("toggle_items_editable");
frm.trigger("set_fg_warehouse_mandatory");
},
skip_transfer(frm) {
frm.trigger("set_fg_warehouse_mandatory");
},
toggle_items_editable(frm) {
@@ -277,6 +282,11 @@ frappe.ui.form.on("Work Order", {
return has_reserved_stock;
},
set_fg_warehouse_mandatory(frm) {
let mandatory = frm.doc.skip_transfer === 1 || frm.doc.track_semi_finished_goods === 1 ? false : true;
frm.toggle_reqd("fg_warehouse", mandatory);
},
add_custom_button_to_return_components: function (frm) {
if (frm.doc.docstatus === 1 && ["Closed", "Completed"].includes(frm.doc.status)) {
let non_consumed_items = frm.doc.required_items.filter((d) => {