Replacing previous array of objects with new array of objects using express/mongoose

Question

I am trying to update the favourite stations array in my database, however I just can't seem to be able to update it? I have searched for ages to try and find a solution, but nothing seems to work.

I have the ID saved in the cookie, and am sending that in the body when the update request is made...

"selected" is an array of objects. this is what will need replace the contents of the to the favouriteStations property, or add it if it is empty.

the database structure is like this:

"id": id,
"email: "test@test",
"password": "test",
"userInfo": {
    "firstName": "test",
    "lastName": "test",
    "favouriteStations": [{array i want to replace}]
}

i have tried many different combinations, however this is what i have so far, and it doesnt work

app.post("/api/update", (req, res) => {
  console.log("updating user details");
  const { id, selected } = req.body;

  UserModel.findOneAndUpdate(
    { _id: id },
    { userInfo: { favouriteStations: { $set: selected } } },
    { useFindAndModify: false }
  )
    .then((user, err) => {
      if (err) {
        console.log("an error?");
        res.status(500).json({
          error: "Server error",
        });
      } else if (!user) {
        console.log("user not exists?");
        res.status(401).json({
          error: "Incorrect credentials",
        });
      } else {
        console.log("updated?");
        console.log(user);
      }
    })
    .catch((err) => {
      console.log(err);
    });
});

Kuyashii · Accepted Answer · 2020-04-27 14:30:37Z

1

It is unclear as to what you mean by ".. it doesn't work", can you elaborate please i.e. what errors do you receive? If none, what does happen?

The $set operator should be used in the following form, from MongoDB docs:

{ $set: { <field1>: <value1>, ... } }

In saying that, you may not be accessing favouriteStations correctly, use dot-notation to access the subdocument as follows:

...
    UserModel.findOneAndUpdate(
        { _id: id },
        { $set: {'userInfo.favouriteStations': selected} } ,
        { useFindAndModify: false }
      )
...

edited Apr 27, 2020 at 14:30

answered Apr 27, 2020 at 14:11

Kuyashii

3981 gold badge4 silver badges25 bronze badges

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

2 Comments

norrisollie Over a year ago

Thanks. I ended up changing the database structure instead, but I think I was using the $set wrong in the first place. I also think the schema was wrong too. It works now so I’m pleased. Thanks :)

Kuyashii Over a year ago

You're welcome @norrisollie - yes, as I wrote - I think you were close, just needed to alter how you were using $set. Please mark as answer if was helpful, thanks.

Collectives™ on Stack Overflow

Replacing previous array of objects with new array of objects using express/mongoose

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related