SpellerPages can lead to Permission Denied errors with IE

[Mark Bryson]

One set of steps to reproduce with IE7:

1. Have an anchor tag in content.
2. Use SpellerPages to correct at least one word.
3. Press 'Link' button.

You then get Permission Denied error on line 357. This isn't a problem with FF. I've reproduced with IE7 and several FCKeditor versions from SVN back to version 2.4.3.

One way to work around it is to replace the SetHTML() [or SetData()] function in 'fck_spellerpages.html' with:

oEditor.FCK.Commands.GetCommand('SelectAll').Execute();
oEditor.FCK.InsertHtml(document.getElementById('txtHtml').value);

(but with extra prefixed blank paragraph noted by ticket #2395)

I have NOT been able to reproduce by using SetData() in a much simpler, but very similar scenario (plugin) that replaces content containing an anchor tag. So, there seems to be more to causing this than just simply using setData() with content containing an anchor tag.

[Frederico Caldeira Knabben]

I was able to reproduce the reported problem. I was also having problems by running SpellerPages in the default editor contents of sample01.

The first problem here was that old things were remaining in the TempBin when calling SetData. It was causing the "Permission Denied" error I had because it was trying to access elements from a document that doesn't exist anymore. I've fixed it with the FCKTempBin.Reset() call.

The second problem is the reported issue. The fact is that SpellerPages is using innerHTML to retrieve the editor data, which contains all our "fake stuff". The problem is that we feed it back to the editor with SetData, which creates a new document, the the "fake stuff" references get lost.

To fix it, I've simply used innerHTML to move the data back to the editor, which seems logic and should not cause any undesirable effect.

[Frederico Caldeira Knabben]

Fixed with [2286].

[Mark Bryson]

Wow... That was quick. Do you also think inserting the following line before line 56 is a good idea?

    oEditor.FCKUndo.SaveUndoStep() ;

[Frederico Caldeira Knabben]

Yes, that would be a good idea. Please open a dedicated ticket for it, as this ticket has been already committed and released.