Consolidate usage of convertDate into common location

RESOLVED FIXED in Sunbird 0.5

Status

--
minor
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: mattwillis, Assigned: jminta)

Tracking

Trunk
Sunbird 0.5
All
Mac OS X

Details

Attachments

(1 attachment)

(Reporter)

Description

12 years ago
convertDate is defined identically in calEvent.js calTodo.js and calRecurrenceInfo.js

We should combine all uses into one location (a la calUtils.js)

(spinoff from bug 349788)
(Assignee)

Comment 1

12 years ago
Created attachment 248042 [details] [diff] [review]
do it

This is a pure code-maintenance bug, so there's not a lot to say here.  convertDate() doesn't seem to exist in calRecurrenceInfo.js anymore.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #248042 - Flags: second-review?(dmose)
Attachment #248042 - Flags: first-review?(lilmatt)
(Reporter)

Comment 2

12 years ago
Comment on attachment 248042 [details] [diff] [review]
do it

r=lilmatt
Attachment #248042 - Flags: first-review?(lilmatt) → first-review+
Comment on attachment 248042 [details] [diff] [review]
do it

r2=dmose
Attachment #248042 - Flags: second-review?(dmose) → second-review+
(Reporter)

Updated

12 years ago
Whiteboard: [needs checkin]
Target Milestone: Future → Sunbird 0.5
(Reporter)

Comment 4

12 years ago
Patch checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
(Reporter)

Updated

12 years ago
Whiteboard: [needs checkin]
You need to log in before you can comment on or make changes to this bug.