Construct mongodb query with AJAX

Question

I'm trying to get a count of users with the email [email protected] via the ajax below:

function getCount(event) {
event.preventDefault();

var queryCount = {
    email:'[email protected]'
}

$.ajax({
    type: 'GET',
    data: queryCount,
    url: '/countuser',
    dataType: 'JSON'
}).done(function(response) {
    //Check for successful (blank) response
    if(response.msg === '') {

    } else {

    }
});
}

and this in users.js:

exports.countuser = function(db) {
return function(req, res) {
    db.collection('userlist').find(req.body).count(function(err,count) {
      res.send(count);
    });
  }
};

but this isn't working (getting the count of the complete user list, which makes me think that req.body is empty). What would be a correct way of going about this?

Since this is a get, is the req.body empty and req.params populated with your data? — tymeJV
– tymeJV, Commented Feb 27, 2014 at 22:48
bah! Pop open your network request tab in dev tools - what is the url the request is being made to? Also, what does your app.get route look like? — tymeJV
– tymeJV, Commented Feb 27, 2014 at 22:52
Change your route to: app.get('/countuser/:email' and then check req.params — tymeJV
– tymeJV, Commented Feb 27, 2014 at 22:58
@tymeJV he can't have that in the url, it's an email address — Neil Lunn
– Neil Lunn, Commented Feb 27, 2014 at 23:00

Neil Lunn · Accepted Answer · 2014-02-27 23:36:30Z

1

You probably don't want to pass in just req.body but get the param that you posted. But this might be okay as long as you have set up the body parser. There is a better invocation of this but

app.use(express.bodyParser());

But probably your problem with your function is you are not passing req in.

exports.countuser = function(db,req,res) {

Then you need to pass in not only db but req and res as well.

And though you did not post it in your question, your route definition needs to change. Arguments to routes are req and res only as that is what gets passed in:

app.get('/countuser',function(req,res) {

    user.countuser(db,req,res);

So at the moment you are just getting the count, as a number in the response. Which to this point is what you asked for. But that's just a number, it's not JSON. So you need to change a bit.

var countResponse = {};

countResponse.count = count;
res.body(countResponse);

And you also have an object as the data you are passing in. This is not url encoded, use the $.param helper:

.ajax({
    type: 'GET',
    data: $.param(queryCount),
    url: '/countuser',
    dataType: 'JSON'
}).done(function(response) {

In short there are lot of things wrong here. Take some time and read through the documentation.

edited Feb 27, 2014 at 23:36

answered Feb 27, 2014 at 22:50

Neil Lunn

151k36 gold badges356 silver badges327 bronze badges

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

5 Comments

Allen Over a year ago

thanks, but the problem right now isn't the countResponse, its that my ajax request's body and params are empty

Neil Lunn Over a year ago

@Allen was halfway through writing and triggered a save.

Allen Over a year ago

I'm getting a req is not defined: app.get('/countuser', user.countuser(db, req ,res)); exports.countuser = function(db, req, res) {...}

Neil Lunn Over a year ago

@Allen the call on the route is not right. See the edit.

Allen Over a year ago

changed it to how you have implemented it, but not getting a response from ajax call completely

Collectives™ on Stack Overflow

Construct mongodb query with AJAX

1 Answer 1

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related