[MIG] report_substitute: Migration to 16.0

report_substitute/__init__.py

@@ -1,3 +1,2 @@
 from . import models
 from . import wizards
-from . import tests

report_substitute/__manifest__.py

@@ -6,7 +6,7 @@
     "summary": """
         This module allows to create substitution rules for report actions.
         """,
-    "version": "15.0.1.0.0",
+    "version": "16.0.1.0.0",
     "license": "AGPL-3",
     "author": "ACSONE SA/NV," "Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/reporting-engine",

report_substitute/demo/action_report.xml

@@ -3,14 +3,14 @@
     <template id="substitution_report">
         <div class="page">Substitution Report</div>
     </template>
-    <report
-        id="substitution_report_print"
-        string="Substitution For Technical guide"
-        model="ir.module.module"
-        report_type="qweb-pdf"
-        file="report_substitute.substitution_report"
-        name="report_substitute.substitution_report"
-    />
+    <record id="substitution_report_print" model="ir.actions.report">
+        <field name="name">Substitution For Technical guide</field>
+        <field name="model">ir.module.module</field>
+        <field name="report_type">qweb-pdf</field>
+        <field name="report_name">report_substitute.substitution_report</field>
+        <field name="report_file">report_substitute.substitution_report</field>
+        <field name="binding_type">report</field>
+    </record>
     <record id="substitution_rule_demo_1" model="ir.actions.report.substitution.rule">
         <field name="action_report_id" ref="base.ir_module_reference_print" />
         <field
@@ -21,12 +21,12 @@
     <template id="substitution_report_2">
         <div class="page">Substitution Report 2</div>
     </template>
-    <report
-        id="substitution_report_print_2"
-        string="Substitution 2 For Technical guide"
-        model="ir.module.module"
-        report_type="qweb-pdf"
-        file="report_substitute.substitution_report_2"
-        name="report_substitute.substitution_report_2"
-    />
+    <record id="substitution_report_print_2" model="ir.actions.report">
+        <field name="name">Substitution 2 For Technical guide</field>
+        <field name="model">ir.module.module</field>
+        <field name="report_type">qweb-pdf</field>
+        <field name="report_name">report_substitute.substitution_report_2</field>
+        <field name="report_file">report_substitute.substitution_report_2</field>
+        <field name="binding_type">report</field>
+    </record>
 </odoo>

report_substitute/models/ir_actions_report.py

@@ -50,9 +50,12 @@ def get_substitution_report_action(self, action, active_ids):
 
         return action
 
-    def _render(self, res_ids, data=None):
-        substitution_report = self.get_substitution_report(res_ids)
-        return super(IrActionReport, substitution_report)._render(res_ids, data)
+    def _render(self, report_ref, res_ids, data=None):
+        report = self._get_report(report_ref)
+        substitution_report = report.get_substitution_report(res_ids)
+        return super(IrActionReport, self)._render(
+            substitution_report.report_name, res_ids, data=data
+        )
 
     def report_action(self, docids, data=None, config=True):
         if docids:

report_substitute/tests/test_report_substitute.py

@@ -20,15 +20,27 @@ def setUp(self):
         ).id
 
     def test_substitution(self):
-        res = str(self.action_report.render(res_ids=self.res_ids)[0])
+        res = str(
+            self.action_report._render(
+                self.action_report.report_name, res_ids=self.res_ids
+            )[0]
+        )
         self.assertIn('<div class="page">Substitution Report</div>', res)
         # remove the substation rule
         self.substitution_rule.unlink()
-        res = str(self.action_report.render(res_ids=self.res_ids)[0])
+        res = str(
+            self.action_report._render(
+                self.action_report.report_name, res_ids=self.res_ids
+            )[0]
+        )
         self.assertNotIn('<div class="page">Substitution Report</div>', res)
 
     def test_recursive_substitution(self):
-        res = str(self.action_report.render(res_ids=self.res_ids)[0])
+        res = str(
+            self.action_report._render(
+                self.action_report.report_name, res_ids=self.res_ids
+            )[0]
+        )
         self.assertNotIn('<div class="page">Substitution Report 2</div>', res)
         self.env["ir.actions.report.substitution.rule"].create(
             {
@@ -40,15 +52,27 @@ def test_recursive_substitution(self):
                 ).id,
             }
         )
-        res = str(self.action_report.render(res_ids=self.res_ids)[0])
+        res = str(
+            self.action_report._render(
+                self.action_report.report_name, res_ids=self.res_ids
+            )[0]
+        )
         self.assertIn('<div class="page">Substitution Report 2</div>', res)
 
     def test_substitution_with_domain(self):
         self.substitution_rule.write({"domain": "[('name', '=', 'base')]"})
-        res = str(self.action_report.render(res_ids=self.res_ids)[0])
+        res = str(
+            self.action_report._render(
+                self.action_report.report_name, res_ids=self.res_ids
+            )[0]
+        )
         self.assertIn('<div class="page">Substitution Report</div>', res)
         self.substitution_rule.write({"domain": "[('name', '!=', 'base')]"})
-        res = str(self.action_report.render(res_ids=self.res_ids)[0])
+        res = str(
+            self.action_report._render(
+                self.action_report.report_name, res_ids=self.res_ids
+            )[0]
+        )
         self.assertNotIn('<div class="page">Substitution Report</div>', res)
 
     def test_substitution_with_action_dict(self):