How do I filter objects in my function using mongoose?

Question

I only want to pull data unique to a user where my req.user === foundListing.uid. How can I adjust this function to only show those objects?

function show(req, res) {
  db.User.findById(req.user, function (err, user) {
    console.log(req.user)
    if (err) {console.log(err);}
    db.Listing.findById(req.params.listingId, function(err, foundListing) {
      console.log(foundListing.uid );
      if(err) { console.log('listingsController.show error', err); }
      res.json(foundListing);
    });
  })
}

Why do you select user first, and after that listings? If listings is related to user you can create only one request, otherwise this select is unneeded. — alexmac
– alexmac, Commented Sep 6, 2017 at 16:09

Mikey · Accepted Answer · 2017-09-06 14:55:17Z

1

Use find().

function show(req, res) {
  db.User.findById(req.user, function (err, user) {
    console.log(req.user)
    if (err) {console.log(err);}
    db.Listing.find({ uid: req.user }, function(err, listings) {
      console.log(listings);
      if(err) { console.log('listingsController.show error', err); }
      res.json(listings);
    });
  })
}

I don't know if you can directly use req.user. Normally, I use req.user._id.

edited Sep 6, 2017 at 14:55

answered Sep 6, 2017 at 14:54

Mikey

6,7564 gold badges25 silver badges49 bronze badges

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

3 Comments

Mikey Over a year ago

@KennethVo It worked with just req.user too? (I'm assuming req.user is an object and not an ID.)

Kenzo Over a year ago

Yes it worked with req.user. Was following a Satellizer tutorial and that's how it was set up so I guess it was actually an object.

Mikey Over a year ago

@KennethVo Ah interesting. We both learned something new today. Kindly mark your question as answered.

Collectives™ on Stack Overflow

How do I filter objects in my function using mongoose?

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related