fix(process statement of accounts): validate pdf_name and validate permission before triggering send_auto_email (#55781)

This commit is contained in:
Diptanil Saha
2026-06-10 00:22:28 +05:30
committed by GitHub
parent dfd7cd0bae
commit dfc824ded6

View File

@@ -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},