diff --git a/frappe/desk/doctype/notification_log/notification_log.py b/frappe/desk/doctype/notification_log/notification_log.py index c192eccede..5b4f49526c 100644 --- a/frappe/desk/doctype/notification_log/notification_log.py +++ b/frappe/desk/doctype/notification_log/notification_log.py @@ -167,7 +167,7 @@ def format_email_header(header_map, language, docname): @frappe.whitelist() @http_cache(max_age=60, stale_while_revalidate=60 * 60) -def get_notification_logs(limit=20): +def get_notification_logs(limit: int = 20): notification_logs = frappe.db.get_list( "Notification Log", fields=["*"], limit=limit, order_by="creation desc" ) diff --git a/frappe/frappeclient.py b/frappe/frappeclient.py index f8c619fc6b..65c4b17fa3 100644 --- a/frappe/frappeclient.py +++ b/frappe/frappeclient.py @@ -109,11 +109,11 @@ class FrappeClient: def get_list( self, - doctype, + doctype: str, fields='["name"]', filters=None, - limit_start=0, - limit_page_length=None, + limit_start: int = 0, + limit_page_length: int | None = None, order_by=None, group_by=None, ): diff --git a/frappe/model/qb_query.py b/frappe/model/qb_query.py index a187e9b4d5..6e99dd50c4 100644 --- a/frappe/model/qb_query.py +++ b/frappe/model/qb_query.py @@ -182,8 +182,8 @@ class DatabaseQuery: "or_filters": or_filters, "group_by": group_by, "order_by": order_by, - "limit": limit, - "offset": offset, + "limit": frappe.cint(limit), + "offset": frappe.cint(offset), "distinct": distinct, "ignore_permissions": ignore_permissions, "user": user, diff --git a/frappe/utils/__init__.py b/frappe/utils/__init__.py index c9975d643c..b5bd7397b7 100644 --- a/frappe/utils/__init__.py +++ b/frappe/utils/__init__.py @@ -849,7 +849,7 @@ def get_site_info(): kwargs = { "fields": ["user", "creation", "full_name"], "filters": {"operation": "Login", "status": "Success"}, - "limit": "10", + "limit": 10, } site_info = {