JS error will cause CKEDITOR instances to not update their data to the iframe.

[Michael Hopkins]

We are using mutliple CKEDITOR instances on one page. We hide and display them by killing sections of the page and later repopulating them. Because we kill the CKEDITOR HTML before the final getData processing was finished it left the CKEDITOR in a state where getData and setData would not update the iframe contents.

I have attached the file with the fix that we added.

In /_source/plugins/editingblock/plugin.js line 47. You get the data from the editor. If an exception happens in the getData code then isHandlingData doesn't get set to false. After that the iframe contents become impossible to update. This will effect all other CKEDITOR instances on the screen.

We just added an empty try catch so as to ensure that regardless of errors that happen isHandlingData will always get reset to false. Thus not effecting the usability of any other CKEDITOR instance.

We did the same at line 74.

To reproduce: In our case, we would call editor.destroy() and remove the ckeditor instance from the screen. Destroy would fire an event that would later try to access the contentDocument of the iframe. That would throw the error because it no longer existed on the screen. We also had an blur event set to the editor that would grab the data.

Anyway I hope this helps.

[Michael Hopkins]

File that includes fix for this bug. editingblock plugin

[Jakub Ś]

Could I ask you first to provide a reproducible and reduced test case (html page) so that we could see the problem on our own?

Second: would it be possible to provide the fix you propose in a *.patch file?

[Michael Hopkins]

I added the test case.

Since the error i was having was caused by a race condition, reproducing it was successful only some of the time. Therefore I forced it by killing the innerHTML of the div and then calling getData() on the editor. So it reproduces everytime.

[Jakub Ś]

Message: $.contentWindow is null
URI: /3.6.2/ckeditor/_source/core/dom/element.js
Line: 1492

Reproducible from CKEditor 3.0