Business
Jobs
  • About Us
  • Solutions
    • Job Postings
      Post your job and receive qualified candidates in 48h.
    • Candidate Assessments
      500+ technical and psychological tests, plus anti-fraud.
    • Headhunting
      Tailor-made executive search from start to finish.
    • Payroll + EOR
      Payroll dispersal and EOR across 15+ LATAM countries.
  • Pricing
  • Jobs

0

233
Views
MongoDb: Getting error while pushing documents in an array

I need help with my backend. I'm using postman and here's the model of my back end:

name: {
        type: String,
        required: [true, 'Department Name is Required']
    },
    description: {
        type: String,
        required: [true, 'Description is Required']
    },
agents: [
        {
            agentId: {
                type: String,
                required: [true, 'Agent ID is Required']
            },
            createdOn: {
                type: String,
                default: new Date()
            }
        }
]

What I'm trying to do is push documents in agents array but I'm getting some errors.

The routes and controllers are as follows:

Routes:
router.post('/enroll', (req, res) => {
    UserController.enroll(req.body).then(result => res.send(result))
});
Controllers:
module.exports.enroll = (params) => {
    return Department.findById({departmentId: params.departmentId}).then(department => {
        department.agents.push({userId: params.userId})

        return department.save().then((department, error) => {
            return (err) ? false : true
        })
    })
}

This is the error that I'm getting: (node:9916) UnhandledPromiseRejectionWarning: CastError: Cast to ObjectId failed for value "{ departmentId: '60e27549c36af1272812c4e3' }" (type Object) at path "_id" for model "Department"

The target is look for the department id and will push the agent id that I already acquire.

over 4 years ago · Santiago Trujillo
2 answers
Answer question

0

Try

Change

return Department.findById({departmentId: params.departmentId}).

to

return Department.findById(params.departmentId)


findById accept only id not Object

over 4 years ago · Santiago Trujillo Report

0

Thanks Tushar! You're very helpful.

I was able to find the answer using the link you've provided and realized that I'm making some mistakes with the code itself. Hahahaha

I've modified routes to:
router.post('/enroll/:id', (req, res) => {
    const params = {
        departmentId: req.params.id,
       agentId: req.body.agentId
    }
    
    UserController.enroll(params).then(department => res.send(department))
});
Then, some of the lines in my controllers too:
module.exports.enroll = (params) => {
    return Department.findById(params.departmentId).then(department => {
        department.agents.push({agentId: params.agentId})

        return department.save().then((department, error) => {
            return (error) ? false : true
        })
    })
}

It is now working.

over 4 years ago · Santiago Trujillo Report
Answer question
Find remote jobs

Discover the new way to find a job!

Top jobs
Top job categories
Business
Post vacancy Pricing Sales
Legal
Terms and conditions Privacy policy
© 2026 PeakU Inc. All Rights Reserved.
Andres GPT
Show me some job opportunities
There's an error!