diff --git a/erpnext/stock/doctype/material_request/mapper.py b/erpnext/stock/doctype/material_request/mapper.py index 1ce6438116c..cb96075cb21 100644 --- a/erpnext/stock/doctype/material_request/mapper.py +++ b/erpnext/stock/doctype/material_request/mapper.py @@ -7,7 +7,7 @@ import frappe from frappe import _ from frappe.model.document import Document from frappe.model.mapper import get_mapped_doc -from frappe.utils import cint, flt, getdate, nowdate +from frappe.utils import cint, comma_and, flt, get_link_to_form, getdate, nowdate from erpnext.setup.doctype.brand.brand import get_brand_defaults from erpnext.setup.doctype.item_group.item_group import get_item_group_defaults @@ -242,6 +242,12 @@ def make_purchase_orders_by_supplier(source_name: str, item_suppliers: str | lis purchase_order.insert() purchase_orders.append(purchase_order.name) + frappe.msgprint( + _("{0} created").format( + comma_and([get_link_to_form("Purchase Order", name) for name in purchase_orders]) + ) + ) + return purchase_orders diff --git a/erpnext/stock/doctype/material_request/material_request.js b/erpnext/stock/doctype/material_request/material_request.js index 1aefaefd514..62bdb23fd4c 100644 --- a/erpnext/stock/doctype/material_request/material_request.js +++ b/erpnext/stock/doctype/material_request/material_request.js @@ -534,23 +534,9 @@ frappe.ui.form.on("Material Request", { args: { source_name: frm.doc.name, item_suppliers: item_suppliers }, freeze: true, callback: function (r) { - if (r.exc) return; - - dialog.hide(); - - const purchase_orders = r.message || []; - if (purchase_orders.length === 1) { - frappe.set_route("Form", "Purchase Order", purchase_orders[0]); - return; + if (!r.exc) { + dialog.hide(); } - - frappe.msgprint({ - title: __("Purchase Orders Created"), - indicator: "green", - message: purchase_orders - .map((name) => frappe.utils.get_form_link("Purchase Order", name, true)) - .join(", "), - }); }, }); },