Angular $ q.all viene chiamato dopo la prima promessa terminata

Question

Sto provando ad usare $ q.all per aspettare che tutte le promesse siano risolte ma si chiama dopo che la prima promessa è finita!

Cosa sto facendo male?

function sendAudits(audits) { 
    var promises = []; 

    $scope.sendAudits = { 
     progress: 0 
    }; 
    angular.forEach(audits, function (audit, idAudit) { 
     promises.push(saveAudit(audit)); 
    }); 

    $q 
     .all(promises) 
     .then(function (data) { 
      console.log(data); 
     }, function (errors) { 
      console.log(errors); 
     }); 
} 

function saveAudit(audit) { 
    var filename = audit.header.id + ".txt"; 

    return $http({ 
     method: 'PUT', 
     url: '/audits/audits.php?filename=' + encodeURIComponent(filename), 
     data: AuditSvc.getPlainAudit(audit.header.id) 
    }).finally(function() { 
     $scope.sendAudits.progress += 1; 
     console.log("FINALLY: " + audit.header.id); 
    }); 
} 

EDIT

Analizzando un po 'più profondo il problema, questa situazione si verifica quando alcune delle risposte sono errore. Per esempio, quando il server restituisce header("HTTP/1.0 418 I'm A Teapot: " . $filename);, console client sarebbe come:

PUT http://localhost:8182/audits/audits.php?filename=1.txt 418 (I'm A Teapot: 1.txt) 
FINALLY: 1 
Object {data: "", status: 418, config: Object, statusText: "I'm A Teapot: 1.txt"} 
PUT http://localhost:8182/audits/audits.php?filename=2.txt 418 (I'm A Teapot: 2.txt) 
FINALLY: 2 
PUT http://localhost:8182/audits/audits.php?filename=3.txt 418 (I'm A Teapot: 3.txt) 
FINALLY: 3 
PUT http://localhost:8182/audits/audits.php?filename=4.txt 418 (I'm A Teapot: 4.txt) 
FINALLY: 4 

Answer 1

La documentazione angolare non va nei dettagli, ma credo che $q.all() si comporta in questo caso allo stesso modo come l'es2015 Promise.all():

Se una delle promesse passate viene respinta, la Promessa del tutto respinge immediatamente con il valore della promessa che ha respinto, scartando tutte le altre promesse che siano state o meno risolte.

Molto probabilmente quello che sta accadendo qui è che almeno una delle vostre richieste sta fallendo. Le istruzioni del tuo log non distinguono se lo $q.all() è riuscito o meno, ma se fallisce, tutto quello che vedrai è il primo errore.

Vedere https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/all per la fonte della citazione.

Edit:

Se si desidera ottenere tutte le risposte, anche se alcuni non riescono, allora si dovrebbe aggiungere un gestore catch in saveAudit per convertire i fallimenti in risposte di successo:

function saveAudit(audit) { 
    var filename = audit.header.id + ".txt"; 

    return $http({ 
     method: 'PUT', 
     url: '/audits/audits.php?filename=' + encodeURIComponent(filename), 
     data: AuditSvc.getPlainAudit(audit.header.id) 
    }).catch(function(error) { 
     return { error:error}; 
    }) 
    .finally(function() { 
     $scope.sendAudits.progress += 1; 
     console.log("FINALLY: " + audit.header.id); 
    }); 
} 

e quindi è necessario controllare ogni risposta per vedere se contiene un errore o dati validi.

Answer 2

Come notato da altri, $q.all è non resiliente. Se una delle promesse viene rifiutata, lo $q.all viene rifiutato con il primo errore.

Per creare un resiliente promessa composito, che è una promessa che attende tutte le promesse per completare passaggio o fallire, utilizzare .catch su ogni singola promessa di convertire la promessa respinto ad una promessa di successo.

var resilientPromises = []; 

angular.forEach(promises, function(p) { 
    var resilientP = p.catch(function(result) { 
     //return to convert rejection to success 
     return result; 
    }); 
    resilientPromises.push(resilientP); 
}); 

$q.all(resilientPromises).then(function (results) { 
    //process results 
}); 

Le due cose da portare via da questa risposta:

1. Un $q.all promessa è non resilienti. È respinto con la prima promessa respinta.
2. Una promessa soddisfatta può essere creata da una promessa respinta da restituendo un valore alla funzione onRejected del metodo .then o del metodo .catch.