Merge pull request #20960 from AnandBaburajan/custom_report_export_and_auto_email
This commit is contained in:
commit
d4efef204f
2 changed files with 15 additions and 2 deletions
|
|
@ -298,7 +298,9 @@ def export_query():
|
|||
if isinstance(visible_idx, str):
|
||||
visible_idx = json.loads(visible_idx)
|
||||
|
||||
data = run(report_name, form_params.filters, custom_columns=custom_columns)
|
||||
data = run(
|
||||
report_name, form_params.filters, custom_columns=custom_columns, are_default_filters=False
|
||||
)
|
||||
data = frappe._dict(data)
|
||||
if not data.columns:
|
||||
frappe.respond_as_web_page(
|
||||
|
|
|
|||
|
|
@ -93,7 +93,7 @@ frappe.ui.form.on("Auto Email Report", {
|
|||
wrapper
|
||||
);
|
||||
|
||||
var filters = JSON.parse(frm.doc.filters || "{}");
|
||||
var filters = {};
|
||||
|
||||
let report_filters;
|
||||
|
||||
|
|
@ -102,8 +102,19 @@ frappe.ui.form.on("Auto Email Report", {
|
|||
frappe.query_reports[frm.doc.reference_report] &&
|
||||
frappe.query_reports[frm.doc.reference_report].filters
|
||||
) {
|
||||
if (frm.doc.filters) {
|
||||
filters = JSON.parse(frm.doc.filters);
|
||||
} else {
|
||||
frappe.db.get_value("Report", frm.doc.report, "json", (r) => {
|
||||
if (r && r.json) {
|
||||
filters = JSON.parse(r.json).filters || {};
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
report_filters = frappe.query_reports[frm.doc.reference_report].filters;
|
||||
} else {
|
||||
filters = JSON.parse(frm.doc.filters || "{}");
|
||||
report_filters = frappe.query_reports[frm.doc.report].filters;
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue