From dfc824ded67c2221420bb1cd9c01cccf69183363 Mon Sep 17 00:00:00 2001 From: Diptanil Saha Date: Wed, 10 Jun 2026 00:22:28 +0530 Subject: [PATCH] fix(process statement of accounts): validate pdf_name and validate permission before triggering send_auto_email (#55781) --- .../process_statement_of_accounts.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/erpnext/accounts/doctype/process_statement_of_accounts/process_statement_of_accounts.py b/erpnext/accounts/doctype/process_statement_of_accounts/process_statement_of_accounts.py index 9cd15cb76f7..d8d21ecb698 100644 --- a/erpnext/accounts/doctype/process_statement_of_accounts/process_statement_of_accounts.py +++ b/erpnext/accounts/doctype/process_statement_of_accounts/process_statement_of_accounts.py @@ -101,6 +101,7 @@ class ProcessStatementOfAccounts(Document): validate_template(self.subject) validate_template(self.body) + validate_template(self.pdf_name) if not self.customers: frappe.throw(_("Customers not selected.")) @@ -578,6 +579,7 @@ def send_emails(document_name: str, from_scheduler: bool = False, posting_date: @frappe.whitelist() def send_auto_email(): + frappe.has_permission("Process Statement Of Accounts", throw=True) selected = frappe.get_list( "Process Statement Of Accounts", filters={"enable_auto_email": 1},