diff --git a/frappe/config/__init__.py b/frappe/config/__init__.py index ebd75cd70a..947f61e9e0 100644 --- a/frappe/config/__init__.py +++ b/frappe/config/__init__.py @@ -8,10 +8,8 @@ from frappe.desk.moduleview import ( ) -def get_modules_from_all_apps_for_user(user=None): - if not user: - user = frappe.session.user - +def get_modules_from_all_apps_for_user(user: str = None) -> list[dict]: + user = user or frappe.session.user all_modules = get_modules_from_all_apps() global_blocked_modules = frappe.get_doc("User", "Administrator").get_blocked_modules() user_blocked_modules = frappe.get_doc("User", user).get_blocked_modules() @@ -61,7 +59,7 @@ def get_all_empty_tables_by_module(): empty_tables_by_module = {} for doctype, module in results: - if "tab" + doctype in empty_tables: + if f"tab{doctype}" in empty_tables: if module in empty_tables_by_module: empty_tables_by_module[module].append(doctype) else: diff --git a/frappe/core/notifications.py b/frappe/core/notifications.py index 9a127e567e..093418e345 100644 --- a/frappe/core/notifications.py +++ b/frappe/core/notifications.py @@ -32,11 +32,10 @@ def get_things_todo(as_list=False): if as_list: return data - else: - return data[0][0] + return data[0][0] -def get_todays_events(as_list=False): +def get_todays_events(as_list: bool = False): """Returns a count of todays events in calendar""" from frappe.desk.doctype.event.event import get_events from frappe.utils import nowdate diff --git a/frappe/desk/doctype/event/event.py b/frappe/desk/doctype/event/event.py index e9104ef897..7bcb8207a7 100644 --- a/frappe/desk/doctype/event/event.py +++ b/frappe/desk/doctype/event/event.py @@ -205,7 +205,7 @@ def send_event_digest(): @frappe.whitelist() -def get_events(start, end, user=None, for_reminder=False, filters=None): +def get_events(start, end, user=None, for_reminder=False, filters=None) -> list[frappe._dict]: if not user: user = frappe.session.user