JavaScript error in TinyMCE

[pavelevap]

Using latest trunk, no plugin, IE8.

Message: 'content' is null or not object.
Line: 1
Char: 8525
Code: 0
URI: http://localhost/Wordpress/wp-includes/js/tinymce/plugins/wordpress/editor_plugin.js?ver=3393

This error appears using visual editor, only after saving post. When adding new post, there is no error.

[SergeyBiryukov]

Can't reproduce with IE 8.0.6001.18702. Perhaps something else is going on there.

[pavelevap]

I tried to debug this error in IE8 (Developer tools, F12) and script stops with error on following code:

g.content=g.content.replace(/<!--more(.*?)-->/g,e);

My IE8 is 8.0.6001.18999.

I am not sure where else should be problem?

[filosofo]

I had to turn off debug mode to reproduce.

I don't have TinyMCE source, and patch diff highlighting isn't very helpful, so here's the sanity check I added to wp-includes/js/tinymce/plugins/media/editor_plugin.js

I changed

d=e.content

to

d=(e.content?e.content:'')

Note that wp-includes/js/tinymce/wp-tinymce.js.gz will also have to be rebuilt, as it contains the script above.

[westi]

You can get the TinyMCE source in the development download of 3.3.9.3 from upstream here ​http://tinymce.moxiecode.com/download/download.php

[nacin]

I imagine it's ​https://github.com/tinymce/tinymce/blob/master/jscripts/tiny_mce/plugins/media/editor_plugin_src.js#L207.

[filosofo]

Yes, that looks like it.

[nacin]

Some list table javascript is messing with TinyMCE. koopersmith is working on a fix.

[koopersmith]

Currently, the list table JavaScript resets all forms whenever a page is loaded. This is heavyhanded code.

In this case, it caused TinyMCE to break in IE8 while it was in the process of loading.

However, a reset of some sort is necessary. Without a reset, if you change various inputs on the page (i.e. search and filters) and refresh, the results will be reverted to the default, but the inputs will not. The solution I used was to selectively reset form values—only those found within the tablenav, search-box, and wp-list-table classes.

[ocean90]

Patch works for me, thx koopersmith.

[ryan]

(In [17239]) Selectively reset list-table forms rather than all forms to avoid stompage. Props koopersmith. fixes #16098