[15.0][MIG] base_report_to_printer: Migration to 15.0

base_report_to_printer/__manifest__.py

@@ -7,7 +7,7 @@
 
 {
     "name": "Report to printer",
-    "version": "14.0.2.0.0",
+    "version": "15.0.1.0.0",
     "category": "Generic Modules/Base",
     "author": "Agile Business Group & Domsense, Pegueroles SCP, NaN,"
     " LasLabs, Camptocamp, Odoo Community Association (OCA),"
@@ -18,7 +18,6 @@
     "data": [
         "data/printing_data.xml",
         "security/security.xml",
-        "views/assets.xml",
         "views/printing_printer.xml",
         "views/printing_server.xml",
         "views/printing_job.xml",
@@ -27,6 +26,11 @@
         "views/ir_actions_report.xml",
         "wizards/printing_printer_update_wizard_view.xml",
     ],
+    "assets": {
+        "web.assets_backend": [
+            "/base_report_to_printer/static/src/js/qweb_action_manager.js",
+        ],
+    },
     "installable": True,
     "application": False,
     "external_dependencies": {"python": ["pycups"]},

base_report_to_printer/models/printing_server.py

@@ -71,13 +71,16 @@ class PrintingServer(models.Model):
             connection = cups.Connection(host=self.address, port=self.port)
         except Exception:
             message = _(
-                "Failed to connect to the CUPS server on %s:%s. "
+                "Failed to connect to the CUPS server on %(address)s:%(port)s. "
                 "Check that the CUPS server is running and that "
                 "you can reach it from the Odoo server."
-            ) % (self.address, self.port)
+            ) % {
+                "address": self.address,
+                "port": self.port,
+            }
             _logger.warning(message)
             if raise_on_error:
-                raise exceptions.UserError(message)
+                raise exceptions.UserError(message) from Exception
 
         return connection
 

base_report_to_printer/tests/test_report.py

@@ -27,7 +27,10 @@ class TestReport(common.HttpCase):
             </t>""",
             }
         )
-        self.report_imd = self.env["ir.model.data"].create(
+        self.report_imd = (
+            self.env["ir.model.data"]
+            .sudo()
+            .create(
                 {
                     "name": "test",
                     "module": "base_report_to_printer",
@@ -35,6 +38,7 @@ class TestReport(common.HttpCase):
                     "res_id": self.report_view.id,
                 }
             )
+        )
         self.report = self.Model.create(
             {
                 "name": "Test",

base_report_to_printer/views/assets.xml

@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="utf-8" ?>
-<odoo>
-    <template
-        id="assets_backend"
-        name="base_report_to_printer assets"
-        inherit_id="web.assets_backend"
-    >
-        <xpath expr="." position="inside">
-            <script
-                type="text/javascript"
-                src="/base_report_to_printer/static/src/js/qweb_action_manager.js"
-            />
-        </xpath>
-    </template>
-</odoo>