[MIG] report_py3o: Migration to 19.0

Author: kopeyev

report_py3o/__manifest__.py

@@ -4,7 +4,7 @@
     "name": "Py3o Report Engine",
     "summary": "Reporting engine based on Libreoffice (ODT -> ODT, "
     "ODT -> PDF, ODT -> DOC, ODT -> DOCX, ODS -> ODS, etc.)",
-    "version": "18.0.1.0.2",
+    "version": "19.0.1.0.0",
     "category": "Reporting",
     "license": "AGPL-3",
     "author": "XCG Consulting, ACSONE SA/NV, Odoo Community Association (OCA)",

report_py3o/tests/test_report_py3o.py

@@ -26,9 +26,9 @@
 
 from odoo.addons.base.tests.test_mimetypes import PNG
 
-from ..models._py3o_parser_context import format_multiline_value
 from ..models.ir_actions_report import PY3O_CONVERSION_COMMAND_PARAMETER
-from ..models.py3o_report import TemplateNotFound
+from ..wizard._py3o_parser_context import format_multiline_value
+from ..wizard.py3o_report import TemplateNotFound
 
 logger = logging.getLogger(__name__)
 
@@ -91,8 +91,16 @@ def test_reports(self):
 
     def test_reports_merge_zip(self):
         self.report.py3o_filetype = "odt"
-        users = self.env["res.users"].search([])
-        self.assertTrue(len(users) > 0)
+        users = self.env["res.users"].search([], limit=2)
+        if len(users) < 2:
+            users |= self.env["res.users"].create(
+                {
+                    "name": "Test User 2",
+                    "login": f"test_user_2_{self.env.cr.dbname}",
+                    "email": f"test_user_2_{self.env.cr.dbname}@example.com",
+                }
+            )
+        self.assertGreaterEqual(len(users), 2)
         py3o_report = self.env["py3o.report"]
         _zip_results = self.py3o_report._zip_results
         with mock.patch.object(
@@ -147,7 +155,7 @@ def test_report_post_process(self):
         self.assertEqual(self.env.user.id, attachements.res_id)
         self.assertEqual(b"test result", b64decode(attachements.datas))
 
-    @tools.misc.mute_logger("odoo.addons.report_py3o.models.py3o_report")
+    @tools.misc.mute_logger("odoo.addons.report_py3o.wizard.py3o_report")
     def test_report_template_configs(self):
         # the demo template is specified with a relative path in in the module
         # path
@@ -169,9 +177,9 @@ def test_report_template_configs(self):
             self.report._render(self.report.id, self.env.user.ids)
         with temporary_copy(flbk_filename) as tmp_filename:
             self.report.py3o_template_fallback = tmp_filename
-            tools.config.misc["report_py3o"] = {
-                "root_tmpl_path": os.path.realpath(os.path.dirname(tmp_filename))
-            }
+            tools.config["root_tmpl_path"] = os.path.realpath(
+                os.path.dirname(tmp_filename)
+            )
             res = self.report._render(self.report.id, self.env.user.ids)
             self.assertTrue(res)
 
@@ -192,7 +200,7 @@ def test_report_template_configs(self):
         res = self.report._render(self.report.id, self.env.user.ids)
         self.assertTrue(res)
 
-    @tools.misc.mute_logger("odoo.addons.report_py3o.models.py3o_report")
+    @tools.misc.mute_logger("odoo.addons.report_py3o.wizard.py3o_report")
     def test_report_template_fallback_validity(self):
         tmpl_name = self.report.py3o_template_fallback
         with as_file(files(f"odoo.addons.{self.report.module}")) as _asf:

report_py3o/views/py3o_template.xml

@@ -7,13 +7,11 @@
             <search>
                 <field name="name" />
                 <field name="filetype" />
-                <group string="Group By" name="groupby">
-                    <filter
-                        name="filetype_groupby"
-                        string="File Type"
-                        context="{'group_by': 'filetype'}"
-                    />
-                </group>
+                <filter
+                    name="filetype_groupby"
+                    string="File Type"
+                    context="{'group_by': 'filetype'}"
+                />
             </search>
         </field>
     </record>

report_py3o/wizard/py3o_report.py

@@ -86,7 +86,7 @@ class Py3oReport(models.TransientModel):
     def _is_valid_template_path(self, path):
         """Check if the path is a trusted path for py3o templates."""
         real_path = os.path.realpath(path)
-        root_path = tools.config.get_misc("report_py3o", "root_tmpl_path")
+        root_path = tools.config.get("root_tmpl_path")
         if not root_path:
             logger.warning(
                 "You must provide a root template path into odoo.cfg to be "