diff --git a/addons/calendar/models/calendar.py b/addons/calendar/models/calendar.py index 6de011b2915fe5f08b7bf2bd789f0a385be0bbd1..01ad86558dc73bdaed147a8501ad5f1d9cc72181 100644 --- a/addons/calendar/models/calendar.py +++ b/addons/calendar/models/calendar.py @@ -738,13 +738,6 @@ class Meeting(models.Model): allday = fields.Boolean('All Day', states={'done': [('readonly', True)]}, default=False) start_date = fields.Date('Start Date', compute='_compute_dates', inverse='_inverse_dates', store=True, states={'done': [('readonly', True)]}, track_visibility='onchange') start_datetime = fields.Datetime('Start DateTime', compute='_compute_dates', inverse='_inverse_dates', store=True, states={'done': [('readonly', True)]}, track_visibility='onchange') - # FIXME - # If you wonder why `start_datetime` is sometimes not properly recomputed - # and desperately returns `False`, this is due to the override of `read()` - # hereunder that pollutes the cache at recomputing time. - # According to RCO, fixing this should require a redesing of recurring - # events and is probably not trivial... Use `start` instead! - stop_date = fields.Date('End Date', compute='_compute_dates', inverse='_inverse_dates', store=True, states={'done': [('readonly', True)]}, track_visibility='onchange') stop_datetime = fields.Datetime('End Datetime', compute='_compute_dates', inverse='_inverse_dates', store=True, states={'done': [('readonly', True)]}, track_visibility='onchange') # old date_deadline duration = fields.Float('Duration', states={'done': [('readonly', True)]}) @@ -1538,7 +1531,7 @@ class Meeting(models.Model): if not fields: fields = list(self._fields) fields2 = fields and fields[:] - EXTRAFIELDS = ('privacy', 'user_id', 'duration', 'allday', 'start', 'start_date', 'start_datetime', 'rrule') + EXTRAFIELDS = ('privacy', 'user_id', 'duration', 'allday', 'start', 'rrule') for f in EXTRAFIELDS: if fields and (f not in fields): fields2.append(f) diff --git a/addons/calendar/tests/test_calendar.py b/addons/calendar/tests/test_calendar.py index facdf9f70fd60c1a8ddbbb92d2a26e906c3d236e..e10976af980c7ed974f248fbf4d7be49ba6e1f01 100644 --- a/addons/calendar/tests/test_calendar.py +++ b/addons/calendar/tests/test_calendar.py @@ -4,8 +4,6 @@ import datetime from datetime import datetime, timedelta -from unittest2 import skip - from odoo import fields from odoo.tests.common import TransactionCase @@ -103,7 +101,6 @@ class TestCalendar(TransactionCase): self.assertEqual(calendar_event_sprint_review.byday, '1', 'rrule_type should be mothly') self.assertEqual(calendar_event_sprint_review.week_list, 'MO', 'rrule_type should be mothly') - @skip('Need to fix why start_datetime is not set randomly') def test_validation_error(self): """ Ideally this should build the base event in such a way that calling