fix: add all formating fields inside one function

This commit is contained in:
sokumon 2025-03-10 14:54:54 +05:30
parent 1d096040d8
commit 2b8a6352ea

View file

@ -345,8 +345,7 @@ def export_query():
)
return
format_duration_fields(data)
format_currency_fields(data)
format_fields(data)
xlsx_data, column_widths = build_xlsx_data(
data, visible_idx, include_indentation, include_filters=include_filters
)
@ -372,7 +371,7 @@ def export_query():
provide_binary_file(report_name, file_extension, content)
def format_duration_fields(data: frappe._dict) -> None:
def format_fields(data: frappe._dict) -> None:
for i, col in enumerate(data.columns):
if col.get("fieldtype") != "Duration":
continue
@ -381,10 +380,6 @@ def format_duration_fields(data: frappe._dict) -> None:
index = col.get("fieldname") if isinstance(row, dict) else i
if row[index]:
row[index] = format_duration(row[index])
def format_currency_fields(data: frappe._dict) -> None:
for i, col in enumerate(data.columns):
if col.get("fieldtype") == "Currency" and col.get("precision"):
for row in data.result:
index = col.get("fieldname") if isinstance(row, dict) else i