310842 - calRecurrenceInfo.getNextOccurrenceDate: this.mBaseItem.getRecurrenceStartDate is not a function

Status: RESOLVED FIXED

(Calendar :: Internal Components, defect)

Description

[gekacheka]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b4) Gecko/20050908 Firefox/1.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051001 Mozilla Sunbird/0.2+

a couple bugs in calRecurrenceInfo.getNextOccurrenceDate:

this.mBaseItem.getRecurrenceStartDate is not a function

Array.push does not return the array.



Reproducible: Always

Steps to Reproduce:

Comment 1

[gekacheka]

Attachment: patch: use .recurrenceStartDate, don't set dates to result of dates.push

(patch -l -p 2 -i file.patch)

Comment 2

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

Comment on attachment 198247 [details] [diff] [review]
patch: use .recurrenceStartDate, don't set dates to result of dates.push

gekacheka, please set review req

Comment 3

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

Comment on attachment 198247 [details] [diff] [review]
patch: use .recurrenceStartDate, don't set dates to result of dates.push

(now with fill text) gekacheka, please set review requests, and set them to
sopmebody specific. Otherwise patches will get lost!

Comment 4

[Joey Minta]

Comment on attachment 198247 [details] [diff] [review]
patch: use .recurrenceStartDate, don't set dates to result of dates.push

r=jminta.  Can I ask for a little more context in future patches, please? (diff
-pU 8 works nicely)

Comment 5

[Joey Minta]

patch checked in.