From daf1553d70e3e4d2a38a8873c30f47932873725a Mon Sep 17 00:00:00 2001 From: Saif Date: Sat, 2 Nov 2019 11:05:16 +0500 Subject: [PATCH] fix(Query Report View): Do not force set filters from URL Args No need to force set filters from URL Args It is already set in the method `set_route_filters` to the Query Report's model Currently if there are URL Arguments then even if you change the filters in the report view, the URL Args will override them. --- frappe/public/js/frappe/views/reports/query_report.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/frappe/public/js/frappe/views/reports/query_report.js b/frappe/public/js/frappe/views/reports/query_report.js index 1eb5ff1cbd..4b72a6b7b5 100644 --- a/frappe/public/js/frappe/views/reports/query_report.js +++ b/frappe/public/js/frappe/views/reports/query_report.js @@ -287,12 +287,6 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList { refresh() { this.toggle_message(true); let filters = this.get_filter_values(true); - let query = frappe.utils.get_query_string(frappe.get_route_str()); - - if(query) { - let obj = frappe.utils.get_query_params(query); - filters = Object.assign(filters || {}, obj); - } // only one refresh at a time if (this.last_ajax) {