Closed Bug 353707 Opened 13 years ago Closed 13 years ago

Consolidate usage of convertDate into common location

Categories

(Calendar :: Internal Components, defect, minor)

All
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Sunbird 0.5

People

(Reporter: mattwillis, Assigned: jminta)

Details

Attachments

(1 file)

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)
Attached patch do itSplinter Review
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)
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+
Whiteboard: [needs checkin]
Target Milestone: Future → Sunbird 0.5
Patch checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [needs checkin]
You need to log in before you can comment on or make changes to this bug.