Merge pull request #28331 from ljain112/fix-report-print
This commit is contained in:
commit
3da56632d7
1 changed files with 15 additions and 9 deletions
|
|
@ -1481,10 +1481,14 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList {
|
|||
get_filters_html_for_print() {
|
||||
const applied_filters = this.get_filter_values();
|
||||
return Object.keys(applied_filters)
|
||||
.filter((fieldname) => !frappe.utils.is_empty(applied_filters[fieldname]))
|
||||
.map((fieldname) => {
|
||||
const docfield = frappe.query_report.get_filter(fieldname).df;
|
||||
const value = applied_filters[fieldname];
|
||||
|
||||
if (frappe.utils.is_empty(value) || docfield.hidden_due_to_dependency) {
|
||||
return null;
|
||||
}
|
||||
|
||||
return `<div class="filter-row">
|
||||
<b>${__(docfield.label, null, docfield.parent)}:</b> ${frappe.format(value, docfield)}
|
||||
</div>`;
|
||||
|
|
@ -1532,14 +1536,16 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList {
|
|||
|
||||
let filters = this.get_filter_values(true);
|
||||
let boolean_labels = { 1: __("Yes"), 0: __("No") };
|
||||
let applied_filters = Object.fromEntries(
|
||||
Object.entries(filters).map(([key, value]) => [
|
||||
frappe.query_report.get_filter(key).df.label,
|
||||
frappe.query_report.get_filter(key).df.fieldtype == "Check"
|
||||
? boolean_labels[value]
|
||||
: value,
|
||||
])
|
||||
);
|
||||
let applied_filters = {};
|
||||
|
||||
for (const [key, value] of Object.entries(filters)) {
|
||||
const df = frappe.query_report.get_filter(key).df;
|
||||
if (!df.hidden_due_to_dependency) {
|
||||
applied_filters[df.label] =
|
||||
df.fieldtype === "Check" ? boolean_labels[value] : value;
|
||||
}
|
||||
}
|
||||
|
||||
if (this.prepared_report_name) {
|
||||
filters.prepared_report_name = this.prepared_report_name;
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue