Skip to content

Commit 8d2590d

Browse files
committed
BE #4 fix 404 on delete method
1 parent ef80d6e commit 8d2590d

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

routes/article.js

+11-3
Original file line numberDiff line numberDiff line change
@@ -150,9 +150,17 @@ router.route('/:articleId')
150150
if (mongoose.Types.ObjectId.isValid(req.params.articleId)) {
151151
Article.findByIdAndRemove(req.params.articleId)
152152
.then((resp) => {
153-
res.statusCode = 200;
154-
res.setHeader('Content-Type', 'application/json');
155-
res.json(resp);
153+
if(null===resp){
154+
var err = new Error('Article ' + req.params.articleId + ' not found');
155+
err.status = 404;
156+
return next(err);
157+
}
158+
else{
159+
res.statusCode = 200;
160+
res.setHeader('Content-Type', 'application/json');
161+
res.json(resp);
162+
}
163+
156164
}, (err) => next(err))
157165
.catch((err) => next(err));
158166
}

0 commit comments

Comments
 (0)