Browse Source

Merge pull request #2174 from ZacBrownBand/dataview_include_olddata

Adds oldData to the update event payload in dataview to match dataset.
codeClimate
yotamberk 8 years ago
committed by GitHub
parent
commit
8b39874faf
1 changed files with 3 additions and 1 deletions
  1. +3
    -1
      lib/DataView.js

+ 3
- 1
lib/DataView.js View File

@ -300,6 +300,7 @@ DataView.prototype._onEvent = function (event, params, senderId) {
var ids = params && params.items;
var data = this._data;
var updatedData = [];
var oldData = [];
var added = [];
var updated = [];
var removed = [];
@ -330,6 +331,7 @@ DataView.prototype._onEvent = function (event, params, senderId) {
if (this._ids[id]) {
updated.push(id);
updatedData.push(params.data[i]);
oldData.push(params.oldData[i]);
}
else {
this._ids[id] = true;
@ -368,7 +370,7 @@ DataView.prototype._onEvent = function (event, params, senderId) {
this._trigger('add', {items: added}, senderId);
}
if (updated.length) {
this._trigger('update', {items: updated, data: updatedData}, senderId);
this._trigger('update', {items: updated, oldData: oldData, data: updatedData}, senderId);
}
if (removed.length) {
this._trigger('remove', {items: removed}, senderId);

Loading…
Cancel
Save