[MIG] partner_event: Migration to 19.0

nihelgabsi-acsone · CRogos · commit 99de351b3c91 · 2026-05-21T16:27:44.000Z
diff --git a/partner_event/README.rst b/partner_event/README.rst
@@ -21,13 +21,13 @@ Link partner to events
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fevent-lightgray.png?logo=github
-    :target: https://github.com/OCA/event/tree/18.0/partner_event
+    :target: https://github.com/OCA/event/tree/19.0/partner_event
     :alt: OCA/event
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/event-18-0/event-18-0-partner_event
+    :target: https://translation.odoo-community.org/projects/event-19-0/event-19-0-partner_event
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/event&target_branch=18.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/event&target_branch=19.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -74,7 +74,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/event/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/event/issues/new?body=module:%20partner_event%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/event/issues/new?body=module:%20partner_event%0Aversion:%2019.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -124,6 +124,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-This module is part of the `OCA/event <https://github.com/OCA/event/tree/18.0/partner_event>`_ project on GitHub.
+This module is part of the `OCA/event <https://github.com/OCA/event/tree/19.0/partner_event>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/partner_event/__manifest__.py b/partner_event/__manifest__.py
@@ -11,7 +11,7 @@
 
 {
     "name": "Link partner to events",
-    "version": "18.0.1.0.0",
+    "version": "19.0.1.0.0",
     "category": "Marketing",
     "author": "Tecnativa,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/event",
diff --git a/partner_event/hooks.py b/partner_event/hooks.py
@@ -5,14 +5,14 @@
 def post_init_hook(env):
     """Preload proper attendee partner for existing registrations using
     the same rules the module does"""
-    attendees_emails = env["event.registration"].read_group(
-        [("email", "!=", False)], ["email"], groupby="email"
+    attendees_emails = env["event.registration"].formatted_read_group(
+        domain=[("email", "!=", False)], groupby=["email"]
     )
     for email in attendees_emails:
         # Order was done for avoiding extra queries for sorting the results
         attendee_partner = env["res.partner"].search(
             [("email", "=ilike", email["email"])], limit=1, order="id"
         )
         if attendee_partner:
-            attendees = env["event.registration"].search(email["__domain"])
+            attendees = env["event.registration"].search(email["__extra_domain"])
             attendees.write({"attendee_partner_id": attendee_partner.id})
diff --git a/partner_event/models/event_registration.py b/partner_event/models/event_registration.py
@@ -94,7 +94,7 @@ def partner_data_update(self, data):
         if reg_data:
             # Only update registration data if this event is not old
             registrations = self.filtered(
-                lambda x: x.event_end_date >= fields.Datetime.now()
+                lambda x: x.event_id.date_end >= fields.Datetime.now()
             )
             registrations.write(reg_data)
 
@@ -112,13 +112,5 @@ def _onchange_partner_id(self):
                     vals = registration._synchronize_partner_values(
                         registration.partner_id
                     )
-                    # On Odoo 19.0 "mobile" field is removed
-                    # https://github.com/odoo/odoo/pull/189739
-                    # so this should not be ported to 19.0+
-                    if "mobile" not in registration._fields and "mobile" in vals:
-                        mobile = vals.pop("mobile")
-                        phone = vals.get("phone") or mobile
-                        if phone:
-                            vals["phone"] = phone
                     registration.update(vals)
         return {}
diff --git a/partner_event/static/description/index.html b/partner_event/static/description/index.html
@@ -374,7 +374,7 @@ <h1>Link partner to events</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:23c386ee6a1368b2996b3a08ee30b9f0bd78971c8b980dfa889c4635a2db9b86
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/event/tree/18.0/partner_event"><img alt="OCA/event" src="https://img.shields.io/badge/github-OCA%2Fevent-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/event-18-0/event-18-0-partner_event"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/event&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/event/tree/19.0/partner_event"><img alt="OCA/event" src="https://img.shields.io/badge/github-OCA%2Fevent-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/event-19-0/event-19-0-partner_event"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/event&amp;target_branch=19.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module links partners with the events they are registered through a
 smart button.</p>
 <p>It also includes:</p>
@@ -420,7 +420,7 @@ <h2><a class="toc-backref" href="#toc-entry-2">Bug Tracker</a></h2>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/event/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/event/issues/new?body=module:%20partner_event%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/event/issues/new?body=module:%20partner_event%0Aversion:%2019.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -467,7 +467,7 @@ <h3><a class="toc-backref" href="#toc-entry-6">Maintainers</a></h3>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/event/tree/18.0/partner_event">OCA/event</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/event/tree/19.0/partner_event">OCA/event</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>
diff --git a/partner_event/tests/test_event_registration.py b/partner_event/tests/test_event_registration.py
@@ -11,7 +11,6 @@
 from psycopg2 import IntegrityError
 
 from odoo import fields
-from odoo.tests import Form
 from odoo.tools import mute_logger
 
 from odoo.addons.base.tests.common import BaseCommon
@@ -54,26 +53,6 @@ def setUpClass(cls):
             }
         )
 
-        # On Odoo 19.0 "mobile" field is removed
-        # https://github.com/odoo/odoo/pull/189739
-        # so these example and related tests
-        # should not be ported to 19.0+
-        cls.partner_with_mobile = partner_model.create(
-            {
-                "name": "Test Partner with mobile",
-                "email": "email_with_mobile@test.com",
-                "mobile": "+1254728912",
-            }
-        )
-        cls.partner_with_phone_and_mobile = partner_model.create(
-            {
-                "name": "Test Partner with mobile and phone",
-                "email": "email_with_mobile_and_phone@test.com",
-                "phone": "254728913",
-                "mobile": "+1254728913",
-            }
-        )
-
     def test_create(self):
         self.assertEqual(self.partner_01.name, self.registration_01.name)
         self.assertEqual(self.partner_01.email, self.registration_01.email)
@@ -132,40 +111,3 @@ def test_delete_registered_partner(self):
         partner3 = self.env["res.partner"].create({"name": "unregistered partner"})
         partner3.unlink()
         self.assertFalse(partner3.exists())
-
-    def test_partner_only_mobile(self):
-        reg_form = Form(self.env["event.registration"])
-        reg_form.event_id = self.event_0
-        reg_form.attendee_partner_id = self.partner_with_mobile
-        reg = reg_form.save()
-        self.assertEqual(reg.partner_id, self.partner_with_mobile)
-        self.assertEqual(reg.phone, self.partner_with_mobile.mobile)
-
-    def test_partner_mobile_and_phone(self):
-        reg_form = Form(self.env["event.registration"])
-        reg_form.event_id = self.event_0
-        reg_form.attendee_partner_id = self.partner_with_phone_and_mobile
-        reg = reg_form.save()
-        self.assertEqual(reg.phone, self.partner_with_phone_and_mobile.phone)
-        self.assertNotEqual(
-            reg.phone,
-            self.partner_with_phone_and_mobile.mobile,
-            "Incorrect test. Partners phone and mobile must differ",
-        )
-
-    @mute_logger("odoo.models.unlink")
-    def test_action_merge(self):
-        partner_1 = self.partner_with_mobile
-        partner_2 = self.partner_with_phone_and_mobile
-        self.registration_01.partner_id = partner_1
-        self.registration_02.partner_id = partner_2
-        partners = partner_1 + partner_2
-        wizard = (
-            self.env["base.partner.merge.automatic.wizard"]
-            .with_context(active_ids=partners.ids, active_model=partners._name)
-            .create({})
-        )
-        self.assertEqual(wizard.dst_partner_id, partner_2)
-        wizard.action_merge()
-        self.assertEqual(self.registration_01.partner_id, partner_2)
-        self.assertEqual(self.registration_02.partner_id, partner_2)

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`
`12`	`12`	`{`
`13`	`13`	`"name": "Link partner to events",`
`14`		`- "version": "18.0.1.0.0",`
	`14`	`+ "version": "19.0.1.0.0",`
`15`	`15`	`"category": "Marketing",`
`16`	`16`	`"author": "Tecnativa,Odoo Community Association (OCA)",`
`17`	`17`	`"website": "https://github.com/OCA/event",`