openerp-community-reviewer team mailing list archive

[Pedro Manuel Baeza <pedro.baeza@xxxxxxxxx>]

Pedro Manuel Baeza has proposed merging lp:~pedro.baeza/ocb-web/7.0-web_calendar-fix-1306136 into lp:ocb-web.

Requested reviews:
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #1306136 in OpenERP Community Backports (Web): "[7.0] web_calendar: Error while removing a task from calendar view on week or day layout"
  https://bugs.launchpad.net/ocb-web/+bug/1306136

For more details, see:
https://code.launchpad.net/~pedro.baeza/ocb-web/7.0-web_calendar-fix-1306136/+merge/215256

[FIX] web_calendar: Error when deleting an element from calendar view.

It solves lp:1306136. See bug report for details.
-- 
https://code.launchpad.net/~pedro.baeza/ocb-web/7.0-web_calendar-fix-1306136/+merge/215256
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~pedro.baeza/ocb-web/7.0-web_calendar-fix-1306136 into lp:ocb-web.

=== 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-04-10 17:03:19 +0000
@@ -533,7 +533,7 @@
         var self = this;
         var index = this.dataset.get_id_index(event_id);
         if (index !== null) {
-            this.dataset.unlink(this.dataset.ids[index]);
+            this.dataset.unlink([this.dataset.ids[index]]);
         }
     },
 });