Cannot call method 'split' of undefined on CKEDITOR.replace

[Michael Camden]

Our page has several divs that are placeholders for CKEDITOR instances. When a user clicks on one, it closes the active editor, and creates a new instance of CKEDITOR on the clicked div using CKEDITOR.replace. When you have an editor instance open, right click to open the context menu twice, and click on another div to open the editor, I get this error.

Cannot call method 'split' of undefined

This error only occurs when I am using ckeditor.js and not ckeditor_source.js. I've tracked the error down to line 23 of ckeditor.js. Here is a snippet of the issue:

q=function(x){var y=x.config.skin.split(','),z=y[0],A=a.getUrl(y[1]||'skins/'+z+'/');

Interestingly, I cannot find any reference in the source to 'config.skin.split'. It seems that this only exists in the ckeditor.js file.

Steps to reproduce:

1. Start out on a page with two div elements with ids, 'editor1', 'editor2'.
2. Use CKEDITOR.replace to replace 'editor1'
3. Right click on the active editor to activate the context menu
4. Left click on the active editor, not clicking on the menu items
5. Repeat Step 3
6. Repeat Step 4
7. Use CKEDITOR.replace to replace 'editor2'

Tested in Chrome, IE8/9

[Jakub Ś]

You it would be possible to provide us with a sample file?

[Michael Camden]

Please see attached file. It also includes the steps to reproduce this bug.

[Frederico Caldeira Knabben]

I've confirmed the error in the provided TC with both FF and IE.

Just for reference, we're talking about line 115 of editor.js​. For some unknown reason, editor.config gets mangled and becomes an invalid object.

To note that this happens only when enabling SCAYT, This tells me that this issue is much probably related to the load of SCAYT core from SpellChecker.net.

[WebSpellChecker.net]

This is duplicate of #7697

[Frederico Caldeira Knabben]

Replying to SpellChecker.net:

This is duplicate of #7697

This ticket cannot be a DUP of #7697 because that one is already closed and I can still confirm the issue on the current trunk.

[WebSpellChecker.net]

The original patch - ​https://dev.ckeditor.com/attachment/ticket/7697/7697.patch have contained declaration of loop variable as 'var' in both loops. This prevents of mixing of variables names when minifying a version. But in changeset [6876] you've removed a "var" in the second loop by some reason. What we have observed is that problem is reproduced on 3.5.3 release available for download. If the same version will be built from sources in different environment the problem might not be reproduced. But with the "var" declaration of loop variable in the second loop the problem can not be reproduced in our environment.

[WebSpellChecker.net]

We can reproduce this problem on trunk.

[Frederico Caldeira Knabben]

If we add "var m" at line 671, it'll be a redeclaration of that variable, which is wrong (just like the previous patch).

Additionally, there is no justification for that "var" being the responsible for the fix. This is the minified version of that code (I've added one ENTER):

for(var M in w){delete t._.menuItems[M];delete t._.commands[M];}
for(M in x){delete t._.menuItems[M];delete t._.commands[M];}

Something else must be wrong here.

[WebSpellChecker.net]

While we were investigating the problem we have found issue with 'htmlFilter' and 'dataFilter' used in SCAYT plugin. If we comment out line 241:

delete element.name;

and lines 527, 528:

if ( attrs && attrs[ 'data-scaytid' ] )
   delete element.name;

then the problem cannot be reproduced. We tried to review 'dataProcessor' code and found that there are similar filtering rules for

<span class="Apple-style-span">sample</span>

We tried to put such markup in to the editor and were able to reproduce the problem on nightly build (​http://nightly.ckeditor.com/6956/_samples/). No need to enable SCAYT to reproduce the problem.

Steps to reproduce:

1. Open Replace DIV sample ​http://nightly.ckeditor.com/6956/_samples/divreplace.html
2. Double-click to enable the editor
3. Switch to source mode
4. Paste <span class="Apple-style-span">sample</span>
5. Switch to WYSIWYG
6. Double-click other DIV

Expected results: editor switched to other DIV

Real results: described error observed

[WebSpellChecker.net]

Looks like this issue is fixed by #8000 .
We are not able to reproduce the issue on latest nightly build: ​http://nightly.ckeditor.com/7045/_samples/divreplace.html

[aaronransley]

I can confirm that this issue continues to exist, even after the change in #8000. It cannot, however, be reproduced by including the ckeditor_source.js file. Only the minified/packed ckeditor.js file causes the error. Could this be an issue with the minification routines?