334996 - Month view multiplies events

Status: RESOLVED FIXED

(Calendar :: Sunbird Only, defect)

Description

[Michiel van Leeuwen (email: mvl+moz@)]

When (forward) browsing the month views, a weekly recurring event gets displayed many times. Event-boxes for the new month get added, but the old boxes are not removed.
Steps to reproduce:
- Switch to month view
- Create a weekly recurring event, no end date
- Browse to the next month
- Observer that the event is displayed twice, once for the currently displayed month, and once for the previous month (as the tooltip tells us)

This makes using the month view with recurring events pretty hard, so marking as blocker.

(If we can't find a solution, we could turn of the re-use of the grid. That would be a performance hit, but solve this problem)

Comment 1

[Michiel van Leeuwen (email: mvl+moz@)]

Actually, all kinds of events stay on the view, not just recurring events. Sometimes the boxes are removed, but most are not. I can't tell what determines if a box is removed or not.

Comment 2

[Stefan Sitter [:ssitter]]

Attachment: multiweek screenshot

Event and day boxes in multiweek view are affected too. Also the box style gets screwed up. (See attached screenshot.)

It works in the 2006-04-19-07 build and fails in the 2006-04-20-07 build. Bug 334704 was the only check in to mozilla/calendar during that time.

Comment 3

[Stefan Sitter [:ssitter]]

Attachment: patch

This patch fixes Bug 334996 and Bug 334724. 
Therefore the patch on Bug 334724 is obsolete.

We should check if the change to calIDuration did not regressed other places.

Comment 4

[Stefan Sitter [:ssitter]]

Comment on attachment 219377 [details] [diff] [review]
patch

The patch fixes the problem but is wrong. With this patch we always hit the "shouldn't ever happen" case in relayout(). So we need another solution.

Comment 5

[Joey Minta]

Attachment: fix proper while

The while loop to remove the old events was never getting called.  Ignore the newDuration bit, since that's fixed in another bug.

Comment 6

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 219442 [details] [diff] [review]
fix proper while

looks right, r=mvl

But now we hit the next problem:
[JavaScript Error: "box.date has no properties" {file: "chrome://calendar/content/calendar-month-view.xml" line: 1097}]
(while browsing). I really start to wonder if we should remove the re-use of the grid.

Comment 7

[Michiel van Leeuwen (email: mvl+moz@)]

Patch checked in.
(the error i saw was bogus, a bug in my setup)