Closed
Bug 906107
Opened 11 years ago
Closed 7 years ago
remove cal. namespace from calUtils.js
Categories
(Calendar :: Internal Components, defect)
Calendar
Internal Components
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: Taraman, Assigned: Taraman)
References
Details
On our way to switch from calUtils.js to calUtils.jsm we should avoid using the cal. namespace in the .js to not get reference problems in old dialogs which still use it. (see bug 902916).
Comment 1•7 years ago
|
||
I guess this is WONTFIX since bug 1347149 does the exact opposite?
Comment 2•7 years ago
|
||
This is WONTFIX as soon as calUtils.js is gone. This bug is only about not using cal.XXX in the calUtils.js file, to avoid the same problem as in bug 902916. With the current path we could close it now though as long as we know what we are doing. I added it to all functions for consistency, but eventually we may need to remove them all so we can do ES6-style imports. It is just easier to remove them all than to add them :)
Comment 3•7 years ago
|
||
Since bug 1347149 we don't load calUtils.js except in calUtils.jsm, so I think we can close this one now.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•