Error in creating Rest api in Javascript

Question

The error that is returning is C:\Users\Eric Vitta\Documents\documentos cursos\javascript\src>node index.js TypeError: Cannot read property 'map' of undefined at router.post (C:\Users\Eric Vitta\Documents\documentos cursos\javascript\src\app\controller\projectController.js:37:31) at process._tickCallback (internal/process/next_tick.js:68:7)

I am creating an API to test in Jmeter, I am using Insomnia to test and pass information to the register.

This is the github link with all the code https://github.com/ericvitta/apiteste

It is an API to test REST methods, only for the purpose of study and language improvement. I don’t know why it isn’t returning values in the req.body command because I pass the information in JSON format.

const mongoose = require('../../database');
const bcrypt = require('bcryptjs');

const ProjectSchema = new mongoose.Schema({
  title: {
    type: String,
    require: true,
  },
  description: {
    type: String,
    require: true,
  },
  user: {
    type: mongoose.Schema.Types.ObjectId,
    ref: 'User',
    require: true,
  },
  tasks: [{
    type: mongoose.Schema.Types.ObjectId,
    ref: 'Task',
  }],
  createdAt: {
    type: Date,
    default: Date.now,
  },
});

const Project = mongoose.model('Project', ProjectSchema);

module.exports = Project;

const mongoose = require('../../database');
const bcrypt = require('bcryptjs');

const TaskSchema = new mongoose.Schema({
  title: {
    type: String,
    require: true,
  },
  project: {
    type: mongoose.Schema.Types.ObjectId,
    ref: 'Project',
    require: true,
  },
  assignedTo: {
    type: mongoose.Schema.Types.ObjectId,
    ref: 'User',
    require: true,
  },
  completed: {
    type: Boolean,
    require: true,
    default: false,
  },
  createdAt: {
    type: Date,
    default: Date.now,
  },
});

const Task = mongoose.model('Task', TaskSchema);

module.exports = Task;

const express = require ('express');
const authMiddleware = require('../middlewares/auth');

const Project = require('../models/Project');
const Task = require('../models/Task');

const router = express.Router();

router.use(authMiddleware);

router.get('/', async (_req, res) => {
   try {
       const projects = await Project.find().populate(['user', 'tasks']);
       return res.send({ projects});

   }catch (err) {
       return res.status(400).send({ error: 'Erro ao carregar projetos.'});
   }
});

router.get('/:projectId', async (req, res) => {
    try {
        const project = await Project.findById(req.params.projectId).populate(['user', 'tasks']);
        return res.send({ project});
 
    }catch (err) {
        return res.status(400).send({ error: 'Erro ao carregar projetos.'});
    }
});

router.post('/', async (req, res) => {
    try {
        const { title, description, tasks } = req.body;

        const project = await Project.create({ title, description, user: req.userId });
      
      await Promise.all(tasks.map(async task => {

          const projectTask = new Task({...task, project: project._id}); 

          await projectTask.save();

         project.tasks.push(projectTask);
    }));
        
        await project.save();  

      return res.send({ project });
    } catch (err) {
        console.log(err);
      return res.status(400).send({ error: 'Erro ao criar novo projeto.' });
    }
});

router.put('/:projectId', async (req, res) => {
    try {
        const { title, description, tasks } = req.body;

        const project = await Project.findByIdAndUpdate(req.params.projectId , {
            title,
            description
        }, {new: true});

        project.tasks = [];
        
        await Task.remove({ project: project._id});
      
      await Promise.all(tasks.map(async task => {

          const projectTask = new Task({...task, project: project._id}); 

          await projectTask.save();

         project.tasks.push(projectTask);
         console.log({ tasks });
    }));
        
        await project.save();  

      return res.send({ project });
    } catch (err) {
        console.log(err);
      return res.status(400).send({ error: 'Erro ao criar novo projeto.' });
    }
});

router.delete('/:projectId', async (req, res) => {
    try {
        await Project.findByIdAndRemove(req.params.projectId);
        return res.send({ Message:'Projeto removido com sucesso.'});
 
    }catch (err) {
        return res.status(400).send({ error: 'Erro ao deletar projeto.'});
    }
});



module.exports = app => app.use('/projects', router);

Answer 1

Friend, change in INSONIA your http request the key "tasks" to "tasks" as this in your code. (as mentioned by friend Christian Luã Lemos)

 const { title, description, tasks } = req.body;

Other than that, I noticed that you use the following code:

app.use(bodyParser.urlencoded({ extended: false}));

Change to:

app.use(bodyParser.urlencoded({ extended: true}));

This is because with the true option it will make body-parser use the library Qs, this library can work with nested Objects. I talked about her in this other link: What Expressjs urlencoded means in practice?

Answer 2

You’re picking up the attributes title, description and tasks of the object req.body in this line:

const { title, description, tasks } = req.body;

The error that is occurring in the project is indicating that the variable tasks is undefined, that is, there is no req.body.tasks.

This effect can be observed in the code below:

let obj = {title : "foo", description : "bar"}
let {title, description, tasks} = obj

console.log(title) //foo
console.log(description) //bar
console.log(tasks) //undefined