[MIG] crm_partner_assign: Migration to 17.0

OCA · Feb 28, 2025 · 506712b · 506712b
1 parent 87444d5
commit 506712b
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 11 deletions.
diff --git a/crm_partner_assign/__manifest__.py b/crm_partner_assign/__manifest__.py
@@ -3,8 +3,10 @@
 
 {
     "name": "CRM Partner Assign",
-    "summary": "Assign a Partner to an Opportunity/Lead/Partner to indicate Partnership",
-    "version": "16.0.0.1.2",
+    "summary": """
+        Assign a Partner to an Opportunity/Lead/Partner to indicate Partnership,
+    """,
+    "version": "17.0.1.0.0",
     "development_status": "Alpha",
     "category": "Sales/CRM",
     "website": "https://github.com/OCA/crm",

diff --git a/crm_partner_assign/models/crm_lead.py b/crm_partner_assign/models/crm_lead.py
@@ -21,13 +21,11 @@ class CRMLead(models.Model):
         help="Partner this case has been assigned to.",
         related="partner_contact_assigned_id.commercial_partner_id",
         store=True,
-        readonly=True,
         check_company=True,
     )
     date_partner_assign = fields.Date(
         compute="_compute_date_partner_assign",
         string="Partner Assignment Date",
-        readonly=False,
         store=True,
         copy=True,
         help="Last date this case was assigned to a partner",

diff --git a/crm_partner_assign/models/res_partner.py b/crm_partner_assign/models/res_partner.py
@@ -16,12 +16,18 @@ class ResPartner(models.Model):
         inverse_name="assigned_partner_id",
         string="Implementation References",
     )
-    implemented_count = fields.Integer(
+    implemented_partner_count = fields.Integer(
         compute="_compute_implemented_partner_count",
         store=True,
     )
 
-    @api.depends("implemented_partner_ids", "implemented_partner_ids.active")
+    @api.depends("implemented_partner_ids.active")
     def _compute_implemented_partner_count(self):
+        rg_result = self.env["res.partner"]._read_group(
+            [("assigned_partner_id", "in", self.ids)],
+            ["assigned_partner_id"],
+            ["__count"],
+        )
+        rg_data = {assigned_partner.id: count for assigned_partner, count in rg_result}
         for partner in self:
-            partner.implemented_count = len(partner.implemented_partner_ids)
+            partner.implemented_partner_count = rg_data.get(partner.id, 0)
diff --git a/crm_partner_assign/views/crm_lead_view.xml b/crm_partner_assign/views/crm_lead_view.xml
@@ -13,14 +13,14 @@
                 position="before"
             >
                 <field name="partner_contact_assigned_id" />
-                <field name="partner_assigned_id" />
+                <field name="partner_assigned_id" readonly="True" />
             </xpath>
             <xpath
                 expr="//group[@name='opportunity_partner']/label[@for='email_from']"
                 position="before"
             >
                 <field name="partner_contact_assigned_id" />
-                <field name="partner_assigned_id" />
+                <field name="partner_assigned_id" readonly="True" />
             </xpath>
         </field>
     </record>
@@ -31,7 +31,7 @@
         <field name="inherit_id" ref="crm.crm_case_tree_view_leads" />
         <field name="arch" type="xml">
             <xpath expr="//tree/field[@name='country_id']" position="after">
-                <field name="partner_assigned_id" optional="hide" />
+                <field name="partner_assigned_id" readonly="True" optional="hide" />
                 <field name="partner_contact_assigned_id" optional="hide" />
             </xpath>
         </field>
@@ -43,7 +43,7 @@
         <field name="inherit_id" ref="crm.crm_case_tree_view_oppor" />
         <field name="arch" type="xml">
             <xpath expr="//tree/field[@name='country_id']" position="after">
-                <field name="partner_assigned_id" optional="hide" />
+                <field name="partner_assigned_id" readonly="True" optional="hide" />
                 <field name="partner_contact_assigned_id" optional="hide" />
             </xpath>
         </field>