[MIG] endpoint: Migration to 17.0

Author: SilvioC2C

endpoint/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Endpoint",
     "summary": """Provide custom endpoint machinery.""",
-    "version": "16.0.1.3.0",
+    "version": "17.0.1.0.0",
     "license": "LGPL-3",
     "development_status": "Beta",
     "author": "Camptocamp,Odoo Community Association (OCA)",

endpoint/models/endpoint_mixin.py

@@ -116,8 +116,30 @@ def _code_snippet_log_func(self, message, level="info"):
             cr.execute(
                 """
                 INSERT INTO ir_logging
-                (create_date, create_uid, type, dbname, name, level, message, path, line, func)
-                VALUES (NOW() at time zone 'UTC', %s, %s, %s, %s, %s, %s, %s, %s, %s)
+                (
+                    create_date,
+                    create_uid,
+                    type,
+                    dbname,
+                    name,
+                    level,
+                    message,
+                    path,
+                    line,
+                    func
+                )
+                VALUES (
+                    NOW() at time zone 'UTC',
+                    %s,
+                    %s,
+                    %s,
+                    %s,
+                    %s,
+                    %s,
+                    %s,
+                    %s,
+                    %s
+                )
                 """,
                 (
                     self.env.uid,
@@ -156,8 +178,9 @@ def _code_snippet_valued(self):
         )
 
     def _default_endpoint_options_handler(self):
+        kdp = "odoo.addons.endpoint.controllers.main.EndpointController"
         return {
-            "klass_dotted_path": "odoo.addons.endpoint.controllers.main.EndpointController",
+            "klass_dotted_path": kdp,
             "method_name": "auto_endpoint",
             "default_pargs": (self._name, self.route),
         }
@@ -206,10 +229,9 @@ def _find_endpoint_domain(self, endpoint_route):
         return [("route", "=", endpoint_route)]
 
     def copy_data(self, default=None):
-        result = super().copy_data(default=default)
-        # `route` cannot be copied as it must me unique.
+        # OVERRIDE: ``route`` cannot be copied as it must me unique.
         # Yet, we want to be able to duplicate a record from the UI.
-        for rec, data in zip(self, result):
-            if not data.get("route"):
-                data["route"] = f"{rec.route}/COPY_FIXME"
-        return result
+        self.ensure_one()
+        default = dict(default or {})
+        default.setdefault("route", f"{self.route}/COPY_FIXME")
+        return super().copy_data(default=default)

endpoint/tests/test_endpoint_controller.py

@@ -19,7 +19,8 @@ def setUpClass(cls):
         cls.env["endpoint.endpoint"].search([])._handle_registry_sync()
 
     def tearDown(self):
-        self.env["ir.http"]._clear_routing_map()
+        # Clear cache for method ``ir.http.routing_map()``
+        self.env.registry.clear_cache("routing")
         super().tearDown()
 
     def test_call1(self):

endpoint/views/endpoint_view.xml

@@ -16,13 +16,13 @@
                     name="web_ribbon"
                     title="Archived"
                     bg_color="bg-warning"
-                    attrs="{'invisible': [('active', '=', True)]}"
+                    invisible="active"
                 />
                 <sheet>
                     <div
                         class="alert alert-warning"
                         role="alert"
-                        attrs="{'invisible': ['|', ('registry_sync', '=', True), ('id', '=', False)]}"
+                        invisible="registry_sync or not id"
                     >
                         <p>
                             Registry out of sync. Likely the record has been modified but not sync'ed with the routing registry.
@@ -53,7 +53,7 @@
                                     <field name="exec_mode" />
                                     <field
                                         name="exec_as_user_id"
-                                        attrs="{'required': [('auth_type', '=', 'public')]}"
+                                        required="auth_type == 'public'"
                                     />
                                 </group>
                                 <group name="auth" string="Auth">
@@ -65,23 +65,19 @@
                                     <field name="request_method" />
                                     <field
                                         name="request_content_type"
-                                        attrs="{'required': [('request_method', 'in', ('POST', 'PUT'))],
-                                                   'invisible': [('request_method', 'not in', ('POST', 'PUT'))]}"
+                                        required="request_method in ('POST', 'PUT')"
+                                        invisible="request_method not in ('POST', 'PUT')"
                                     />
                                 </group>
                             </group>
                         </page>
-                        <page
-                            name="code"
-                            string="Code"
-                            attrs="{'invisible': [('exec_mode', '!=', 'code')]}"
-                        >
+                        <page name="code" string="Code" invisible="exec_mode != 'code'">
                             <field name="code_snippet" widget="ace" />
                         </page>
                         <page
                             name="code_help"
                             string="Code Help"
-                            attrs="{'invisible': [('exec_mode', '!=', 'code')]}"
+                            invisible="exec_mode != 'code'"
                         >
                             <field name="code_snippet_docs" />
                         </page>