vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #38730 from Nihantra-Patel/fix-report-letterhead-validation

Browse source 
fix: skip report letter head validation when no letter head is set
This commit is contained in:
Ejaaz Khan 2026-04-20 12:54:01 +05:30 • committed by GitHub
commit 9ead794803
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
frappe/core/doctype/report/report.py
View file

@ -428,6 +428,9 @@ class Report(Document):
frappe.throw(_("Selected Print Format is invalid for this Report."))
def validate_letter_head(self):
if not self.letter_head:
return
letter_head = frappe.db.get_value(
"Letter Head",
self.letter_head,
@ -441,7 +444,11 @@ class Report(Document):
or (self.is_standard == "Yes" and letter_head.standard != "Yes")
or letter_head.disabled
):
frappe.throw(_("Selected Letter Head is invalid for this Report."))
frappe.throw(
_("Selected Letter Head '{0}' is invalid for '{1}' Report.").format(
self.letter_head, self.name
)
)
@frappe.whitelist()
def toggle_disable(self, disable: bool):