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

RESOLVED FIXED

Status

RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: gekacheka, Assigned: shaver)

Tracking

(Blocks: 1 bug)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
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:
(Reporter)

Comment 1

13 years ago
Created attachment 198247 [details] [diff] [review]
patch: use .recurrenceStartDate, don't set dates to result of dates.push

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

Updated

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

gekacheka, please set review req
Attachment #198247 - Flags: first-review?
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!
Attachment #198247 - Flags: first-review?(jminta)

Comment 4

13 years ago
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)
Attachment #198247 - Flags: first-review?(jminta)
Attachment #198247 - Flags: first-review?
Attachment #198247 - Flags: first-review+

Comment 5

13 years ago
patch checked in.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.