[MIG] base_user_role_company: migrate to 19.0

- Replace groups_id with group_ids in views/actions
- Bump version to 19.0.1.0.0; adjust manifests/assets
- Use env attributes and Domain expressions
- Adopt new Constraint/Index APIs where applicable

Functional changes

Author: Milan Topuzov

base_user_role_company/controllers/main.py

@@ -8,9 +8,10 @@
 
 class HomeExtended(Home):
     @http.route()
-    def web_load_menus(self, unique):
-        response = super().web_load_menus(unique)
-        # On logout & re-login we could see wrong menus being rendered
-        # To avoid this, menu http cache must be disabled
-        response.headers.remove("Cache-Control")
+    def web_load_menus(self, lang=None):
+        # v19 signature: (lang=None). Keep behavior of disabling menu HTTP cache.
+        response = super().web_load_menus(lang=lang)
+        # Remove Cache-Control header to disable menu HTTP caching
+        # Using .pop() as Werkzeug Headers no longer supports __delitem__ (del)
+        response.headers.pop("Cache-Control", None)
         return response

base_user_role_company/models/role.py

@@ -25,18 +25,12 @@ def _check_company(self):
                 and record.company_id != record.user_id.company_id
                 and record.company_id not in record.user_id.company_ids
             ):
-                raise ValidationError(
-                    self.env._(
-                        'User "%(user)s" does not have access to the company '
-                        '"%(company)s"'
-                    )
-                    % {"user": record.user_id.name, "company": record.company_id.name}
-                )
+                msg = self.env._("User does not have access to the selected company")
+                raise ValidationError(msg)
 
-    _sql_constraints = [
-        (
-            "user_role_uniq",
-            "unique (user_id,role_id,company_id)",
-            "Roles can be assigned to a user only once at a time",
-        )
-    ]
+    # Override parent unique constraint to allow the same role multiple times
+    # for a user, provided company_id differs.
+    _user_role_uniq = models.Constraint(
+        "UNIQUE (user_id, role_id, company_id)",
+        "Roles can be assigned to a user only once at a time",
+    )

base_user_role_company/models/user.py

@@ -7,14 +7,13 @@
 class ResUsers(models.Model):
     _inherit = "res.users"
 
-    @classmethod
-    def authenticate(cls, db, credential, user_agent_env):
-        auth_info = super().authenticate(db, credential, user_agent_env)
-        # On login, ensure the proper roles are applied
-        # The last Role applied may not be the correct one,
-        # sonce the new session current company can be different
-        with cls.pool.cursor() as cr:
-            env = api.Environment(cr, auth_info["uid"], {})
+    def authenticate(self, credential, user_agent_env):
+        # v19 signature: instance method with (credential, user_agent_env)
+        auth_info = super().authenticate(credential, user_agent_env)
+        # Ensure proper roles are applied for the logged-in user
+        uid = auth_info.get("uid") if isinstance(auth_info, dict) else None
+        if uid:
+            env = api.Environment(self.env.cr, uid, {})
             if env.user.role_line_ids:
                 env.user.set_groups_from_roles()
         return auth_info

base_user_role_company/tests/test_role_per_company.py

@@ -49,15 +49,15 @@ def test_110_company_1(self):
             active_company_ids=self.company1.ids
         ).set_groups_from_roles()
         expected = self.groupA | self.groupB | self.groupC
-        found = self.test_user.groups_id.filtered(lambda x: x in expected)
+        found = self.test_user.group_ids.filtered(lambda x: x in expected)
         self.assertEqual(expected, found)
 
     def test_120_company_2(self):
         "Company 2 selected: Roles A and C are enabled"
         self.test_user.with_context(
             active_company_ids=self.company2.ids
         ).set_groups_from_roles()
-        enabled = self.test_user.groups_id
+        enabled = self.test_user.group_ids
         expected = self.groupA | self.groupC
         found = enabled.filtered(lambda x: x in expected)
         self.assertEqual(expected, found)
@@ -71,7 +71,7 @@ def test_130_all_company(self):
         self.test_user.with_context(
             active_company_ids=[self.company1.id, self.company2.id]
         ).set_groups_from_roles()
-        enabled = self.test_user.groups_id
+        enabled = self.test_user.group_ids
         expected = self.groupA | self.groupC
         found = enabled.filtered(lambda x: x in expected)
         self.assertEqual(expected, found)

base_user_role_company/tests/test_use_only_enabled_roles.py

@@ -46,7 +46,7 @@ def test_110_enabled_role_is_used(self):
             active_company_ids=self.company1.ids
         ).set_groups_from_roles()
         expected = self.groupA | self.groupB
-        found = self.test_user.groups_id.filtered(lambda x: x in expected)
+        found = self.test_user.group_ids.filtered(lambda x: x in expected)
         self.assertEqual(expected, found)
 
     def test_120_disabled_role_is_not_used(self):
@@ -58,5 +58,5 @@ def test_120_disabled_role_is_not_used(self):
             active_company_ids=self.company1.ids
         ).set_groups_from_roles()
         expected = self.groupA
-        found = self.test_user.groups_id.filtered(lambda x: x in expected)
+        found = self.test_user.group_ids.filtered(lambda x: x in expected)
         self.assertEqual(expected, found)