Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Commit

Permalink
Merge pull request #2264 from almende/mojoaxel-fix2189
Browse files Browse the repository at this point in the history
fixes DataView.refresh bug introduced with #2189
  • Loading branch information
yotamberk authored Nov 5, 2016
2 parents feab6c5 + b522f4e commit cac9a09
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/DataView.js
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ DataView.prototype.refresh = function () {

// trigger events
if (addedIds.length) {
this._trigger('add', {items: added});
this._trigger('add', {items: addedIds});
}
if (removedIds.length) {
this._trigger('remove', {items: removedIds, oldData: removedItems});
Expand Down Expand Up @@ -397,4 +397,4 @@ DataView.prototype._trigger = DataSet.prototype._trigger;
DataView.prototype.subscribe = DataView.prototype.on;
DataView.prototype.unsubscribe = DataView.prototype.off;

module.exports = DataView;
module.exports = DataView;

0 comments on commit cac9a09

Please sign in to comment.