From 9720ff2a8cd38b17f56945a24c96f23b5cfa2fc2 Mon Sep 17 00:00:00 2001 From: Akhil Narang Date: Tue, 28 Jan 2025 16:05:59 +0530 Subject: [PATCH] Merge pull request #30907 from akhilnarang/fixup-27805 fix(report_view): skip read_only condition checking for reportview --- frappe/public/js/frappe/views/reports/report_view.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/frappe/public/js/frappe/views/reports/report_view.js b/frappe/public/js/frappe/views/reports/report_view.js index a91be847b6..9fb875aed5 100644 --- a/frappe/public/js/frappe/views/reports/report_view.js +++ b/frappe/public/js/frappe/views/reports/report_view.js @@ -714,6 +714,8 @@ frappe.views.ReportView = class ReportView extends frappe.views.ListView { !df.hidden && // not a standard field i.e., owner, modified_by, etc. frappe.model.is_non_std_field(df.fieldname) && + // don't check read_only_depends_on if there's child table fields + !this.meta.fields.some((df) => df.fieldtype === "Table") && df.read_only_depends_on && !this.evaluate_read_only_depends_on(df.read_only_depends_on, data) );