From 5825dd1479d97056abd5e727d2da13a3e0709aff Mon Sep 17 00:00:00 2001
From: Franco Leyes <lef@adhoc.com.ar>
Date: Thu, 9 Jan 2025 09:02:46 -0300
Subject: [PATCH] [MIG] sales_team_security_crm: Migration to 17.0

---
 sales_team_security_crm/__manifest__.py                   | 2 +-
 sales_team_security_crm/security/security.xml             | 8 +++-----
 .../tests/test_sales_team_security_crm.py                 | 8 ++++++++
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/sales_team_security_crm/__manifest__.py b/sales_team_security_crm/__manifest__.py
index 1a88ce88c13..2c29b975ac9 100644
--- a/sales_team_security_crm/__manifest__.py
+++ b/sales_team_security_crm/__manifest__.py
@@ -4,7 +4,7 @@
 {
     "name": "CRM documents permissions by teams",
     "summary": "Integrates sales_team_security with crm",
-    "version": "16.0.1.0.0",
+    "version": "17.0.1.0.0",
     "category": "Customer Relationship Management",
     "website": "https://github.com/OCA/sale-workflow",
     "author": "Tecnativa, Iván Todorovich, Odoo Community Association (OCA)",
diff --git a/sales_team_security_crm/security/security.xml b/sales_team_security_crm/security/security.xml
index 57c6d7b0d2f..abfb20e75c4 100644
--- a/sales_team_security_crm/security/security.xml
+++ b/sales_team_security_crm/security/security.xml
@@ -5,8 +5,7 @@
         <field name="model_id" ref="crm.model_crm_lead" />
         <field
             name="domain_force"
-        >['|', '|', ('user_id', '=', user.id), ('user_id', '=', False), '|', ('team_id', '=', user.sale_team_id.id),
-            ('team_id', '=', False)]
+        >['|', ('user_id', 'in', [user.id, False]), ('team_id', 'in', user.crm_team_ids.ids + [False])]
         </field>
         <field
             name="groups"
@@ -18,8 +17,7 @@
         <field name="model_id" ref="crm.model_crm_activity_report" />
         <field
             name="domain_force"
-        >['|', '|', ('user_id', '=', user.id), ('user_id', '=', False), '|', ('team_id', '=', user.sale_team_id.id),
-            ('team_id', '=', False)]
+        >['|', ('user_id', 'in', [user.id, False]), ('team_id', 'in', user.crm_team_ids.ids + [False])]
         </field>
         <field
             name="groups"
@@ -29,7 +27,7 @@
     <record id="sales_team_team_rule" model="ir.rule">
         <field name="name">Own Sale Teams</field>
         <field name="model_id" ref="sales_team.model_crm_team" />
-        <field name="domain_force">[('id', '=', user.sale_team_id.id)]</field>
+        <field name="domain_force">[('id', 'in', user.crm_team_ids.ids)]</field>
         <field
             name="groups"
             eval="[(4, ref('sales_team_security.group_sale_team_manager'))]"
diff --git a/sales_team_security_crm/tests/test_sales_team_security_crm.py b/sales_team_security_crm/tests/test_sales_team_security_crm.py
index 904e415b294..d18c0637483 100644
--- a/sales_team_security_crm/tests/test_sales_team_security_crm.py
+++ b/sales_team_security_crm/tests/test_sales_team_security_crm.py
@@ -8,6 +8,11 @@ class TestSalesTeamSecurityCrm(TestCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
+        cls.env["ir.config_parameter"].set_param("sales_team.membership_multi", True)
+        cls.team3 = cls.env["crm.team"].create({"name": "Test channel 3"})
+        cls.env["crm.team.member"].create(
+            {"user_id": cls.user.id, "crm_team_id": cls.team3.id}
+        )
         cls.record = cls.env["crm.lead"].create(
             {
                 "name": "Test lead",
@@ -19,3 +24,6 @@ def setUpClass(cls):
 
     def test_crm_lead_permissions(self):
         self._check_whole_permission_set(extra_checks=False)
+
+    def test_crm_lead_permissions_multi_team(self):
+        self._check_permission(False, self.team3, True)