From ea5a37bb706a0609c1566ea2ecd3e52a2168bfb6 Mon Sep 17 00:00:00 2001 From: Suherdy Yacob Date: Wed, 28 Jan 2026 11:27:54 +0700 Subject: [PATCH] refactor: Remove redundant list conversion when combining Odoo Domain objects. --- models/restricted_models.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/models/restricted_models.py b/models/restricted_models.py index 78ae774..34db932 100644 --- a/models/restricted_models.py +++ b/models/restricted_models.py @@ -20,7 +20,7 @@ class StockWarehouse(models.Model): 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) 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) 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'): allowed_ids = get_allowed_ids(self.env, 'res_users_stock_picking_type_rel', 'picking_type_id', self.env.user.id) 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) class StockLocation(models.Model): @@ -52,7 +52,7 @@ class StockLocation(models.Model): ('id', 'child_of', allowed_ids), ('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) 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'): allowed_ids = get_allowed_ids(self.env, 'res_users_mrp_workcenter_rel', 'workcenter_id', self.env.user.id) 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)