Deleting Array elements about diff HOT 4 CLOSED

Same issue here. I guess if you just loop through the diff and apply the changes one by one, the former index 3 will be index 2 after the deletion of thing3. The next applyChange then triggers a delete on thing4, which is recorded to be index 3, but now there is no index 3, so nothing happens.

from diff.

For a workaround see here:
#35

from diff.

I am also seeing this issue where the index seems to be changed and the diff index is no longer accurate... The workaround mentioned by @jonathan-reisdorf in #35 is to call reverse() on your array of changes...

deep.diff(lhs, rhs).reverse().forEach(function(diffItem) {
  deep.applyChange(lhs, true, diffItem);
});

The default behaviour of deep-diff should be to provide the diffs of arrays already reversed, going from highest index to lowest.

from diff.

+1 Same issue. Can the observableDiff function be changed to apply changes in reversed order in the library?

from diff.