Skip to content

Commit

Permalink
[16.0][MIG] partner_event: Migration to version 16.0
Browse files Browse the repository at this point in the history
Biggest change is that _onchange_partner_id was removed in v16 core code so we no longer super it here. Instead, we call the _synchronize_partner_values method, which was the main job of the old _onchange_partner_id.
  • Loading branch information
kwoychesko committed Jun 23, 2023
1 parent fed5246 commit 0a41543
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 11 deletions.
4 changes: 2 additions & 2 deletions partner_event/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,13 @@

{
"name": "Link partner to events",
"version": "15.0.1.0.0",
"version": "16.0.1.0.0",
"category": "Marketing",
"author": "Tecnativa," "Odoo Community Association (OCA)",
"website": "https://github.com/OCA/event",
"development_status": "Production/Stable",
"license": "AGPL-3",
"depends": ["event"],
"depends": ["base", "event", "website"],
"data": [
"security/ir.model.access.csv",
"views/res_partner_view.xml",
Expand Down
14 changes: 10 additions & 4 deletions partner_event/models/event_registration.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,13 @@ def _onchange_partner_id(self):
get_attendee_partner_address = {
"get_attendee_partner_address": self.attendee_partner_id,
}
return super(
EventRegistration, self.with_context(**get_attendee_partner_address)
)._onchange_partner_id()
return super(EventRegistration, self)._onchange_partner_id()
# onchange for partner_id removed in v16 core - including functionality here
self = self.with_context(**get_attendee_partner_address)
for registration in self:
if registration.partner_id:
registration.update(
registration._synchronize_partner_values(
registration.partner_id
)
)
return {}
10 changes: 5 additions & 5 deletions partner_event/tests/test_event_registration.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,12 +63,12 @@ def test_count_registrations(self):
event_1 = self.event_0.copy()
self.assertEqual(self.partner_01.registration_count, 0)
self.registration_01.state = "open"
self.partner_01.invalidate_cache()
self.partner_01.invalidate_recordset()
self.assertEqual(self.partner_01.registration_count, 1)
self.registration_02.state = "done"
self.registration_02.attendee_partner_id = self.partner_01
self.registration_02.event_id = event_1
self.partner_01.invalidate_cache()
self.partner_01.invalidate_recordset()
self.assertEqual(self.partner_01.registration_count, 2)

def test_button_register(self):
Expand All @@ -81,16 +81,16 @@ def test_data_update(self):
event_2 = self.event_0.copy()
self.yesterday = datetime.now() - timedelta(days=1)
self.tomorrow = datetime.now() + timedelta(days=1)
self.last_moth = datetime.now() - timedelta(days=30)
self.last_month = datetime.now() - timedelta(days=30)
# Set an old event
event_2.write({"date_begin": self.last_moth})
event_2.write({"date_begin": self.last_month})
event_2.write({"date_end": self.yesterday})
self.registration_02.event_id = event_2
self.registration_02.attendee_partner_id = self.partner_01
# Update partner for an old event
self.partner_01.write({"email": "new@test.com"})
self.assertNotEqual(event_2.registration_ids.email, "new@test.com")
# Update partner for an current event
# Update partner for a current event
event_2.write({"date_end": self.tomorrow})
self.partner_01.write({"email": "new@test.com"})
self.assertEqual(event_2.registration_ids.email, "new@test.com")
Expand Down

0 comments on commit 0a41543

Please sign in to comment.