From cd1dfeeab300f81bfba5461cf1c4bfaa81472fae Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Mon, 23 Mar 2026 10:56:15 +0000 Subject: [PATCH] fix: check for `submit` permissions instead of `write` permissions when updating status (backport #53697) (#53703) Co-authored-by: diptanilsaha fix: check for `submit` permissions instead of `write` permissions when updating status (#53697) --- erpnext/buying/doctype/purchase_order/purchase_order.py | 2 +- erpnext/selling/doctype/sales_order/sales_order.py | 2 +- erpnext/stock/doctype/purchase_receipt/purchase_receipt.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/erpnext/buying/doctype/purchase_order/purchase_order.py b/erpnext/buying/doctype/purchase_order/purchase_order.py index 22e731a336c..a0daeca51f2 100644 --- a/erpnext/buying/doctype/purchase_order/purchase_order.py +++ b/erpnext/buying/doctype/purchase_order/purchase_order.py @@ -889,7 +889,7 @@ def get_list_context(context=None): @frappe.whitelist() def update_status(status, name): - po = frappe.get_lazy_doc("Purchase Order", name, check_permission="write") + po = frappe.get_lazy_doc("Purchase Order", name, check_permission="submit") po.update_status(status) po.update_delivered_qty_in_sales_order() diff --git a/erpnext/selling/doctype/sales_order/sales_order.py b/erpnext/selling/doctype/sales_order/sales_order.py index 35ff3c054fd..dcf2ed1ad9b 100755 --- a/erpnext/selling/doctype/sales_order/sales_order.py +++ b/erpnext/selling/doctype/sales_order/sales_order.py @@ -1803,7 +1803,7 @@ def make_work_orders(items, sales_order, company, project=None): @frappe.whitelist() def update_status(status, name): - so = frappe.get_doc("Sales Order", name, check_permission="write") + so = frappe.get_doc("Sales Order", name, check_permission="submit") so.update_status(status) diff --git a/erpnext/stock/doctype/purchase_receipt/purchase_receipt.py b/erpnext/stock/doctype/purchase_receipt/purchase_receipt.py index 3e342222d99..e78faa9511a 100644 --- a/erpnext/stock/doctype/purchase_receipt/purchase_receipt.py +++ b/erpnext/stock/doctype/purchase_receipt/purchase_receipt.py @@ -1581,7 +1581,7 @@ def make_purchase_return(source_name, target_doc=None): @frappe.whitelist() def update_purchase_receipt_status(docname, status): - pr = frappe.get_lazy_doc("Purchase Receipt", docname, check_permission="write") + pr = frappe.get_lazy_doc("Purchase Receipt", docname, check_permission="submit") pr.update_status(status)