Skip to content

Commit

Permalink
Make a wild guess about the json api links issue:
Browse files Browse the repository at this point in the history
- notifyBelongsToChanged -> this._internalModel.notifyPropertyChange(key);
- notifyHasManyAdded -> this._internalModel.notifyPropertyChange(key);

i'll verify this with our app in the morning, which does use json api links

(and will rebase these comments away)
  • Loading branch information
eriktrom committed Jun 28, 2017
1 parent 308525f commit 37a3c60
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions addon/-private/system/model/model.js
Original file line number Diff line number Diff line change
Expand Up @@ -1040,7 +1040,7 @@ const Model = Ember.Object.extend(Ember.Evented, {
},

notifyBelongsToChanged(key) {
this.notifyPropertyChange(key);
this._internalModel.notifyPropertyChange(key);
},
/**
Given a callback, iterates over each of the relationships in the model,
Expand Down Expand Up @@ -1112,7 +1112,7 @@ const Model = Ember.Object.extend(Ember.Evented, {
//TODO(Igor): Consider whether we could do this only if the record state is unloaded

//Goes away once hasMany is double promisified
this.notifyPropertyChange(key);
this._internalModel.notifyPropertyChange(key);
},

eachAttribute(callback, binding) {
Expand Down

0 comments on commit 37a3c60

Please sign in to comment.