Remove row and undo/redo with object data source.

[Shynklaw]

Hello,

Use any examples with object data source at http://handsontable.com/demo/datasources.html, fiddle it and set "contextMenu:true", like in this fiddle http://jsfiddle.net/96XYQ/
Then, use remove row, and now try undo from context menu. You just can't undo it.
It's very specific with arrays of objects, because the first example without objects on the page work fine ( -> http://jsfiddle.net/ydaFm/ ).

Have a nice day.

[Shynklaw]

I think i've spotted the flaw which seems to cause this :
(into 'alter' method, line 560 of v0.9.12)

changes = [];
newData = datamap.getAll();
for (r = 0, rlen = newData.length; r < rlen; r++) {
    for (c = 0, clen = newData[r].length; c < clen; c++) {
        changes.push([r, c, oldData[r] ? oldData[r][c] : null, newData[r][c]]);
    }
}
instance.PluginHooks.run('afterChange', changes, source || action);

As you are only processing only arrays of arrays, each properties of objects aren't stored in the "changes" array, thus not added to the undo/redo array.

[psmolenski]

I've gone through the whole UndoRedo mechanism and found that it does not handles undoing and redoing operations related to altering table structure well. I'm currently rewriting UndoRedo and aim to merge my changes before version 0.9.14 is released.

[psmolenski]

Fixed in version 0.9.17. Please, upgrade and confirm.