Closed Bug 501402 Opened 11 years ago Closed 11 years ago
Day View not updated correctly after deleting multiple events [Error "a
.occurrence is undefined" in calendar-multiday-view .xml]
Looks like a regression between the 1.8.1 and the 1.9.0 branch: Works using Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:188.8.131.52pre) Gecko/20080917 Sunbird/0.9 Fails using Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:184.108.40.206pre) Gecko/20080917 Calendar/0.6a1
Flags: blocking-calendar1.0? → blocking-calendar1.0+
OS: Windows XP → All
Hardware: x86 → All
Whiteboard: [needed beta][no l10n impact]
It appears that the references to the calendar-event-box's stored in the mSelectedChunks array are no longer valid after the clear method is called from the relayout method. Subsequent calls to internalDeleteEvent trigger the error when attempting to compare the a.occurrence.hashId value when a.occurrence is undefined during the mSelectedChunks.filter operation. Proposed fix: 1) add a check for undefined in the a.occurrence property when calling mSelectedChunks.filter in the internalDeleteEvent method 2) clear out the mSelectedChunks array in the clear method as any entries are no longer valid 3) rebuild the mSelectedChunks array in the relayout method as the new calendar-event-box instances are created
Attachment #389403 - Flags: review?(philipp)
Assignee: nobody → matthew.mecca
Status: NEW → ASSIGNED
Attachment #389403 - Flags: review?(philipp) → review+
Comment on attachment 389403 [details] [diff] [review] Patch v1 Fix looks fine and works just as well :-) Thanks for taking care of a blocker bug, I really appreciate it!
Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/cbf01c186ed7> -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.0
Can this be checked with the automatic Mozmill tests from Bug 500469, preferable for all views?
(In reply to comment #5) > Can this be checked with the automatic Mozmill tests from Bug 500469, > preferable for all views? I don't see why not. Merike, thoughts?
It would fit nicely in phase 2, yes.
verified with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:220.127.116.11pre) Gecko/20100313 Calendar/1.0b2pre
Status: RESOLVED → VERIFIED
These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".
Target Milestone: 1.0 → 1.0b1
You need to log in before you can comment on or make changes to this bug.