From eb76248c9952003560a1f13489bc3d181f5b51cb Mon Sep 17 00:00:00 2001 From: Kaushal Shriwas <64089478+kaulith@users.noreply.github.com> Date: Thu, 30 Apr 2026 15:24:38 +0530 Subject: [PATCH] refactor: extract non-labelable fieldtypes to constant --- frappe/desk/query_report.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/frappe/desk/query_report.py b/frappe/desk/query_report.py index 29b18f2d1e..08a00ecc9b 100644 --- a/frappe/desk/query_report.py +++ b/frappe/desk/query_report.py @@ -678,6 +678,9 @@ def get_xlsx_styles(metadata: XLSXMetadata, report_name: str | None = None) -> d return styles +NON_LABELABLE_FIELDTYPES = ("Currency", "Int", "Float", "Percent", "Date", "Datetime", "Time") + + def add_total_row( result, columns, @@ -751,7 +754,7 @@ def add_total_row( else: first_col_fieldtype = columns[0].get("fieldtype") - if first_col_fieldtype not in ["Currency", "Int", "Float", "Percent", "Date", "Datetime", "Time"]: + if first_col_fieldtype not in NON_LABELABLE_FIELDTYPES: total_row[0] = _("Total") result.append(total_row)