6.0.0-alpha14
7/2/25

[#13433] JS error when all day event is hidden
Summary JS error when all day event is hidden
Queue Kronolith
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester Jasper.Olbrich (at) students (dot) uni-marburg (dot) de
Created 08/12/2014 (3977 days ago)
Due
Updated 08/19/2014 (3970 days ago)
Assigned
Resolved 08/19/2014 (3970 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
08/19/2014 09:09:17 AM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (master):

commit dcef41d425941fabafc63d201ad416178caef34d
Author: Jasper Olbrich <Jasper.Olbrich@students.uni-marburg.de>
Date:   Tue Aug 12 13:58:36 2014 +0200

     Fix error when all day event is hidden

     Signed-off-by: Jan Schneider <jan@horde.org>
     Bug: 13433

     Conflicts:
             kronolith/docs/CHANGES
             kronolith/package.xml

  kronolith/js/kronolith.js |    2 ++
  1 files changed, 2 insertions(+), 0 deletions(-)

http://github.com/horde/horde/commit/dcef41d425941fabafc63d201ad416178caef34d
08/19/2014 09:02:53 AM Jan Schneider Comment #3
Assigned to Jan Schneider
State ⇒ Resolved
Reply to this comment
Thanks!
08/19/2014 09:02:02 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2):

commit 566a6cb44bccc653951c69007c58bcabc5a94ed2
Author: Jasper Olbrich <Jasper.Olbrich@students.uni-marburg.de>
Date:   Tue Aug 12 13:58:36 2014 +0200

     Fix error when all day event is hidden

     Signed-off-by: Jan Schneider <jan@horde.org>
     Bug: 13433

  kronolith/docs/CHANGES    |    2 ++
  kronolith/js/kronolith.js |    2 ++
  kronolith/package.xml     |    2 ++
  3 files changed, 6 insertions(+), 0 deletions(-)

http://github.com/horde/horde/commit/566a6cb44bccc653951c69007c58bcabc5a94ed2
08/12/2014 12:07:22 PM Jasper (dot) Olbrich (at) students (dot) uni-marburg (dot) de Comment #1
Priority ⇒ 1. Low
New Attachment: hidden_all_day_events.patch Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ JS error when all day event is hidden
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
If a user has limited the number of events to show in month view, the 
following error can occur:
TypeError: elm.retrieve(...) is undefined

To reproduce:
1) Create more than max_events all-day-events on one day.
2) Navigate to the next month and back again.
3) The TypeError occurs and the JS on the page stops working.

I assume that this is because (~ line 1616 in kronolith.js)

                                         var elm = allDays.pop();
                                         elm.purge();
                                         elm.remove();

here an element from the events array is destroyed, but the reference 
is still contained in the events array. For me, the attached patch 
seems to fix this.

Saved Queries