MongoDb: Getting error while pushing documents in an array

Question

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.

Tushar Gupta - curioustushar · Accepted Answer · 2021-07-05 08:13:27Z

2

Try

Change

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

to

return Department.findById(params.departmentId)

findById accept only id not Object

answered Jul 5, 2021 at 8:13

Tushar Gupta - curioustushar

57.2k24 gold badges106 silver badges110 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

Mark Willow Aldave Over a year ago

Thanks. Could you please tell me how to post it in postman? I mean, how do I test it there? What I'm using is this: { "userId": "60e17238415fab21f8cc6b28", "departmentId": "60e27549c36af1272812c4e3" }

Tushar Gupta - curioustushar Over a year ago

@MarkWillowAldave Please share your route details ? Also check - stackoverflow.com/questions/17007997

Mark Willow Aldave Over a year ago

Not sure if this is correct: router.post('/enroll', (req, res) => { UserController.enroll(req.body).then(result => res.send(result)) });

Mark Willow Aldave Over a year ago

Tushar can you help me with this thread :-) stackoverflow.com/questions/68254282/…

Mark Willow Aldave · Accepted Answer · 2021-07-05 09:06:12Z

1

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.

answered Jul 5, 2021 at 9:06

Mark Willow Aldave

959 bronze badges

1 Comment

Tushar Gupta - curioustushar Over a year ago

Glad that it helped you :)

Collectives™ on Stack Overflow

MongoDb: Getting error while pushing documents in an array

2 Answers 2

4 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related