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

fix: restructure logic to generate report name

Browse source
This commit is contained in:
sokumon 2025-03-28 01:16:02 +05:30
parent 584f63c9b0
commit a2ead69051
1 changed files with 8 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
frappe/desk/query_report.py
View file

@ -316,7 +316,6 @@ def export_query():
form_params = frappe._dict(frappe.local.form_dict)
csv_params = pop_csv_params(form_params)
print(form_params)
clean_params(form_params)
parse_json(form_params)
@ -361,14 +360,14 @@ def export_query():
content = make_xlsx(xlsx_data, "Query Report", column_widths=column_widths).getvalue()
for value in (data.filters or {}).values():
if isinstance(value, list) and valid_report_name(report_name, "_" + ",".join(value)):
report_name += "_" + ",".join(value)
elif (
isinstance(value, str)
and value not in {"Yes", "No"}
and valid_report_name(report_name, f"_{value}")
):
report_name += f"_{value}"
suffix = ""
if isinstance(value, list):
suffix = "_" + ",".join(value)
elif isinstance(value, str) and value not in {"Yes", "No"}:
suffix = f"_{value}"
if valid_report_name(report_name, suffix):
report_name += suffix
provide_binary_file(report_name, file_extension, content)