CODEWITHSUNDEEP
javascriptmongodbexpressmongoose
Hot Zellah
Hot Zellah

Reputation: 1081

MongoError: filter parameter must be an object

I am creating a rest api I have end point for Post/Movies: Request body should contain only movie title, and its presence should be validated Based on passed title, other movie details should be fetched from thememoviedb,and saved to application database.

app.post('/movies', (req, res) => {
        request('https://api.themoviedb.org/3/discover/movie?callback=JSONP_CALLBACK&sort_by=popularity.desc&api_key=2931998c3a80d7806199320f76d65298', function (error, response, body) {
            console.log('error:', error); // Print the error if one occurred and handle it
            console.log('statusCode:', response && response.statusCode); // Print the response status code if a response was received

          });
        db.collection('movies').findOneAndUpdate(req.body.title,{
            title: 'Avengers',
        },(err, result) => {
            if (err) {
                res.send({
                    'error': 'An error has occured'
                });
            } else {
                res.send(result.ops[0]);
            }
        });
});

when I run the app I get this error, what am I doing wrong here,? am new to nodejs and all this stuff just learning

Upvotes: 4

Views: 23079

Answers (2)

Sajeetharan
Sajeetharan

Reputation: 222522

Try the following,

db.collection('movies').findOneAndUpdate({title:req.body.title},{
          $set:{
           'Avengers'
          }})

Upvotes: 0

Adam Azad
Adam Azad

Reputation: 11297

Use $eq operator in the filter object $eq

{ <field>: { $eq: <value> } }

So the final snippet becomes like this:

app.post('/movies', (req, res) => {

    /* code ... */

    let { title } = req.body

    db.collection('movies').findOneAndUpdate({ title: { $eq: title } }, { title: 'Avengers' }, (err, result) => {
        if (err) {
            res.send({ 'error': 'An error has occured' });
        } else {
            res.send(result.ops[0]);
        }
    });

});

Upvotes: 4

Related Questions