vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix(list_view): check that a title field is set before trying to use it

Browse source 
Otherwise enabling the checkbox for `Warehouse` and trying to load `Stock Ledger Entry`
results in `warehouse. as warehouse_`

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
This commit is contained in:
Akhil Narang 2024-11-11 13:55:45 +05:30
parent c62e7334bc
commit a1b4d44b62
No known key found for this signature in database
GPG key ID: 9DCC61E211BF645F
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/public/js/frappe/list/list_view.js
View file

@ -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;