Skip to content

Commit

Permalink
Merge pull request #3156 from sly7-7/remove-deprecated-pushRecord
Browse files Browse the repository at this point in the history
Remove deprecated push record
  • Loading branch information
igorT committed Jun 2, 2015
2 parents 37401a8 + 89a5189 commit 4a52995
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 17 deletions.
2 changes: 1 addition & 1 deletion packages/ember-data/lib/system/record-array-manager.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ export default Ember.Object.extend({

if (shouldBeInArray) {
if (!recordArrays.has(array)) {
array._pushRecord(record);
array.addRecord(record);
recordArrays.add(array);
}
} else if (!shouldBeInArray) {
Expand Down
16 changes: 0 additions & 16 deletions packages/ember-data/lib/system/record-arrays/record-array.js
Original file line number Diff line number Diff line change
Expand Up @@ -135,22 +135,6 @@ export default Ember.ArrayProxy.extend(Ember.Evented, {
}
},

_pushRecord: function(record) {
get(this, 'content').pushObject(record);
},

/**
Adds a record to the `RecordArray`, but allows duplicates
@deprecated
@method pushRecord
@private
@param {DS.Model} record
*/
pushRecord: function(record) {
Ember.deprecate('Usage of `recordArray.pushRecord` is deprecated, use `recordArray.addObject` instead');
this._pushRecord(record);
},
/**
Removes a record to the `RecordArray`.
Expand Down

0 comments on commit 4a52995

Please sign in to comment.