Fix error when all day event is hidden

Signed-off-by: Jan Schneider <jan@horde.org> Bug: 13433
horde · Aug 19, 2014 · 566a6cb · 566a6cb
1 parent 5b7b665
commit 566a6cb
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/kronolith/docs/CHANGES b/kronolith/docs/CHANGES
@@ -2,6 +2,8 @@
 v4.2.2-git
 ----------
 
+[jan] Fix JavaScript error if not showing all events in month view (Jasper
+      Olbrich <Jasper.Olbrich@students.uni-marburg.de>, Bug #13433).
 [jan] Fix displayed CalDAV URLs for shared calendars in basic view too
       (piper@hrz.uni-marburg.de, Bug #12380).
 

diff --git a/kronolith/js/kronolith.js b/kronolith/js/kronolith.js
@@ -1614,6 +1614,8 @@ KronolithCore = {
                                         // We don't want more than one all-day
                                         // event.
                                         var elm = allDays.pop();
+                                        // Remove element from events as well.
+                                        events = events.without(elm);
                                         elm.purge();
                                         elm.remove();
                                         insertBefore = this.findInsertBefore(events, event, date);

diff --git a/kronolith/package.xml b/kronolith/package.xml
@@ -33,6 +33,7 @@
  </stability>
  <license uri="http://www.horde.org/licenses/gpl">GPL-2.0</license>
  <notes>
+* [jan] Fix JavaScript error if not showing all events in month view (Jasper Olbrich &lt;Jasper.Olbrich@students.uni-marburg.de&gt;, Bug #13433).
 * [jan] Fix displayed CalDAV URLs for shared calendars in basic view too (piper@hrz.uni-marburg.de, Bug #12380).
  </notes>
  <contents>
@@ -2441,6 +2442,7 @@
    <date>2014-08-04</date>
    <license uri="http://www.horde.org/licenses/gpl">GPL-2.0</license>
    <notes>
+* [jan] Fix JavaScript error if not showing all events in month view (Jasper Olbrich &lt;Jasper.Olbrich@students.uni-marburg.de&gt;, Bug #13433).
 * [jan] Fix displayed CalDAV URLs for shared calendars in basic view too (piper@hrz.uni-marburg.de, Bug #12380).
    </notes>
   </release>