FilteringTable: onAddData fires also if the data in the store are replaced

[guest]

If the data in the store are replaced with a new objekt (but the same key) the event onAddData will fire - that's ok. The FilteringTable?-Widget should be aware of this. Attached is a patch that solved the problem in this special case.

[ttrenka]

(In [7319]) Fixes #2081 by creating new events on the Store and adding new handlers on the FilteringTable? widget.

[ttrenka]

(In [7337]) Ok. FINALLY fixes #2081. Object comparison for getting the correct row to be swapped was wrong.

[aferber@…]

It is still broken, at least in FireFox? 2.x. Note that dojo.collections.Store.onUpdateData takes the Store entry as a parameter, which has the stored data available as .src property, however getRow() in FilteringTable? compares the real data object.

For performance (and easier to understand code IMHO) it might be the best to compare the row key value against the key directly, instead of going the indirect way by comparing data objects.

I'm going to attach a patch (against r7337) that fixes the problem for me and changes to comparing keys directly. There are some other places that also might be switched to the new getRowByKey method, however I'm leaving them alone for now and only fix the bug at hand.

[aferber@…]

Fix for r7337

[ttrenka]

Your reopening is a dupe of #2469. Closing in favor of that.