clean up pyramid of doom with populate using mongoose

Author: Jrschellenberg

src/routes/course.js

@@ -23,32 +23,16 @@ router.get('/', (req, res, next) => {
 });
 
 router.get('/:courseId', (req, res, next) => {
-	Course.findById(req.params.courseId, (err, course) => {
-		Utils.isError(err, next);
-		course.getUser((err1, user) => {
-			Utils.isError(err1, next);
-			course.user = user;
-			let itemsProcessed = 0;
-			course.reviews.forEach((courseReview, index, array) => {
-				course.getReview(index, (err2, review) => {
-					Utils.isError(err2, next);
-					review.getUser((err3, reviewUser) => {
-						Utils.isError(err3, next);
-						review.user = reviewUser;
-						course.reviews[index] = review;
-						itemsProcessed++;
-						if(itemsProcessed === array.length){
-							res.locals.course = course;
-							next();
-						}
-					});
-				});
-			});
+	Course
+		.findById(req.params.courseId)
+		.populate('user')
+		.populate('reviews')
+		.exec((err, course) => {
+			Utils.isError(err, next);
+			res.locals.course = course;
+			let status = 200;
+			return res.status(status).json({success: true, message: "Course Successfully retrieved!", status: status, course: res.locals.course });
 		});
-	});
-}, (req, res) => {
-	let status = 200;
-	return res.status(status).json({success: true, message: "Course Successfully retrieved!", status: status, course: res.locals.course });
 });
 
 router.post('/', authorizeUser, (req, res, next) => {