nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-08-11 21:51:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: warn about existing draft orders before the supplier selection creates more

Browse Source 
Creating through the dialog calls the endpoint directly instead of going
through open_mapped_doc, so the draft link guard that every other Create action
runs never fired, and a repeated dialog quietly produced a second set of draft
orders for the same quantity.
This commit is contained in:
Mihir Kandoi
2026-08-01 09:16:13 +05:30
parent 8ffe5ba420
commit f0bb70539d
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/stock/doctype/material_request/material_request.js
View File

@@ -512,7 +512,7 @@ frappe.ui.form.on("Material Request", {
},
],
primary_action_label: __("Create"),
primary_action: function (values) {
primary_action: async function (values) {
const item_suppliers = (values.items || []).filter((row) => row.__checked);
if (!item_suppliers.length) {
frappe.throw(__("Select at least one Item"));
@@ -546,6 +546,10 @@ frappe.ui.form.on("Material Request", {
);
}
if (!(await erpnext.utils.confirm_if_drafts_exist(frm.doc, "Purchase Order"))) {
return;
}
frappe.call({
method: "erpnext.stock.doctype.material_request.mapper.make_purchase_orders_by_supplier",
args: { source_name: frm.doc.name, item_suppliers: item_suppliers },