From 678a4041815634f1e1c1b642ff7b5daf3efe34fd Mon Sep 17 00:00:00 2001 From: Akhil Narang Date: Thu, 22 Aug 2024 15:27:40 +0530 Subject: [PATCH] fix(table_multiselect): sentry FRAPPE-6QQ ``` TypeError: Cannot read properties of undefined (reading 'fields') at frappe.ui.form.ControlTableMultiSelectget_link_field(../../../../../apps/frappe/frappe/public/js/frappe/form/controls/table_multiselect.js:151:28) at frappe.ui.form.ControlTableMultiSelectset_formatted_input(../../../../../apps/frappe/frappe/public/js/frappe/form/controls/table_multiselect.js:124:27) at _a.set_input(../../../../../apps/frappe/frappe/public/js/frappe/form/controls/data.js:247:8) at update_input(../../../../../apps/frappe/frappe/public/js/frappe/form/controls/base_input.js:89:8) at _a.refresh_input(../../../../../apps/frappe/frappe/public/js/frappe/form/controls/base_input.js:110:5) at frappe.ui.form.Controlrefresh(../../../../../apps/frappe/frappe/public/js/frappe/form/controls/base_control.js:140:9) at frappe.ui.form.Layoutattach_doc_and_docfields(../../../../../apps/frappe/frappe/public/js/frappe/form/layout.js:490:59) at frappe.ui.form.Layoutrefresh(../../../../../apps/frappe/frappe/public/js/frappe/form/layout.js:344:8) at frappe.ui.form.Formrefresh_fields(../../../../../apps/frappe/frappe/public/js/frappe/form/form.js:679:15) at (../../../../../apps/frappe/frappe/public/js/frappe/form/form.js:618:16)``` Signed-off-by: Akhil Narang --- frappe/public/js/frappe/form/controls/table_multiselect.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/form/controls/table_multiselect.js b/frappe/public/js/frappe/form/controls/table_multiselect.js index 52fe8e1faa..c4372e54ac 100644 --- a/frappe/public/js/frappe/form/controls/table_multiselect.js +++ b/frappe/public/js/frappe/form/controls/table_multiselect.js @@ -148,7 +148,7 @@ frappe.ui.form.ControlTableMultiSelect = class ControlTableMultiSelect extends ( get_link_field() { if (!this._link_field) { const meta = frappe.get_meta(this.df.options); - this._link_field = meta.fields?.find((df) => df.fieldtype === "Link"); + this._link_field = meta?.fields?.find((df) => df.fieldtype === "Link"); if (!this._link_field) { throw new Error("Table MultiSelect requires a Table with atleast one Link field"); }