DELETE /todos/:id rout

Author: mateuszskibicki

playground/mongoose-remove.js

@@ -0,0 +1,20 @@
+const {ObjectID} = require('mongodb');
+
+const {mongoose} = require('./../server/db/mongoose');
+const {Todo} = require('./../server/models/todo');
+const {User} = require('./../server/models/user');
+
+// Todo.remove({}).then((result) => {
+//   console.log('Removed all');
+// });
+
+//Todo.findOneAndRemove()
+//Todo.findByIdAndRemove()
+// Todo.findOneAndRemove({_id: '5af49db1685fbf0494f3dd59'}).then((todo) => {
+//   console.log(todo);
+// });
+
+
+Todo.findByIdAndRemove('5af49db1685fbf0494f3dd59').then((todo) => {
+  console.log(todo);
+});

server/server.js

@@ -55,7 +55,7 @@ app.get('/users/:id', (req, res) => {
 app.get('/todos/:id', (req, res) => {
   var todoID = req.params.id;
   if (!ObjectID.isValid(todoID)) {
-    return res.status(400).send({error: 'Todo ID is not valid'});
+    return res.status(404).send({error: 'Todo ID is not valid'});
   } else {
     Todo.findById(todoID).then((todo) => {
       if (!todo) {
@@ -67,6 +67,34 @@ app.get('/todos/:id', (req, res) => {
   }
 });
 
+// delete
+
+app.delete('/todos/:id', (req, res) => {
+  // get the id
+  // not valid ? return 404
+  // remove todo by id
+    //success
+      //if no doc, send 404
+      //send doc back with 200
+    //error 
+      //400 with empty body
+
+  var todoID = req.params.id;
+  if(!ObjectID.isValid(todoID)) {
+    return res.status(404).send();
+  } 
+
+  Todo.findByIdAndRemove(todoID).then((todo) => {
+    if(!todo) {
+      return res.status(404).send();
+    } else {
+      return res.status(200).send(todo);
+    }
+  }).catch((e) => {
+    res.status(400).send();
+  })
+});
+
 
 
 app.listen(port, () => {