Closed Bug 906107 Opened 11 years ago Closed 7 years ago

remove cal. namespace from calUtils.js

Categories

(Calendar :: Internal Components, defect)

defect
Not set
normal

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).
I guess this is WONTFIX since bug 1347149 does the exact opposite?
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 :)
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.