Ajax sample: undo behaves improperly

[Frederico Caldeira Knabben]

1. Load the Ajax sample.
2. Type some text.
3. Undo until the last possible undo step.
4. Redo until the last redo step.

In step 3, the last undo should be from a few text to blank contents.

In step 4, the last redo should restore the entire typed text.

Currently, the user looses the text when performing the above steps.

[Alfonso Martínez de Lizarrondo]

Proposed patch

[Alfonso Martínez de Lizarrondo]

The problem is similar to #4622. There's a first snapshot done automatically with the initial code that happens to be "<br>", which is turned into "<p><br></p>", but that patch doesn't fix this situation.

I've adjusted the save function so that in case that it's doing it for a redo call, it checks also the next snapshot to verify if that matches the current content, and so skip the save.

[Frederico Caldeira Knabben]

There are still issues:

• In the reported TC, there is an additional empty undo.

• It still breaks the contents in the following TC:

1. Load the ajax sample.

2. Type some text.

3. Remove the editor and recreate it.

4. Type more text after the current text.

5. Undo everything last step. The contents is empty, which is wrong... we should have the initial contents.

6. Redo everything. Nothings happens... the redo looks blocked.

[Frederico Caldeira Knabben]

Fixed with [5163].