Merge pull request #51068 from frappe/mergify/bp/version-15-hotfix/pr-51047

fix(manufacturing): get items for disassembly order (backport #51047)
This commit is contained in:
rohitwaghchaure
2025-12-12 13:10:36 +05:30
committed by GitHub
3 changed files with 12 additions and 8 deletions

View File

@@ -403,6 +403,10 @@ frappe.ui.form.on("Work Order", {
erpnext.work_order
.show_prompt_for_qty_input(frm, "Disassemble")
.then((data) => {
if (flt(data.qty) <= 0) {
frappe.msgprint(__("Disassemble Qty cannot be less than or equal to <b>0</b>."));
return;
}
return frappe.xcall("erpnext.manufacturing.doctype.work_order.work_order.make_stock_entry", {
work_order_id: frm.doc.name,
purpose: "Disassemble",

View File

@@ -1509,7 +1509,7 @@ def make_stock_entry(work_order_id, purpose, qty=None, target_warehouse=None):
stock_entry.to_warehouse = target_warehouse or work_order.source_warehouse
stock_entry.set_stock_entry_type()
stock_entry.get_items(qty, work_order.production_item)
stock_entry.get_items()
if purpose != "Disassemble":
stock_entry.set_serial_no_batch_for_finished_good()

View File

@@ -1905,7 +1905,7 @@ class StockEntry(StockController):
},
)
def get_items_for_disassembly(self, disassemble_qty, production_item):
def get_items_for_disassembly(self):
"""Get items for Disassembly Order"""
if not self.work_order:
@@ -1918,7 +1918,7 @@ class StockEntry(StockController):
items_dict = get_bom_items_as_dict(
self.bom_no,
self.company,
disassemble_qty,
self.fg_completed_qty,
fetch_exploded=self.use_multi_level_bom,
fetch_qty_in_stock_uom=False,
)
@@ -1935,8 +1935,8 @@ class StockEntry(StockController):
child_row.qty = bom_items.get("qty", child_row.qty)
child_row.amount = bom_items.get("amount", child_row.amount)
if row.item_code == production_item:
child_row.qty = disassemble_qty
if row.is_finished_item:
child_row.qty = self.fg_completed_qty
child_row.s_warehouse = (self.from_warehouse or s_warehouse) if row.is_finished_item else ""
child_row.t_warehouse = row.s_warehouse
@@ -1972,12 +1972,12 @@ class StockEntry(StockController):
)
@frappe.whitelist()
def get_items(self, qty=None, production_item=None):
def get_items(self):
self.set("items", [])
self.validate_work_order()
if self.purpose == "Disassemble" and qty is not None:
return self.get_items_for_disassembly(qty, production_item)
if self.purpose == "Disassemble":
return self.get_items_for_disassembly()
if not self.posting_date or not self.posting_time:
frappe.throw(_("Posting date and posting time is mandatory"))