[MIG] base_user_effective_permissions: Migration to 18.0

Author: BhaveshHeliconia

base_user_effective_permissions/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Effective permissions",
     "summary": "Inspect effective permissions applying to a user",
-    "version": "16.0.1.0.0",
+    "version": "18.0.1.0.0",
     "development_status": "Alpha",
     "category": "Technical",
     "website": "https://github.com/OCA/server-backend",

base_user_effective_permissions/models/res_users.py

@@ -17,6 +17,7 @@ def action_show_effective_permissions(self):
             "type": "ir.actions.act_window",
             "name": _("Effective permissions for %s") % self.name,
             "res_model": "res.users.effective.permission",
-            "view_mode": "tree",
+            "view_mode": "list",
+            "views": [[False, "list"]],
             "domain": [("id", "in", permissions.ids)],
         }

base_user_effective_permissions/models/res_users_effective_permission.py

@@ -3,6 +3,7 @@
 
 
 from odoo import fields, models
+from odoo.exceptions import AccessError
 
 
 class ResUsersEffectivePermission(models.TransientModel):
@@ -13,7 +14,7 @@ class ResUsersEffectivePermission(models.TransientModel):
     model_id = fields.Many2one("ir.model", string="Model")
     model_name = fields.Char(related="model_id.model", string="Model name")
     model_human_name = fields.Char(
-        related="model_id.name", store=True, string="Human readable model name"
+        related="model_id.name", string="Human readable model name"
     )
     create_permission = fields.Boolean("Create")
     create_domain = fields.Char("Create restrictions")
@@ -49,21 +50,17 @@ def _generate_permissions(self, user):
                 .with_context(allowed_company_ids=user.company_id.ids)
             )
             vals = {"model_id": model_record.id}
-            vals.update(
-                {
-                    f"{operation}_permission": model.check_access_rights(
-                        operation, False
-                    )
-                    for operation in operations
-                }
-            )
-            vals.update(
-                {
-                    f"{operation}_domain": IrRule._compute_domain(
+            ir_model_access = self.env["ir.model.access"].with_user(user)
+            for operation in operations:
+                vals[f"{operation}_permission"] = ir_model_access.check(
+                    model._name, operation, raise_exception=False
+                )
+            for operation in operations:
+                try:
+                    vals[f"{operation}_domain"] = IrRule._compute_domain(
                         model._name, operation
                     )
-                    for operation in operations
-                }
-            )
+                except AccessError:
+                    vals[f"{operation}_domain"] = "[]"
             permissions += self.create(vals)
         return permissions

base_user_effective_permissions/views/res_users_effective_permission.xml

@@ -5,7 +5,7 @@
     <record id="res_users_effective_permission_tree" model="ir.ui.view">
         <field name="model">res.users.effective.permission</field>
         <field name="arch" type="xml">
-            <tree>
+            <list>
                 <field name="model_id" />
                 <field name="model_name" invisible="1" />
                 <field name="create_permission" optional="show" />
@@ -40,7 +40,7 @@
                     widget="domain"
                     options="{'model': 'model_name'}"
                 />
-            </tree>
+            </list>
         </field>
     </record>
     <record id="res_users_effective_permission_search" model="ir.ui.view">