From 1db28b7472de233216126661e5f9f645e8a34cf0 Mon Sep 17 00:00:00 2001 From: "adrien.didenot" Date: Thu, 28 Sep 2017 17:28:17 +0200 Subject: [PATCH] [Fix OCA/project#317] Add the property date_delay to TimeLine views (same as base Calendar view) In Odoo calendar view, the attribute date_delay is an alternative to date_stop, to provides the duration of the event instead of its end date. This commit integrate this attribute to the Timeline view, but contrary to the base Calendar view the Timeline view could use both arguments (date_stop and date_delay) simultaneously. Signed-off-by: adrien.didenot --- web_timeline/README.rst | 3 +- web_timeline/static/src/js/web_timeline.js | 46 ++++++++++++++++------ 2 files changed, 34 insertions(+), 15 deletions(-) diff --git a/web_timeline/README.rst b/web_timeline/README.rst index c58102b0..61a9b6c8 100755 --- a/web_timeline/README.rst +++ b/web_timeline/README.rst @@ -22,9 +22,8 @@ the possible attributes for the tag: * date_end (optional): it defines the name of the field of type date that contains the end of the event. The date_end can be equal to the attribute date_start to display events has 'point' on the Timeline (instantaneous event) -* date_delay (optional): it defines the name of the field of type integer +* date_delay (optional): it defines the name of the field of type float/integer that contain the duration in hours of the event, default = 1 - contains the end of the event. * default_group_by (required): it defines the name of the field that will be taken as default group by when accessing the view or when no other group by is selected. diff --git a/web_timeline/static/src/js/web_timeline.js b/web_timeline/static/src/js/web_timeline.js index 7a31fa59..6369ff25 100644 --- a/web_timeline/static/src/js/web_timeline.js +++ b/web_timeline/static/src/js/web_timeline.js @@ -108,6 +108,7 @@ odoo.define('web_timeline.TimelineView', function (require) { } this.date_start = attrs.date_start; this.date_stop = attrs.date_stop; + this.date_delay = attrs.date_delay; this.no_period = this.date_start == this.date_stop; this.zoomKey = attrs.zoomKey || ''; @@ -194,9 +195,9 @@ odoo.define('web_timeline.TimelineView', function (require) { event_data_transform: function (evt) { var self = this; var date_start = new moment(); - var date_stop = new moment(); + var date_stop; - var date_delay = evt[this.date_delay] || 1.0, + var date_delay = evt[this.date_delay] || false, all_day = this.all_day ? evt[this.all_day] : false, res_computed_text = '', the_title = '', @@ -208,15 +209,19 @@ odoo.define('web_timeline.TimelineView', function (require) { } else { date_start = time.auto_str_to_date(evt[this.date_start].split(' ')[0], 'start'); - date_stop = this.date_stop ? time.auto_str_to_date(evt[this.date_stop].split(' ')[0], 'stop') : null; + if (this.no_period) { + date_stop = date_start + } else { + date_stop = this.date_stop ? time.auto_str_to_date(evt[this.date_stop].split(' ')[0], 'stop') : null; + } } - if (!date_start) { date_start = new moment(); } - if (!date_stop && !this.no_period) { + if (!date_stop && date_delay) { date_stop = moment(date_start).add(date_delay, 'hours').toDate(); } + var group = evt[self.last_group_bys[0]]; if (group) { group = _.first(group); @@ -236,7 +241,7 @@ odoo.define('web_timeline.TimelineView', function (require) { 'style': 'background-color: ' + self.color + ';' }; // Check if the event is instantaneous, if so, display it with a point on the timeline (no 'end') - if (!this.no_period && !moment(date_start).isSame(date_stop)) { + if (date_stop && !moment(date_start).isSame(date_stop)) { r.end = date_stop; } self.color = undefined; @@ -361,7 +366,12 @@ odoo.define('web_timeline.TimelineView', function (require) { // Initialize default values for creation var default_context = {}; default_context['default_'.concat(this.date_start)] = item.start; - default_context['default_'.concat(this.date_stop)] = moment(item.start).add(1, 'hours').toDate(); + if (this.date_delay) { + default_context['default_'.concat(this.date_delay)] = 1; + } + if (this.date_stop) { + default_context['default_'.concat(this.date_stop)] = moment(item.start).add(1, 'hours').toDate(); + } if (item.group > 0) { default_context['default_'.concat(this.last_group_bys[0])] = item.group; } @@ -412,17 +422,27 @@ odoo.define('web_timeline.TimelineView', function (require) { on_move: function (item, callback) { var self = this; - var start = item.start; - var end = item.end; + var event_start = item.start; + var event_end = item.end; var group = false; if (item.group != -1) { group = item.group; } var data = {}; - data[self.fields_view.arch.attrs.date_start] = - time.auto_date_to_str(start, self.fields[self.fields_view.arch.attrs.date_start].type); - data[self.fields_view.arch.attrs.date_stop] = - time.auto_date_to_str(end, self.fields[self.fields_view.arch.attrs.date_stop].type); + // In case of a move event, the date_delay stay the same, only date_start and stop must be updated + data[this.date_start] = time.auto_date_to_str(event_start, self.fields[this.date_start].type); + if (this.date_stop) { + // In case of instantaneous event, item.end is not defined + if (event_end) { + data[this.date_stop] = time.auto_date_to_str(event_end, self.fields[this.date_stop].type); + } else { + data[this.date_stop] = data[this.date_start] + } + } + if (this.date_delay && event_end) { + var diff_seconds = Math.round((event_end.getTime() - event_start.getTime()) / 1000); + data[this.date_delay] = diff_seconds / 3600; + } if (self.grouped_by) { data[self.grouped_by[0]] = group; }