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

Merge pull request #17163 from frappe/mergify/bp/develop/pr-17161

Browse source 
fix: Handle case where document title can be NONE (backport #17161)
This commit is contained in:
Suraj Shetty 2022-06-13 11:47:27 +05:30 • committed by GitHub
commit b1e97c1ff4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/model/document.py
View file

@ -438,7 +438,7 @@ class Document(BaseDocument):
def get_title(self):
"""Get the document title based on title_field or `title` or `name`"""
return self.get(self.meta.get_title_field())
return self.get(self.meta.get_title_field()) or ""
def set_title_field(self):
"""Set title field based on template"""

2
frappe/www/printview.py
View file

@ -63,7 +63,7 @@ def get_context(context):
"body": body,
"print_style": print_style,
"comment": frappe.session.user,
"title": frappe.utils.strip_html(doc.get_title()),
"title": frappe.utils.strip_html(doc.get_title() or doc.name),
"lang": frappe.local.lang,
"layout_direction": "rtl" if is_rtl() else "ltr",
"doctype": frappe.form_dict.doctype,