Merge pull request frappe#27001 from frappe/revert-26893-query-report…

…-filters

Revert "fix: query report filters"

frappe/public/js/frappe/views/reports/query_report.js

@@ -1330,14 +1330,7 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList {
 		raise && this.toggle_message(false);
 
 		return this.filters
-			.filter((f) => {
-				const filter_value = f.get_value();
-				if (typeof filter_value === "object") {
-					return filter_value.length > 0;
-				} else {
-					return filter_value;
-				}
-			})
+			.filter((f) => f.get_value())
 			.map((f) => {
 				var v = f.get_value();
 				// hidden fields dont have $input
@@ -1480,23 +1473,16 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList {
 
 	get_filters_html_for_print() {
 		const applied_filters = this.get_filter_values();
-		const filter_html = Object.keys(applied_filters)
+		return Object.keys(applied_filters)
 			.map((fieldname) => {
 				const docfield = frappe.query_report.get_filter(fieldname).df;
 				const value = applied_filters[fieldname];
-				return `<div class="filter-row">
-					<b>${__(docfield.label, null, docfield.parent)}:</b> ${frappe.format(value, docfield)}
-				</div>`;
+				return `<h6>${__(docfield.label, null, docfield.parent)}: ${frappe.format(
+					value,
+					docfield
+				)}</h6>`;
 			})
 			.join("");
-
-		return `<div>${filter_html}</div>
-			<style>
-				.filter-row div {
-					/* prevent newline + right alignment of number fields */
-					display: inline-block;
-				}
-			</style>`;
 	}
 
 	export_report() {