IndexMixin: set fetchStatus property to 'fetched' when fetch completes

The implementation of fetch does not call into BaseCollection's, so at least the status has to be updated the same way to actually prevent re-fetching in repeated calls of applyToAll etc.
Everyplay · Sep 7, 2015 · 3a715c3 · 3a715c3
1 parent 2c7574d
commit 3a715c3
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/collections/index_mixin.js b/lib/collections/index_mixin.js
@@ -39,7 +39,11 @@ var IndexMixin = _.extend({}, IndexedCollectionMixin, {
     if (!options.skipModelsFetch) {
       fns.push(_.bind(this.modelsFetch, this, options));
     }
-    return sequence(fns);
+    return sequence(fns)
+      .with(this)
+      .then(function() {
+        this.fetchStatus = 'fetched';
+      });
   },
 
   // deprecated