Opened 12 years ago
Closed 12 years ago
#9649 closed Bug (fixed)
Global variable leak ("i")?
| Reported by: | Wiktor Walc | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.0 |
| Component: | General | Version: | 4.0 |
| Keywords: | Cc: |
Description
1) I just noticed in Firebug some weird "i" variable. This variable is pretty popular, so I can image quite an interesting results of overwriting it accidentally
Can be reproduced on the nightly demo:
http://nightly-v4.ckeditor.com/demo/replacebyclass.html
2) after running the inlineall sample I see also data-cke-expando:
Attachments (1)
Change History (4)
comment:1 Changed 12 years ago by
comment:2 Changed 12 years ago by
Make sure to click into editable elemetns to launch CKEditor, in order to see this variable.
Changed 12 years ago by
| Attachment: | data-cke-expando.png added |
|---|
comment:3 Changed 12 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
The leaked variable is fixed by #9652, while the expando property seen on window is a unique DOM object identifier which cannot be removed (part of domobject impl.)
Note: See
TracTickets for help on using
tickets.
window.i comes from htmldataprocessor.js, I cannot reproduce data-cke-expando issue though (dev/release).
I created a ticket #9652 to fix pending JSLint issues, to which window.i belongs.