Mieszasz obietnice i oddzwonienia. exec()
zwróci Promise, ale tylko wtedy, gdy nie ma przekazanych żadnych argumentów. Dodatkowo musisz zwrócić obietnicę zwróconą przez exec()
.
return budget.save().then((res) => {
return Budget.findById(res._id) // missing return here
.populate('User')
.populate('Vehicle')
.exec() // don't need anything else
})
Możesz to trochę bardziej posprzątać:
return budget.save()
.then(res => Budget.findById(res._id)
.populate('User')
.populate('Vehicle')
.exec())
Jeśli potrzebujesz przekształcić wyniki zwrócone przez findById
przed przekazaniem ich klientowi:
return budget.save()
.then(res => Budget.findById(res._id)
.populate('User')
.populate('Vehicle')
.exec())
.then(res => {
res.foo = 'Foo'
return res
})