From b483800a06bf5c988881686f06b89ef122c2ee3b Mon Sep 17 00:00:00 2001 From: Shariq Ansari Date: Thu, 7 Dec 2023 19:23:35 +0530 Subject: [PATCH] fix: pass filter to formatter to handle custom logic --- frappe/public/js/frappe/views/reports/query_report.js | 8 +++----- package.json | 2 +- yarn.lock | 8 ++++---- 3 files changed, 8 insertions(+), 10 deletions(-) diff --git a/frappe/public/js/frappe/views/reports/query_report.js b/frappe/public/js/frappe/views/reports/query_report.js index a31d1c21a1..37c1cd6ea8 100644 --- a/frappe/public/js/frappe/views/reports/query_report.js +++ b/frappe/public/js/frappe/views/reports/query_report.js @@ -1259,17 +1259,15 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList { width: parseInt(column.width) || null, editable: false, compareValue: compareFn, - format: (value, row, column, data, for_filter = false) => { - if (for_filter && column?.fieldtype === "Link") { - return value || ""; - } + format: (value, row, column, data, filter) => { if (this.report_settings.formatter) { return this.report_settings.formatter( value, row, column, data, - format_cell + format_cell, + filter ); } return format_cell(value, row, column, data); diff --git a/package.json b/package.json index 8e53dcb29a..cfde368887 100644 --- a/package.json +++ b/package.json @@ -47,7 +47,7 @@ "fast-deep-equal": "^2.0.1", "fast-glob": "^3.2.5", "frappe-charts": "2.0.0-rc22", - "frappe-datatable": "^1.17.2", + "frappe-datatable": "^1.17.5", "frappe-gantt": "^0.6.0", "highlight.js": "^10.4.1", "html5-qrcode": "^2.3.8", diff --git a/yarn.lock b/yarn.lock index 68fcad21cf..f6045227c5 100644 --- a/yarn.lock +++ b/yarn.lock @@ -1497,10 +1497,10 @@ frappe-charts@2.0.0-rc22: resolved "https://registry.yarnpkg.com/frappe-charts/-/frappe-charts-2.0.0-rc22.tgz#9a5a747febdc381a1d4d7af96e89cf519dfba8c0" integrity sha512-N7f/8979wJCKjusOinaUYfMxB80YnfuVLrSkjpj4LtyqS0BGS6SuJxUnb7Jl4RWUFEIs7zEhideIKnyLeFZF4Q== -frappe-datatable@^1.17.2: - version "1.17.4" - resolved "https://registry.yarnpkg.com/frappe-datatable/-/frappe-datatable-1.17.4.tgz#b003a8097f462fa5933f236590865be0cf370b44" - integrity sha512-uqtTzgdYITZM8hDvTEnmYnYRmPsoG+AjbTDls/NMxMBCkzRAE0VTVaPs5yag7HWoYJg/9LFW+sHwgZbVpjyBEA== +frappe-datatable@^1.17.5: + version "1.17.5" + resolved "https://registry.yarnpkg.com/frappe-datatable/-/frappe-datatable-1.17.5.tgz#342814d7f9eb502f55639c1b09d44d9eca782f78" + integrity sha512-FJdpsj/xACuk553FXMMqvtTQZII9P9TEtAwOiaKN+AwDAjfCO4b5vhSTmdNp2Kgf26bnZU9QVfRvh2gDtpj3OA== dependencies: hyperlist "^1.0.0-beta" lodash "^4.17.5"