From 159f6c9e1fdde47277c40c55cb5d3f578cbe2a0e Mon Sep 17 00:00:00 2001 From: Jigar Patel Date: Fri, 26 Feb 2016 17:25:09 +0530 Subject: [PATCH] [IMP] event: The timezone of the event should be required field because if the user does not set timezone then in website_event(frontend) for event's timezone is not intuitive --- addons/event/models/event.py | 2 +- addons/event/tests/common.py | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/addons/event/models/event.py b/addons/event/models/event.py index a40d29a46cd..03bc96d1d1a 100644 --- a/addons/event/models/event.py +++ b/addons/event/models/event.py @@ -118,7 +118,7 @@ class event_event(models.Model): 'event.registration', 'event_id', string='Attendees', readonly=False, states={'done': [('readonly', True)]}) # Date fields - date_tz = fields.Selection('_tz_get', string='Timezone', default=lambda self: self.env.user.tz) + date_tz = fields.Selection('_tz_get', string='Timezone', required=True, default=lambda self: self.env.user.tz) date_begin = fields.Datetime( string='Start Date', required=True, track_visibility='onchange', states={'done': [('readonly', True)]}) diff --git a/addons/event/tests/common.py b/addons/event/tests/common.py index 1b59c662da6..55e5b2c0200 100644 --- a/addons/event/tests/common.py +++ b/addons/event/tests/common.py @@ -28,6 +28,7 @@ class TestEventCommon(common.TransactionCase): 'login': 'Armande', 'alias_name': 'armande', 'email': 'armande.eventuser@example.com', + 'tz': 'Europe/Brussels', 'groups_id': [(6, 0, [self.group_employee_id, self.group_event_user_id])] }) self.user_eventmanager = self.Users.with_context({'no_reset_password': True}).create({ @@ -35,5 +36,6 @@ class TestEventCommon(common.TransactionCase): 'login': 'bastien', 'alias_name': 'bastien', 'email': 'bastien.eventmanager@example.com', + 'tz': 'Europe/Brussels', 'groups_id': [(6, 0, [self.group_employee_id, self.group_event_manager_id])] })