diff --git a/addons/website_event_meet/models/event_meeting_room.py b/addons/website_event_meet/models/event_meeting_room.py index 8e56889e567..b8e17ed8f39 100644 --- a/addons/website_event_meet/models/event_meeting_room.py +++ b/addons/website_event_meet/models/event_meeting_room.py @@ -24,6 +24,7 @@ class EventMeetingRoom(models.Model): event_id = fields.Many2one("event.event", string="Event", required=True, ondelete="cascade") is_pinned = fields.Boolean("Is Pinned") chat_room_id = fields.Many2one("chat.room", required=True, ondelete="restrict") + room_max_capacity = fields.Selection(default="8", copy=True) summary = fields.Char("Summary", translate=True) target_audience = fields.Char("Audience", translate=True) diff --git a/addons/website_event_meet/tests/test_website_event_meet.py b/addons/website_event_meet/tests/test_website_event_meet.py index a6687a3623f..8d1b8a95b0f 100644 --- a/addons/website_event_meet/tests/test_website_event_meet.py +++ b/addons/website_event_meet/tests/test_website_event_meet.py @@ -25,10 +25,11 @@ class TestWebsiteEventMeet(TestEventCommon): "event_id": self.event_0.id, "target_audience": "dev" }) - self.assertTrue(meeting_room.chat_room_id) self.assertTrue(meeting_room.chat_room_id.name) self.assertEqual(meeting_room.chat_room_id.max_capacity, "8") + # Ensure default value for room_max_capacity in event.meeting.room + self.assertEqual(meeting_room.room_max_capacity, "8") def test_meeting_room_copy(self): """Test the duplication of the meeting room.""" @@ -54,6 +55,10 @@ class TestWebsiteEventMeet(TestEventCommon): meeting_room_3 = meeting_room_1.copy() self.assertEqual(meeting_room_3.room_name, 'odoo-room-test-meeting-room-2') + # Ensure room_max_capacity is copied to new meeting room + self.assertEqual(meeting_room_3.room_max_capacity, "20") + # Ensure max_capacity in linked chat room is same + self.assertEqual(meeting_room_3.chat_room_id.max_capacity, "20") def test_meeting_room_unlink(self): """Test the duplication of the meeting room.""" diff --git a/addons/website_jitsi/models/chat_room_mixin.py b/addons/website_jitsi/models/chat_room_mixin.py index 247872be9fc..41e9f7c92d2 100644 --- a/addons/website_jitsi/models/chat_room_mixin.py +++ b/addons/website_jitsi/models/chat_room_mixin.py @@ -38,7 +38,7 @@ class ChatRoomMixin(models.AbstractModel): if any(values.get(fmatch[0]) for fmatch in self.ROOM_CONFIG_FIELDS) and not values.get('chat_room_id'): if values.get('room_name'): values['room_name'] = self._jitsi_sanitize_name(values['room_name']) - room_values = dict((fmatch[1], values.pop(fmatch[0])) for fmatch in self.ROOM_CONFIG_FIELDS if values.get(fmatch[0])) + room_values = dict((fmatch[1], values[fmatch[0]]) for fmatch in self.ROOM_CONFIG_FIELDS if values.get(fmatch[0])) values['chat_room_id'] = self.env['chat.room'].create(room_values).id return super(ChatRoomMixin, self).create(values_list)