diff --git a/report_custom_filename/controllers/reports.py b/report_custom_filename/controllers/reports.py index 011a88e1..8fd349bc 100644 --- a/report_custom_filename/controllers/reports.py +++ b/report_custom_filename/controllers/reports.py @@ -39,7 +39,14 @@ class Reports(main.Reports): for report in report_xml.browse(report_ids): if not report.download_filename: continue - objects = http.request.session.model(context['active_model'])\ + # When you are on the form view of a customer and you + # click on the "Sales" button, context['active_model'] + # is still res.partner (probably a bug...) but + # context['active_ids'] has the ID of the sale order + # cf https://github.com/OCA/reporting-engine/issues/56 + # That's why we use report.model instead of context['active_model'] + # together with context['active_ids'] + objects = http.request.session.model(report.model)\ .browse(context['active_ids']) generated_filename = email_template.mako_template_env\ .from_string(report.download_filename)\