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

Merge pull request #28436 from frappe/mergify/bp/develop/pr-28249

Browse source 
fix: Resolve App Logo considering Website & Navbar Settings  (backport #28249)
This commit is contained in:
Akhil Narang 2024-11-12 15:48:44 +05:30 • committed by GitHub
commit 7d95f4577c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
frappe/core/doctype/navbar_settings/navbar_settings.py
View file

@ -24,7 +24,10 @@ class NavbarSettings(Document):
def get_app_logo():
app_logo = frappe.db.get_single_value("Navbar Settings", "app_logo", cache=True)
app_logo = frappe.get_website_settings("app_logo") or frappe.db.get_single_value(
"Navbar Settings", "app_logo", cache=True
)
if not app_logo:
logos = frappe.get_hooks("app_logo_url")
app_logo = logos[0]