[[IE]] 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.

[Frederico Caldeira Knabben]

I was wondering if it's possible to fix the race condition in the auto paragraph code. I this way our API "just works". I'm reviewing- it for investigation.

[Garry Yao]

You're right, we do have better option here.

[Frederico Caldeira Knabben]

Let's just have a clearer implementation of restoreSelection when committing it:

	function restoreSelection( selection )
	{
		if ( selection.isLocked )
		{
			selection.unlock();
			setTimeout( function() { selection.lock(); }, 0 );
		}
	}

[Garry Yao]

Fixed with [5263] and [5264].

[Satya Minnekanti]

This is still re producible on the nightly build.you may not get it the first time you open the editor.Try opening and closing the Editor a couple of times.

To Reproduce the defect:

1. Open Ajax Sample.

2. Click on Create Editor button.

3. Click on Table Icon to get Table Properties dialog.

4. Click on OK button to Insert the Table.

We are getting an Error and i have attached the Screen shot for the Error.

We have to click OK button second time to insert the Table.

[Garry Yao]

It's not a regression, instead it's unbelievable that we still have logic hole in range::select method.

[Frederico Caldeira Knabben]

Please replace the change with the following when committing:

this.moveToPosition( dummySpan, CKEDITOR.POSITION_BEFORE_START );

[Garry Yao]

Fixed with [5327].