Merge pull request facebook#1575 from spicyj/dispatch-index

Make promise selection logic clearer
nickkeers · May 21, 2014 · 49652c8 · 49652c8
2 parents cccdc54 + e58899c
commit 49652c8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/examples/todomvc-flux/js/dispatcher/Dispatcher.js b/examples/todomvc-flux/js/dispatcher/Dispatcher.js
@@ -108,8 +108,8 @@ Dispatcher.prototype = merge(Dispatcher.prototype, {
    * A more robust Dispatcher would issue a warning in this scenario.
    */
   waitFor: function(/*array*/ promiseIndexes, /*function*/ callback) {
-    var selectedPromises = _promises.filter(function(/*object*/ _, /*number*/ j) {
-      return promiseIndexes.indexOf(j) !== -1;
+    var selectedPromises = promiseIndexes.map(function(idx) {
+      return _promises[idx];
     });
     Promise.all(selectedPromises).then(callback);
   }