Q обещание цепочки, обработчик ошибок не называется

Question

Рассмотрим этот код

var tryWithoutReindexing = function(indexName, properties) { 
     var settings = properties["settings"]; 
     var mappings = properties["mappings"]; 
     return elastic.closeIndex(indexName) 
      .then(elastic.putSettings(indexName, settings)) 
      .then(elastic.putMapping(indexName, mappings)) 
      .then(elastic.openIndex(indexName)); 
}; 

И позвони:

tryWithoutReindexing(indexName, newProperties) 
.then(function success(value){ 
     console.log('migration successful'); 
    }, function error(){ 
     console.log('migration unsuccessful'); 
    }); 

Метод elastic.putSettings бросает ошибку, но по какой-то причине, console журналы 'migration is successful'. Я бы ожидал вызова обработчика ошибок.

Если изменить способ это:

var tryWithoutReindexing = function(indexName, properties) { 
     var settings = properties["settings"]; 
     var mappings = properties["mappings"]; 
     return elastic.closeIndex(indexName) 
      .then(elastic.putSettings(indexName, settings)) 
       .then(function success() { 
       console.log('err'); 
      }, function(error) { 
       console.log(error); 
      }) 
      .then(elastic.putMapping(indexName, mappings)) 
      .then(elastic.openIndex(indexName)); 
}; 

, и поставить точку останова в строке console.log(error);, обработчик ошибок называется, так что кажется, что метод putSettings работает правильно.

Может ли кто-нибудь объяснить мне, почему первый пример не обрабатывает ошибки, возникшие в цепочке обещаний?

Answer 1

Я предполагаю, что elastic.putSettings() и другие возвращают обещание. Вы не можете использовать обещание в качестве аргумента для .then; этот метод ожидает функция аргументы. В свою очередь, эти функции могут вернуть обещание.

Итак, вам нужно обернуть свои функции, возвращающие обещания, анонимной функцией и использовать эту функцию в качестве аргумента для .then. Вроде так:

var tryWithoutReindexing = function(indexName, properties) { 
    var settings = properties["settings"]; 
    var mappings = properties["mappings"]; 

    return elastic.closeIndex(indexName) 
       .then(function() { 
        return elastic.putSettings(indexName, settings); 
       }) 
       .then(function() { 
        return elastic.putMapping(indexName, mappings); 
       }) 
       .then(function() { 
        return elastic.openIndex(indexName); 
       }); 
};