From ecf2976681ceedd7eabd0b67c68e24947f0bbfec Mon Sep 17 00:00:00 2001 From: Corentin Forler <10946971+cogk@users.noreply.github.com> Date: Sat, 23 Nov 2024 00:26:35 +0100 Subject: [PATCH] fix(quick_list): Ensure fields exist before fetching --- frappe/public/js/frappe/widgets/quick_list_widget.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/frappe/public/js/frappe/widgets/quick_list_widget.js b/frappe/public/js/frappe/widgets/quick_list_widget.js index 764c8faab8..4117e77de0 100644 --- a/frappe/public/js/frappe/widgets/quick_list_widget.js +++ b/frappe/public/js/frappe/widgets/quick_list_widget.js @@ -205,8 +205,13 @@ export default class QuickListWidget extends Widget { let add_fields = frappe.listview_settings?.[this.document_type]?.add_fields; if (Array.isArray(add_fields)) { - fields.push(...add_fields); - fields = [...new Set(fields)]; + for (const fieldname of add_fields) { + // Only keep fields that exist and are permitted + if (frappe.meta.has_field(this.document_type, fieldname)) { + fields.push(fieldname); + } + } + fields = [...new Set(fields)]; // Remove duplicates } let quick_list_filter = frappe.utils.process_filter_expression(this.quick_list_filter);