[IE7 3.2.x branch] Have to click the Table dialog's OK button twice when there is no content.

[Joe Kavanagh]

1. Open the Ajax sample. Notice that the elements path is empty.
2. Click the Table toolbar button.
3. Click the Table dialog's OK button.

Observe the javascript error: 'type' is null or not an object.

This does not occur on the nightly 3.2 demo as the content area gets focus before the dialog appears. "body p" appears in the elements path.

The error is occurring in core/dom/walker.js guardRTL function.

I believe the error is occurring because the node to replace is null. The wysiwyg plugin's onInsertElement function thinks it is in an empty p element and attempts to replace it with the table element. See the if statement following the below comment in the onInsertFunction function:

If we're in an empty block which indicate a new paragraph, simply replace it with the inserting block.(#3664)

[Frederico Caldeira Knabben]

Confirmed with IE8 on trunk also.

[Garry Yao]

Defer the table element insertion to avoid a race-condition with the auto paragraph feature which happens to mangle the range.