Opened 13 years ago
Closed 12 years ago
#8940 closed Bug (duplicate)
filebrowser plugin breaks when the Array prototype is extended
| Reported by: | Pivotal Labs | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | File Browser | Version: | 3.1.1 |
| Keywords: | HasPatch | Cc: |
Description
We are using CKEditor with Ember.js.
Ember.js extends the native javacript Array prototype to include some helpful properties and functions. This causes some usages of for(i in j) loops within the filebrowser plugin to break, as it is iterating over all the properties on Array (instead of the elements in the array).
The following patch works for us.
Attachments (1)
Change History (3)
Changed 13 years ago by
| Attachment: | 0001-ckeditor-working-with-patched-array-prototype.patch added |
|---|
comment:1 Changed 13 years ago by
| Keywords: | HasPatch added |
|---|---|
| Status: | new → confirmed |
| Version: | 3.6.3 → 3.1.1 |
Good finding @pivotal
Ticket reproducible form CKEditor 3.1.1
Note: See
TracTickets for help on using
tickets.
patch to safely iterate over elements in Array