Node.js promisses цепочки

Question

Существует следующий код:

router.put('/:id', function(req, res) { 
    models.Task.find(req.params.id).then(function(task) { 
    task.updateAttributes({ 
     title: req.body.title, 
     description: req.body.description 
    }).then(function(task) { 
     res.json(task); 
    }) 
    }); 
}); 

Я пытаюсь заново фактор этого кода следующим образом:

router.put('/:id', function(req, res) { 
    models.Task.find(req.params.id).then(function(task) { 
    task.updateAttributes({ 
     title: req.body.title, 
     description: req.body.description 
    }) 
    }).then(function(task) { 
    res.json(task); 
    }); 
}); 

В первом случае я вижу правильный JSON выход, но во втором я могу видеть пустой вывод. Зачем?

Answer 1

Вы не ждете updateAttributes завершить больше. Вы можете сделать, хотя, просто добавив в return:

router.put('/:id', function(req, res) { 
    models.Task.find(req.params.id).then(function(task) { 
    return task.updateAttributes({ 
// ^^^^^^ 
     title: req.body.title, 
     description: req.body.description 
    }); 
    }).then(function(task) { 
    res.json(task); 
    }); 
}); 

Когда вы возвращаетесь обещание от более раннего обработчика в цепочке, последующие обработчики ждать его (и получить его значение разрешения).

---

Side Примечание: Не забывайте обрабатывать сбои в обещаниях, тоже с помощью второго аргумента then и/или с помощью catch.

Answer 2

Вы должны вернуть 2-ое обещание:

router.put('/:id', function(req, res) { 
    models.Task.find(req.params.id).then(function(task) { 
    return task.updateAttributes({ 
     title: req.body.title, 
     description: req.body.description 
    }); 
    }).then(function(task) { 
    res.json(task); 
    }); 
});