MongoDB: Removing an array object in Mongoose

Question

I have the following Schema:

var userSchema = mongoose.Schema({

    local            : {
        email        : String,
        password     : String,
        movies       : [{
                          moviename   : String,
                          rating      : Number
                        }],
    }

});

And I use the following way to add entries to the array:

user.local.movies.push({ moviename  : "Top Gun", rating     : 80});
user.save(function (err) {
                if (err)
                    console.log("Error in saving");

                res.end(0);
            });

But I need to remove entries too. I need to be able to remove entries by the "moviename" name. I tried using pull:

 user.local.movies.pull({ moviename  : "Top Gun"});

but it did not work.

Could someone please let me know how I can remove entries from the array?

Thank you.

Do you call save() after pull()?

Gergo Erdosi
– Gergo Erdosi

2014-05-24 17:37:41 +00:00
Commented May 24, 2014 at 17:37 — Gergo Erdosi
– Gergo Erdosi, Commented May 24, 2014 at 17:37
Yes I do. Same as I did for Push.

madu
– madu

2014-05-25 05:29:37 +00:00
Commented May 25, 2014 at 5:29 — madu
– madu, Commented May 25, 2014 at 5:29

JohnnyHK · Accepted Answer · 2014-05-24 23:23:24Z

2

I think it's easier to use an explicit update call instead of Mongoose's array manipulation methods which don't always work as you'd expect:

User.update({_id: user._id}, 
    {$pull: {'local.movies': {moviename: 'Top Gun'}}}, callback);

answered May 24, 2014 at 23:23

JohnnyHK

313k70 gold badges632 silver badges479 bronze badges

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

Comments

dylants · Accepted Answer · 2014-05-25 00:52:44Z

2

One way of doing this is to use the splice function to remove the element from the array, assuming you can find the index. So for instance:

User.findOne(function(err, user) {
    var movies, index;

    movies = user.movies;

    for (index = 0; index < movies.length; index++) {
        if (movies[index].moviename === "Top Gun") {
            break;
        }
    }
    if (index !== movies.length) {
        movies.splice(index, 1);
    }

    user.save(function(err, user) {
        res.send(user);
    });

});

(Know that the above code does this for only one user, and hard codes the movie name to remove, but you get the idea.)

edited May 25, 2014 at 0:52

answered May 24, 2014 at 17:37

dylants

23.6k3 gold badges28 silver badges22 bronze badges

Collectives™ on Stack Overflow

MongoDB: Removing an array object in Mongoose

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related