From a8ca40b4449d6b6f4cde1e788d784be30fdfa997 Mon Sep 17 00:00:00 2001 From: sokumon Date: Tue, 21 Apr 2026 16:58:54 +0530 Subject: [PATCH] fix(dialog): add a flag to include default value --- frappe/public/js/frappe/ui/field_group.js | 5 ++++- frappe/public/js/frappe/views/reports/report_view.js | 1 + 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/ui/field_group.js b/frappe/public/js/frappe/ui/field_group.js index f40038fe55..d23d75197a 100644 --- a/frappe/public/js/frappe/ui/field_group.js +++ b/frappe/public/js/frappe/ui/field_group.js @@ -142,7 +142,10 @@ frappe.ui.FieldGroup = class FieldGroup extends frappe.ui.form.Layout { for (let key in this.fields_dict) { let f = this.fields_dict[key]; if (f.get_value) { - let v = f.get_value() || f.df.default; + let v = f.get_value(); + if (!v && f.df.include_default) { + v = f.df.default; + } if (f.df.reqd && is_null(typeof v === "string" ? strip_html(v) : v)) errors.push(__(f.df.label)); diff --git a/frappe/public/js/frappe/views/reports/report_view.js b/frappe/public/js/frappe/views/reports/report_view.js index 7eeee457c9..05d4637757 100644 --- a/frappe/public/js/frappe/views/reports/report_view.js +++ b/frappe/public/js/frappe/views/reports/report_view.js @@ -579,6 +579,7 @@ frappe.views.ReportView = class ReportView extends frappe.views.ListView { options: y_fields, description: __("Showing only Numeric fields from Report"), default: defaults.y_axes ? defaults.y_axes.join(", ") : null, + include_default: true, }, { label: __("Chart Type"),