treeStoreModel's pasteItem return different results when using insertIndex

[Jean-Rubin Leonard]

Hi, Consider the following tree:

NodeA
|--NodeA1
NodeB
|--NodeB1

If I drag and drop NodeB1 squarely on NodeA, the drop part of the DnD happens on the few lines starting at line 256 of treeStoreModel.js.

In the case where there's no insertIndex (drop squarely on a specific node) the store's data gets changed and the method carries on. After that the store's onSet is invoked and receives the item, the attribute, the old Value and the new value as attributes. One can then proceed to see what has changed by comparing old value to new and then doing the proper treatment.

When there is an insertIndex however (instead of DnDing? directly on a node, we used betweenThreshold to add a node as a sibling of one of the children of a node) the treatment seems to be incorrect. The reason for this is that when there is an insertIndex the splice statement changes the array (which changes the store's data, (childItems.splice(insertIndex, 0, childItem);, line 259) THEN the store's data is saved again (store.setValues(newParentItem, parentAttr, childItems);, line 260) What this basically causes is for oldValue and newValue to be the same when the store's onSet ensues.

Hope this is clear. I'm usually around on IRC with a live test case if needed but I can't provide one with this ticket at this moment.

[bill]

Thanks for catching that, I'll check in the fix

[bill]

(In [22067]) Need to copy array before splicing it, otherwise the data store thinks the old and new value is the same (because the "old value" has also been modified). Fixes #11085.

[bill]

(In [22069]) Need to copy array before splicing it, otherwise the data store thinks the old and new value is the same (because the "old value" has also been modified). Fixes #11085.

[millennium]

in trac #10474 the same issue is fixed at store (ItemFileReadStore?) level. I know not all stores are inherited from the ItemFileReadStore?. But in my opinion fixing it at store level is logical. Not at model level. just myHO. Keep up the good work guys