Merge lp:~openerp-dev/openerp-web/7.0-opw-606297-msh into lp:openerp-web/7.0

[Mohammed Shekha(Open ERP)]

Hello,

Fixed the issue of calendar popup opens twice in chrome due to two event fires when one try to create multi day event using drag on month mode, onBeforeLightbox as well as click(which eventually trigger onEmptyClick).

Demo:- Run OpenERP with Chrome 33... or 34... and then just go to Calendar view say for example Human Resources -> Leaves -> Leave Requests and then in calendar view, create multi day leave by dragging mouse on two days in month mode.

You will notice that two popups created, instead of one.

Reason: When we drag mouse so dhtmlx scheduler will call onBeforeLightbox from Lightbox but Chrome also triggers click event(eventually onEmptyClick fired) when mouse is released.

Fixed it using workaround flag.

Thanks.

=== modified file 'addons/web_calendar/static/src/js/calendar.js'
--- addons/web_calendar/static/src/js/calendar.js	2014-02-14 11:42:58 +0000
+++ addons/web_calendar/static/src/js/calendar.js	2014-05-08 13:47:15 +0000
@@ -57,6 +57,7 @@
         this.range_stop = null;
         this.update_range_dates(Date.today());
         this.selected_filters = [];
+        this.is_slow_open = false;
     },
     view_loading: function(r) {
         return this.load_calendar(r);
@@ -465,6 +466,13 @@
     },
     slow_create: function(event_id, event_obj) {
         var self = this;
+        //Workaround for Chrome, Chrome triggers click event as well as onBeforeLightbox
+        //It is produced when one try to create event for multi day using drag
+        if (this.is_slow_open) {
+            scheduler.deleteEvent(event_id);
+            return;
+        }
+        this.is_slow_open = true;
         if (this.current_mode() === 'month') {
             event_obj['start_date'].addHours(8);
             if (event_obj._length === 1) {
@@ -487,6 +495,7 @@
             view_id: pop_infos.view_id,
         });
         pop.on('closed', self, function() {
+            this.is_slow_open = false;
             if (!something_saved) {
                 scheduler.deleteEvent(event_id);
             }