From 1941bb0a6907a65a3f67b8a8953c6280bf07ae00 Mon Sep 17 00:00:00 2001 From: KerollesFathy Date: Sun, 29 Mar 2026 10:45:35 +0000 Subject: [PATCH] refactor: use lookup map for options field descriptions instead of if else --- .../form_builder/components/FieldProperties.vue | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/frappe/public/js/form_builder/components/FieldProperties.vue b/frappe/public/js/form_builder/components/FieldProperties.vue index a43e94e500..f86f73fd54 100644 --- a/frappe/public/js/form_builder/components/FieldProperties.vue +++ b/frappe/public/js/form_builder/components/FieldProperties.vue @@ -61,15 +61,14 @@ let docfield_df = computed(() => { df.options = ["", "Email", "Name", "Phone", "URL", "Barcode", "IBAN"]; } - if (store.form.selected_field.fieldtype === "Select") { - df.description = __("Enter list of Options, each on a new line."); - } else if (store.form.selected_field.fieldtype === "Currency") { - df.description = __( + const df_desc = { + Select: __("Enter list of Options, each on a new line."), + Currency: __( "fieldname of the currency field or a cached value (e.g. Company:company:default_currency)." - ); - } else { - df.description = ""; - } + ), + }; + + df.description = df_desc[store.form.selected_field.fieldtype] || ""; } // show link_filters docfield only when link field is selected