Explicitly set length after setting a new content property

This is a workaround for emberjs/ember.js#12218
emberjs · Aug 27, 2015 · 67f14b8 · 67f14b8
1 parent 9485564
commit 67f14b8
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/ember-data/lib/system/model/errors.js b/packages/ember-data/lib/system/model/errors.js
@@ -276,7 +276,13 @@ export default Ember.ArrayProxy.extend(Ember.Evented, {
     if (get(this, 'isEmpty')) { return; }
 
     let content = this.rejectBy('attribute', attribute);
+    let length = get(content, 'length');
     set(this, 'content', content);
+    // Explicitly set length after setting a new content property as
+    // a work around for https://github.com/emberjs/ember.js/pull/12218
+    if (this.get('length') !== length) {
+      this.set('length', length);
+    }
     get(this, 'errorsByAttributeName').delete(attribute);
 
     this.notifyPropertyChange(attribute);