diff --git a/report_xlsx_helper/__manifest__.py b/report_xlsx_helper/__manifest__.py index 3e03a18734..c61a477438 100644 --- a/report_xlsx_helper/__manifest__.py +++ b/report_xlsx_helper/__manifest__.py @@ -6,7 +6,7 @@ "author": "Noviat, Odoo Community Association (OCA)", "website": "https://github.com/OCA/reporting-engine", "category": "Reporting", - "version": "13.0.1.1.0", + "version": "14.0.1.0.0", "license": "AGPL-3", "depends": ["report_xlsx"], "installable": True, diff --git a/report_xlsx_helper/controllers/main.py b/report_xlsx_helper/controllers/main.py index 115832bc9c..cc64289f1c 100644 --- a/report_xlsx_helper/controllers/main.py +++ b/report_xlsx_helper/controllers/main.py @@ -37,7 +37,7 @@ def report_routes(self, reportname, docids=None, converter=None, **data): context.update(data["context"]) context["report_name"] = reportname - xlsx = report.with_context(context).render_xlsx(docids, data=data)[0] + xlsx = report.with_context(context)._render_xlsx(docids, data=data)[0] report_file = context.get("report_file") if not report_file: active_model = context.get("active_model", "export") @@ -52,6 +52,4 @@ def report_routes(self, reportname, docids=None, converter=None, **data): ("Content-Disposition", content_disposition(report_file + ".xlsx")), ] return request.make_response(xlsx, headers=xlsxhttpheaders) - return super(ReportController, self).report_routes( - reportname, docids, converter, **data - ) + return super().report_routes(reportname, docids, converter, **data) diff --git a/report_xlsx_helper/models/ir_actions_report.py b/report_xlsx_helper/models/ir_actions_report.py index b0a12fd690..c64657298b 100644 --- a/report_xlsx_helper/models/ir_actions_report.py +++ b/report_xlsx_helper/models/ir_actions_report.py @@ -9,11 +9,11 @@ class IrActionsReport(models.Model): _inherit = "ir.actions.report" @api.model - def render_xlsx(self, docids, data): + def _render_xlsx(self, docids, data): if not self and self.env.context.get("report_name"): report_model_name = "report.{}".format(self.env.context["report_name"]) report_model = self.env.get(report_model_name) if report_model is None: raise UserError(_("%s model was not found" % report_model_name)) return report_model.create_xlsx_report(docids, data) - return super(IrActionsReport, self).render_xlsx(docids, data) + return super()._render_xlsx(docids, data) diff --git a/report_xlsx_helper/report/report_xlsx_abstract.py b/report_xlsx_helper/report/report_xlsx_abstract.py index 479560ae12..bb02878058 100644 --- a/report_xlsx_helper/report/report_xlsx_abstract.py +++ b/report_xlsx_helper/report/report_xlsx_abstract.py @@ -10,6 +10,8 @@ from odoo import _, fields, models from odoo.exceptions import UserError +_xxx = {} + class ReportXlsxAbstract(models.AbstractModel): _inherit = "report.report_xlsx.abstract" diff --git a/report_xlsx_helper/tests/test_report_xlsx_helper.py b/report_xlsx_helper/tests/test_report_xlsx_helper.py index 6ffa41259c..461484d36a 100644 --- a/report_xlsx_helper/tests/test_report_xlsx_helper.py +++ b/report_xlsx_helper/tests/test_report_xlsx_helper.py @@ -23,5 +23,5 @@ def setUp(self): self.report = self.env["ir.actions.report"].with_context(ctx) def test_report_xlsx_helper(self): - report_xls = self.report.render_xlsx(None, None) + report_xls = self.report._render_xlsx(None, None) self.assertEqual(report_xls[1], "xlsx") diff --git a/setup/report_xlsx_helper/odoo/addons/report_xlsx_helper b/setup/report_xlsx_helper/odoo/addons/report_xlsx_helper new file mode 120000 index 0000000000..b860d1726c --- /dev/null +++ b/setup/report_xlsx_helper/odoo/addons/report_xlsx_helper @@ -0,0 +1 @@ +../../../../report_xlsx_helper \ No newline at end of file diff --git a/setup/report_xlsx_helper/setup.py b/setup/report_xlsx_helper/setup.py new file mode 100644 index 0000000000..28c57bb640 --- /dev/null +++ b/setup/report_xlsx_helper/setup.py @@ -0,0 +1,6 @@ +import setuptools + +setuptools.setup( + setup_requires=['setuptools-odoo'], + odoo_addon=True, +)