vndb/mithril-vndb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #102 from purcell/patch-1

Browse source 
Return results of sync promises in the same order they were given
This commit is contained in:
Leo Horie 2014-05-27 16:09:55 -04:00
commit 133bc88c01
1 changed files with 7 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
mithril.js
View file

@ -301,7 +301,7 @@ Mithril = m = new function app(window) {
currentRoute = arguments[0]
var querystring = typeof arguments[1] == "object" ? buildQueryString(arguments[1]) : null
if (querystring) currentRoute += (currentRoute.indexOf("?") === -1 ? "?" : "&") + querystring
var shouldReplaceHistoryEntry = (arguments.length == 3 ? arguments[2] : arguments[1]) === true
if (window.history.pushState) {
@ -427,11 +427,11 @@ Mithril = m = new function app(window) {
}
m.sync = function(args) {
var method = "resolve"
function synchronizer(resolved) {
function synchronizer(pos, resolved) {
return function(value) {
results.push(value)
results[pos] = value
if (!resolved) method = "reject"
if (results.length == args.length) {
if (--outstanding == 0) {
deferred.promise(results)
deferred[method](results)
}
@ -440,9 +440,10 @@ Mithril = m = new function app(window) {
}
var deferred = m.deferred()
var results = []
var outstanding = args.length
var results = new Array(outstanding)
for (var i = 0; i < args.length; i++) {
args[i].then(synchronizer(true), synchronizer(false))
args[i].then(synchronizer(i, true), synchronizer(false))
}
return deferred.promise
}