Paste as plain text broken when Mootools is loaded (Array's prototype contains $constructor)

[edwtests]

1) Extract CKEditor full

2) Use the following settings:

CKEDITOR.editorConfig = function(config) {
    config.allowedContent = true;
};

3) Open a page with CKEditor in it.

4) Use the paste as plain text button, and paste rich text from a webpage. For example, I copied the side bar content at ​http://msdn.microsoft.com/en-us/library/aa288454%28v=vs.71%29.aspx

5) The text is not inserted.

See screenshot. PS: Tried with version 4.3.1 too, no change.

Short TC

1. Download 11307.html​ to samples/.
2. Follow instructions.

[edwtests]

Screenshot

[edwtests]

(Sorry, I messed up the WikiFormat on the original ticket, it was my first one)

1) Extract CKEditor full

2) Use the following settings:

CKEDITOR.editorConfig = function(config) {

config.allowedContent = true;

};

3) Open a page with CKEditor in it.

4) Use the paste as plain text button, and paste rich text from a webpage. For example, I copied the side bar content at ​​http://msdn.microsoft.com/en-us/library/aa288454%28v=vs.71%29.aspx

5) The text is not inserted.

See screenshot. PS: Tried with version 4.3.1 too, no change.

[Piotrek Koszuliński]

I couldn't reproduce this issue (see papt.webm​). On which browser do you test it? And could you check what error is thrown? Are you testing the same TC I checked?

[edwtests]

error on chrome

[edwtests]

error on firefox 26

[edwtests]

error on ie11

[edwtests]

Just attached screenshots of the consoles. On IE11 the popup to enter the text doesn't even appear, the text is pasted directly, but the error is still there.

All tests on Win7 Pro x64 Firefox 26 Chrome 32 IE11

[edwtests]

Hello again. I have traced this down to an incompatibility with "mootools-core-1.3.1-full-compat-yc.js", which was being loaded by a base page.

You may want to close this ticket now, thanks.

Just FYI, I downloaded the uncompressed CKEditor pack and found the line with the conflict: ckeditor\core\htmlparser\filter.js:230.

if ( rulesGroup )
	return rulesGroup.exec( context, value, element, this );

I changed it to

if ( rulesGroup && typeof(rulesGroup.exec) == "function" )
	return rulesGroup.exec( context, value, element, this );

and now the feature works correctly. I have no idea which other features are affected by mootools; will try to post back if I find any.

Regards

[Piotrek Koszuliński]

This incompatibility turned out to be caused by issue in CKEditor. It took a while to figure out why element.attributes becomes a reference to Object constructor, but I managed to locate this.

​https://github.com/ckeditor/ckeditor-dev/blob/master/plugins/clipboard/plugin.js#L1096

Attributes are set to an array, but they should be an object. Most likely then some code starts iterating over that array and weird things happen.

[Piotrek Koszuliński]

Pushed ​t/11307 on dev and tests.

[Olek Nowodziński]

Rebased branches on latest master.

[Olek Nowodziński]

A single commit pushed to tests.

[Piotrek Koszuliński]

Fixed on master with ​git:feba862 on dev and 60f0e27 on tests.