CKEditor does not work with elements with hyphen in their ID

[Wiktor Walc]

Open replacebycode.html sample, change the ID "editor2" into "edi-tor2", launch it in your browser and an error will occur (and as a result the event instanceReady is not triggered):

invalid assignment left-hand side
javascript:void(%20setTimeout(%20function()%7Bdocument.open()%3Bdocument.write(%20window.parent._cke_htmlToLoad_edi-tor2%20)%3Bdocument.close()%3Bwindow.parent._cke_htmlToLoad_edi-tor2%20%3D%20null%3B%7D%2C%200%20)%20)
Line 1

Note: window.parent._cke_htmlToLoad_edi-tor

CKEditor should support at least this:

ID and NAME tokens must begin with a letter ([A-Za-z]) and may be followed by any number of letters, digits ([0-9]), hyphens ("-"), underscores ("_"), colons (":"), and periods (".").

It seems that the source of this problem is the wysiwygarea plugin: 'window.parent._cke_htmlToLoad_' + editor.name + ' (line 244-246)

[Garry Yao]

Ticket test added with [4104] at : source:CKEditor/trunk/_test/tt/4240/4240.html

[Garry Yao]

Wiktor point me that it's critical for CKEditor to work with Drupal, so retarget to this release.

[Wiktor Walc]

I've searched in CKEditor source code for + editor.name and wysiwygarea plugin was the only place where it was incorrectly used, so R+.

[Garry Yao]

Fixed with [4111] on trunk and [4110] on 3.0.x branch.