diff --git a/report_xlsx/controllers/main.py b/report_xlsx/controllers/main.py index 25c79fe2b..01674c223 100644 --- a/report_xlsx/controllers/main.py +++ b/report_xlsx/controllers/main.py @@ -3,7 +3,8 @@ import json -from odoo.http import content_disposition, request, route +from odoo.http import content_disposition, request, route, serialize_exception +from odoo.tools import html_escape from odoo.tools.safe_eval import safe_eval from odoo.addons.web.controllers import main as report @@ -13,6 +14,13 @@ class ReportController(report.ReportController): @route() def report_routes(self, reportname, docids=None, converter=None, **data): if converter == "xlsx": + return self._report_routes_xlsx(reportname, docids, converter, **data) + return super(ReportController, self).report_routes( + reportname, docids, converter, **data + ) + + def _report_routes_xlsx(self, reportname, docids=None, converter=None, **data): + try: report = request.env["ir.actions.report"]._get_report_from_name(reportname) context = dict(request.env.context) if docids: @@ -42,6 +50,7 @@ class ReportController(report.ReportController): ("Content-Disposition", content_disposition(report_name + ".xlsx")), ] return request.make_response(xlsx, headers=xlsxhttpheaders) - return super(ReportController, self).report_routes( - reportname, docids, converter, **data - ) + except Exception as e: + se = serialize_exception(e) + error = {"code": 200, "message": "Odoo Server Error", "data": se} + return request.make_response(html_escape(json.dumps(error))) diff --git a/report_xlsx/readme/CONTRIBUTORS.rst b/report_xlsx/readme/CONTRIBUTORS.rst index 9e5dff7eb..8e49a2cb5 100644 --- a/report_xlsx/readme/CONTRIBUTORS.rst +++ b/report_xlsx/readme/CONTRIBUTORS.rst @@ -5,3 +5,4 @@ * Graeme Gellatly * Cristian Salamea * Rod Schouteden +* Eugene Molotov