From 26ae708d6d59ea04f862a20dcc08555f7bf553ab Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Wed, 3 Jan 2024 20:32:55 +0530 Subject: [PATCH] fix(UX): dont override framework's permission check messages (backport #39118) (#39119) fix(UX): dont override framework's permission check messages (#39118) (cherry picked from commit e84c9f7c51fc6e8c5c631c74131f9290624b2a05) Co-authored-by: Ankush Menat --- erpnext/accounts/party.py | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/erpnext/accounts/party.py b/erpnext/accounts/party.py index c24d28b32490..c83969b34bcc 100644 --- a/erpnext/accounts/party.py +++ b/erpnext/accounts/party.py @@ -114,14 +114,12 @@ def _get_party_details( set_account_and_due_date(party, account, party_type, company, posting_date, bill_date, doctype) ) party = party_details[party_type.lower()] + party = frappe.get_doc(party_type, party) - if not ignore_permissions and not ( - frappe.has_permission(party_type, "read", party) - or frappe.has_permission(party_type, "select", party) - ): - frappe.throw(_("Not permitted for {0}").format(party), frappe.PermissionError) + if not ignore_permissions: + ptype = "select" if frappe.only_has_select_perm(party_type) else "read" + frappe.has_permission(party_type, ptype, party, throw=True) - party = frappe.get_doc(party_type, party) currency = party.get("default_currency") or currency or get_company_currency(company) party_address, shipping_address = set_address_details(