admin.suherdy/access_restriction_by_user
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: Remove redundant list conversion when combining Odoo Domain objects.

Browse Source
This commit is contained in:
Suherdy Yacob 2026-01-28 11:27:54 +07:00
parent d32984dc04
commit ea5a37bb70
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
models/restricted_models.py
View File

@ -20,7 +20,7 @@ class StockWarehouse(models.Model):
if not self.env.su and not self.env.user.has_group('base.group_system'): if not self.env.su and not self.env.user.has_group('base.group_system'):
allowed_ids = get_allowed_ids(self.env, 'res_users_stock_warehouse_rel', 'warehouse_id', self.env.user.id) allowed_ids = get_allowed_ids(self.env, 'res_users_stock_warehouse_rel', 'warehouse_id', self.env.user.id)
if allowed_ids: if allowed_ids:
domain = list(Domain(domain or []) & Domain([('id', 'in', allowed_ids)])) domain = Domain(domain or []) & Domain([('id', 'in', allowed_ids)])
return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs) return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs)
class StockPickingType(models.Model): class StockPickingType(models.Model):
@ -33,7 +33,7 @@ class StockPickingType(models.Model):
if not self.env.su and not self.env.user.has_group('base.group_system'): if not self.env.su and not self.env.user.has_group('base.group_system'):
allowed_ids = get_allowed_ids(self.env, 'res_users_stock_picking_type_rel', 'picking_type_id', self.env.user.id) allowed_ids = get_allowed_ids(self.env, 'res_users_stock_picking_type_rel', 'picking_type_id', self.env.user.id)
if allowed_ids: if allowed_ids:
domain = list(Domain(domain or []) & Domain([('id', 'in', allowed_ids)])) domain = Domain(domain or []) & Domain([('id', 'in', allowed_ids)])
return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs) return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs)
class StockLocation(models.Model): class StockLocation(models.Model):
@ -52,7 +52,7 @@ class StockLocation(models.Model):
('id', 'child_of', allowed_ids), ('id', 'child_of', allowed_ids),
('usage', 'not in', ['internal', 'transit']) ('usage', 'not in', ['internal', 'transit'])
] ]
domain = list(Domain(domain or []) & Domain(restrict_domain)) domain = Domain(domain or []) & Domain(restrict_domain)
return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs) return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs)
class MrpWorkcenter(models.Model): class MrpWorkcenter(models.Model):
@ -65,6 +65,6 @@ class MrpWorkcenter(models.Model):
if not self.env.su and not self.env.user.has_group('base.group_system'): if not self.env.su and not self.env.user.has_group('base.group_system'):
allowed_ids = get_allowed_ids(self.env, 'res_users_mrp_workcenter_rel', 'workcenter_id', self.env.user.id) allowed_ids = get_allowed_ids(self.env, 'res_users_mrp_workcenter_rel', 'workcenter_id', self.env.user.id)
if allowed_ids: if allowed_ids:
domain = list(Domain(domain or []) & Domain([('id', 'in', allowed_ids)])) domain = Domain(domain or []) & Domain([('id', 'in', allowed_ids)])
return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs) return super()._search(domain, offset=offset, limit=limit, order=order, **kwargs)