Merge pull request #26201 from frappe/mergify/bp/version-14-hotfix/pr…

…-26165 fix(report_utils): ensure that delimiter and separator can't be empty (backport #26165)
frappe · Apr 30, 2024 · f235018 · f235018
2 parents e889691 + 5301516
commit f235018
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/frappe/public/js/frappe/views/reports/report_utils.js b/frappe/public/js/frappe/views/reports/report_utils.js
@@ -256,8 +256,12 @@ frappe.report_utils = {
 
 		dialog.fields_dict["file_format"].df.onchange = () => update_csv_preview(dialog);
 		dialog.fields_dict["csv_quoting"].df.onchange = () => update_csv_preview(dialog);
-		dialog.fields_dict["csv_delimiter"].df.onchange = () => update_csv_preview(dialog);
-
+		dialog.fields_dict["csv_delimiter"].df.onchange = () => {
+			if (!dialog.get_value("csv_delimiter")) {
+				dialog.set_value("csv_delimiter", ",");
+			}
+			update_csv_preview(dialog);
+		};
 		return dialog;
 	},