From a1b4d44b62eec0a6ccbe73ce80e09944b6cd42b3 Mon Sep 17 00:00:00 2001 From: Akhil Narang Date: Mon, 11 Nov 2024 13:55:45 +0530 Subject: [PATCH] fix(list_view): check that a title field is set before trying to use it Otherwise enabling the checkbox for `Warehouse` and trying to load `Stock Ledger Entry` results in `warehouse. as warehouse_` Signed-off-by: Akhil Narang --- frappe/public/js/frappe/list/list_view.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/list/list_view.js b/frappe/public/js/frappe/list/list_view.js index 3f7745b435..7044200fd9 100644 --- a/frappe/public/js/frappe/list/list_view.js +++ b/frappe/public/js/frappe/list/list_view.js @@ -209,7 +209,7 @@ frappe.views.ListView = class ListView extends frappe.views.BaseList { ) { frappe.model.with_doctype(df.options, () => { const meta = frappe.get_meta(df.options); - if (meta.show_title_field_in_link) { + if (meta.show_title_field_in_link && meta.title_field) { this.link_field_title_fields[ typeof f === "string" ? f : f.fieldname ] = meta.title_field;