Paste from Word: cleanWord not invoked for the first time

[Wiktor Walc]

When using the "Paste from Word" button to paste content, the CKEDITOR.cleanWord function is not executed for the first time. Confirmed in Safari, Opera and Chrome.

Steps to reproduce

1. Replace the default filter with:

   CKEDITOR.cleanWord = function( data, editor )
   {
   	alert("clean word called");
   	return 'xyz';
   }
   

2. Open _samples/replacebyclass.html in Safari.
3. Copy something, but not from Word, e.g. a part of the default text in the editor.
4. Paste it using the "Paste from Word" button.
5. Result: the alert message "clean word called" did not show up.
6. Repeats steps 3-5. The alert message now showed up.

[Wiktor Walc]

I think the following ticket might be related: #5808.

[Garry Yao]

Dialog (paste word) open doesn't always force the cleanup, not a regression though.

[Frederico Caldeira Knabben]

I'm not able to reproduce the problem with IE, Safari and Opera. It works exactly as expected.

Something is really wrong here with FF.

Having the custom cleanWord() defined in the page, with FF:

1. Click the "Paste from Word" button. The alert() message is displayed and "xyz" is inserted into the editor contents (Issue 1).
2. Paste something in the dialog and confirm. The alert() message is displayed again and "xyz" is inserted in the editor. This is expected.

Then after patch, always with FF, things get worst. While the above issue 1 still happens, the expected point 2 is not happening. No alert() and the original paste contents are inserted in the editor (Issue 2).

[Wiktor Walc]

As it looks like things are a bit messed up here, it might be worth to check also #6465 while working on this ticket, perhaps it is related.

[Garry Yao]

Sorry not mentioned that this ticket depends on #7316 and thanks Wiktor it's not related to #6465.

[Sa'ar Zac Elias]

I'm not able to reproduce the issue, any clues?

[Garry Yao]

Firefox + Paste Word button.

[Wiktor Walc]

With this patch I'm still able to reproduce this issue in IE6 with "Security Settings" set to "Allow paste operations via script".

[Garry Yao]

Wiktor just caught a small logic glitch in the patch with IE6.

[Wiktor Walc]

IE8, IE8 Standards mode. "Allow Programmatic clipboard access" set to Disabled. After pressing "Paste From Word" button, the cleanWord function is immediately executed, just like in comment:5

[Wiktor Walc]

The last patch looks good to me, I am unable to find any issues.

[Garry Yao]

Fixed with [6606].