Предупреждение: .then() только excepts функции

Question

в среднем приложение стек/nodejs/мангустов, у меня есть этот код:

User.findByIdAsync(req.params.id) 
 
    .then(handleEntityNotFound(res)) 
 
    .then(saveUpdates(req.body)) 
 
    .then(sendEmail()) // this is my addition 
 
    .then(respondWithoutResult(res)) 
 
    .catch(handleError(res));

функция Sendmail выглядит следующим образом:

function sendEmail(body){ 
 
    var mailOptions = { 
 
    from: 'Excited User <admin@blabla.com>', 
 
    to: 'johny@gmail.com', 
 
    subject: 'Hello', 
 
    text: 'body text here' 
 
    }; 
 

 
    var smtpConfig = { 
 
    host: config.mailgun.smtp_host, 
 
    port: 465, 
 
    secure: true, 
 
    auth: { 
 
     user: config.mailgun.smtp_user, 
 
     pass: config.mailgun.smtp_pass 
 
     } 
 
    }; 
 

 
    
 
    var transporter = nodemailer.createTransport(smtpConfig); 
 
    transporter.sendMail(mailOptions, function(error, info){ 
 
     if(error){ 
 
      return console.log(error); 
 
     } 
 
     console.log('Message sent: ' + info.response); 
 

 
    }); 
 
}

, когда я запускаю его, я получаю сообщение об ошибке: Предупреждение: .then() только за исключением функций, но передано: [объект Undefined]

Что я должен изменить в sendEmail, чтобы он работал с .then() ?

Answer 1

Поскольку sendMail supports promises, это может быть так просто, как это:

function sendEmail(body) { 
    var mailOptions = { 
    from: 'Excited User <admin@blabla.com>', 
    to: 'johny@gmail.com', 
    subject: 'Hello', 
    text: 'body text here' 
    }; 

    var smtpConfig = { 
    host: config.mailgun.smtp_host, 
    port: 465, 
    secure: true, 
    auth: { 
     user: config.mailgun.smtp_user, 
     pass: config.mailgun.smtp_pass 
    } 
    }; 

    var transporter = nodemailer.createTransport(smtpConfig); 

    // Return the promise here. 
    return transporter.sendMail(mailOptions); 
} 

Если вы хотите сохранить запись в журнал, замените последнюю строку с этим:

return transporter.sendMail(mailOptions).then(function(info) { 
    console.log('Message sent: ' + info.response); 
    return info; 
}, function(error) { 
    console.log(error); 
    throw error; 
}); 

EDIT: Я просто заметил аргумент body, который, как я предполагаю, может быть результатом saveUpdates(req.body)). Если это так, вам также необходимо немного перестроить свою цепочку обещаний:

User.findByIdAsync(req.params.id) 
    .then(handleEntityNotFound(res)) 
    .then(saveUpdates(req.body)) 
    .then(sendEmail) 
    .then(respondWithoutResult(res)) 
    .catch(handleError(res));