Closed
Bug 353707
Opened 18 years ago
Closed 18 years ago
Consolidate usage of convertDate into common location
Categories
(Calendar :: Internal Components, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Sunbird 0.5
People
(Reporter: mattwillis, Assigned: jminta)
Details
Attachments
(1 file)
|
5.03 KB,
patch
|
mattwillis
:
first-review+
dmosedale
:
second-review+
|
Details | Diff | Splinter Review |
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•18 years ago
|
||
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•18 years ago
|
||
Comment on attachment 248042 [details] [diff] [review]
do it
r=lilmatt
Attachment #248042 -
Flags: first-review?(lilmatt) → first-review+
|
||
Comment 3•18 years ago
|
||
Comment on attachment 248042 [details] [diff] [review]
do it
r2=dmose
Attachment #248042 -
Flags: second-review?(dmose) → second-review+
|
|
Reporter | |
Updated•18 years ago
|
Whiteboard: [needs checkin]
Target Milestone: Future → Sunbird 0.5
|
|
Reporter | |
Comment 4•18 years ago
|
||
Patch checked in on MOZILLA_1_8_BRANCH and trunk.
-> FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
|
Reporter | |
Updated•18 years ago
|
Whiteboard: [needs checkin]
You need to log in
before you can comment on or make changes to this bug.
Description
•