Opened 15 years ago
Closed 15 years ago
#5356 closed Bug (fixed)
Scayt options dialog doesn't work with prototype.js enabled environments
| Reported by: | Antti Leppä | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | UI : Spell Checker | Version: | 3.2 |
| Keywords: | Confirmed HasPatch | Cc: | WebSpellChecker.net |
Description
When using CKEditor in prototype.js enabled environments SCAYT plugin options dialog opening crashes.
This is caused by plugins/scayt/dialogs/options.js:218
for ( i in buttons )
Normally this clause iterates over all items within a array but in prototype enabled environments this causes loop to iterate also over all extended methods thus causing following doc.getById( button) to fail
This problem can be fixed by changing line 218 into:
for ( i = 0; i < buttons.length; i++ )
The problem was discovered with Prototype 1.6.1
Change History (4)
comment:1 Changed 15 years ago by
| Keywords: | Confirmed added |
|---|---|
| Milestone: | → CKEditor 3.3 |
comment:2 Changed 15 years ago by
| Keywords: | HasPatch added |
|---|
comment:3 Changed 15 years ago by
| Cc: | WebSpellChecker.net added |
|---|
comment:4 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Fixed with [5322].