gaspersic/freeCodeCamp
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fixed obvious error handlers

Browse Source
This commit is contained in:
Ashley Drake
2015-03-28 13:53:26 -04:00
parent e915e74b8d
commit ca3d46a776
1 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
controllers/courseware.js
View File

@ -14,7 +14,7 @@ exports.coursewareNames = function(req, res) {
}); });
}; };
exports.returnNextCourseware = function(req, res) { exports.returnNextCourseware = function(req, res, next) {
if (!req.user) { if (!req.user) {
return res.redirect('../coursewares/start-our-challenges'); return res.redirect('../coursewares/start-our-challenges');
} }
@ -27,6 +27,8 @@ exports.returnNextCourseware = function(req, res) {
return elem; return elem;
} }
}); });
// *****CALLBACK
req.user.save(); req.user.save();
var uncompletedCoursewares = req.user.uncompletedCoursewares; var uncompletedCoursewares = req.user.uncompletedCoursewares;
@ -34,7 +36,7 @@ exports.returnNextCourseware = function(req, res) {
var displayedCoursewares = Courseware.find({'_id': uncompletedCoursewares[0]}); var displayedCoursewares = Courseware.find({'_id': uncompletedCoursewares[0]});
displayedCoursewares.exec(function(err, courseware) { displayedCoursewares.exec(function(err, courseware) {
if (err) { if (err) {
next(err); return next(err);
} }
courseware = courseware.pop(); courseware = courseware.pop();
if (courseware === undefined) { if (courseware === undefined) {
@ -55,7 +57,7 @@ exports.returnIndividualCourseware = function(req, res, next) {
Courseware.find({"name" : new RegExp(coursewareName, 'i')}, function(err, courseware) { Courseware.find({"name" : new RegExp(coursewareName, 'i')}, function(err, courseware) {
if (err) { if (err) {
next(err); return next(err);
} }
// Handle not found // Handle not found
if (courseware.length < 1) { if (courseware.length < 1) {
@ -203,7 +205,7 @@ exports.generateChallenge = function(req, res) {
res.send(response); res.send(response);
}; };
exports.completedCourseware = function (req, res) { exports.completedCourseware = function (req, res, next) {
var isCompletedDate = Math.round(+new Date() / 1000); var isCompletedDate = Math.round(+new Date() / 1000);
var coursewareHash = req.body.coursewareInfo.coursewareHash; var coursewareHash = req.body.coursewareInfo.coursewareHash;
@ -221,7 +223,7 @@ exports.completedCourseware = function (req, res) {
req.user.save(function (err, user) { req.user.save(function (err, user) {
if (err) { if (err) {
throw err; return next(err);
} }
if (user) { if (user) {
res.send(true) res.send(true)