Response JSON object is missing "name" and "completed"

Just found out a solution to my problem - I changed "Text" to "JSON(application/json)" for POST in Postman and now everything works.

I was having the same issue and thought it was my code after going through all of the videos again trying to troubleshoot. In the POST method in Postman, you need to specify the data as "raw" and change the type from "Text" (the default) to "JSON(application/json)" also check that the output is JSON. That fixed the issue for me running body-parser version 1.15.0.

Hey guys, I had the same issue and solved it as following:

• In Postman click on "Headers" and insert "Header" = Content-Type and "Value" = application/json Also make sure that you are sending and receiving JSON.

I hope it helps ! Cheers !

The problem is the Postman extension. You should use the installed Postman app in your browser, not the extension.

Yes i solved it Just try to move the bodyParser middleware above the router middleware!

var express=require('express'); var app=express(); var router = require('./api'); var parser = require('body-parser'); require('./database'); require('./seed.js'); app.use(parser.json()); app.use('/',express.static('public')); app.use('/api',router);

app.listen(3000,function(){ console.log("The server is running on port 3000"); });

Hi Iain

Thx for your help. Here it is:

'use strict'; 

var express = require('express');
var Todo = require('../models/todo');
var todos = require('../../mock/todos.json');

var router = express.Router();

// app.get('/api/todos', function(req, res) {
//  // res.send('These are the todos');
//  res.json({todos:[]});
// });

router.get('/todos', function(req, res) {
    Todo.find({}, function(err, todos){
        if(err) {
            // return console.log(err);
            return res.status(500).json({message: err.message});
        } 

        res.json({todos: todos}); //2nd todos = the array import above
    });

});

// TODO: Add POST route to create new entries
router.post('/todos', function(req, res) {
    var todo = req.body;
    // res.send(todo);
    Todo.create(todo, function(err, todo) {
        if(err) {
            return res.status(500).json({err: err.message});
        }
        res.json({'todo': todo, message: 'Todo created ok'});
    })
});

// TODO: Add put route to update entries
router.put('/todos/:id', function(req, res) {
    var id = req.params.id;
    var todo = req.body;
    // if(todo) {
//          res.status(500).json({err: "ok there is a todo"});
//  }
    // if(todo._id !== id) {
//          res.status(500).json({err: "ok todo._id !== id"});
//  }
    if(todo && todo._id !== id) {
            return res.status(500).json({err: "Ids don't match"});
    }
    Todo.findByIdAndUpdate(id, todo, {new: true}, function(err, todo) {
        if(todo && todo._id !== id) {
            return res.status(500).json({err: err.message});
        }
        res.send({'todo': todo, message: 'Todo updated'});  
    })
});




// TODO: add delete route to delete entries

module.exports = router;

Try changing your last response to use res.json() instead of res.send(), and you also only need to check for err from within the Todo.findByIdAndUpdate function call:

    Todo.findByIdAndUpdate(id, todo, {new: true}, function(err, todo) {
        if(err) {
            return res.status(500).json({err: err.message});
        }
        res.json({'todo': todo, message: 'Todo updated'});  
    })

Hmmm well it's possibly your Mongoose schema/model isn't configured correctly, so it's only creating a document in MongoDB with the fields that it automatically adds (_id and __v).

Can you post the code from your models/todo.js file? If you put the language/syntax/file extension after the first lot of three backticks for the code block you will get the syntax highlighting:

    ```js
    function myFunc(arg) {
      return arg;
    }
    ```

Would look like this:

function myFunc(arg) {
  return arg;
}

You can also use a file name:

    ```app.js
    function myFunc(arg) {
      return arg;
    }
    ```

That would look like this:

function myFunc(arg) {
  return arg;
}

Thanks Iain

I'm posting my /models/todo.js but I compared to the final files I downloaded and they seem identical. I will continue to check each file. Maybe the json parser does not work, but how could I test that? Thanks for the markdown tip :)

'use strict';

var mongoose = require('mongoose');

// todo.name
// todo.completed

var todoSchema = new mongoose.Schema({
    name: String,
    completed: Boolean
});

var model = mongoose.model('Todo', todoSchema);

module.exports = model;

Did you manage to solve this? I have the same problem - no 'name' and 'completed' in response JSON object

Hi Shon. No I started over and carefully followed all instructions, checked my code against the teacher files, but I was not able to make it work. I just moved on.

Great thx I will give it a try.

I had the same problem, however, I realized that I had forgotten to add some codes in src/app.js Adding the code bellow fixed the problem. (video around 3:54)

 var parser = require('body-parser');
.
.
.
app.use('/', express.static('public'));
app.use(parser.json());

Has anyone else solved this issue yet? I also cannot get the todo body to show up.

I changed my code in src/app.js to look like this

app.use('/',express.static('public')); app.use(parser.json()); app.use('/api', router);

instead of

app.use('/',express.static('public')); app.use('/api', router); app.use(parser.json());