Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Can I conditionally add a where() clause to my knex query?

I want to add a where() clause in my query, but conditionally. Specifically, I want it added only if a sepecific querystring parameter is passed in the URL. Is this possible, and if so, how would I go about doing it?

router.get('/questions', function (req, res) {
    knex('questions')
        .select('question', 'correct', 'incorrect')
        .limit(50)
        .where('somecolumn', req.query.param) // <-- only if param exists
        .then(function (results) {
            res.send(results);
        });
});
like image 739
eponymous23 Avatar asked Dec 01 '14 16:12

eponymous23


3 Answers

Yes. Use modify.

As applied to your example:

router.get('/questions', function (req, res) {
    knex('questions')
        .select('question', 'correct', 'incorrect')
        .limit(50)
        .modify(function(queryBuilder) {
            if (req.query.param) {
                queryBuilder.where('somecolumn', req.query.param);
            }
        })   
        .then(function (results) {
            res.send(results);
        });
});
like image 126
Itai Noam Avatar answered Oct 13 '22 17:10

Itai Noam


You can store your query in a variable, apply your conditional where clause and then execute it, like this :

router.get('/questions', function(req, res) {
  var query = knex('questions')
              .select('question', 'correct', 'incorrect')
              .limit(50);

  if(req.query.param == some_condition)
    query.where('somecolumn', req.query.param) // <-- only if param exists
  else
    query.where('somecolumn', req.query.param2) // <-- for instance

  query.then(function(results) {
    //query success
    res.send(results);
  })
  .then(null, function(err) {
    //query fail
    res.status(500).send(err);
  });
});
like image 49
Simon Briche Avatar answered Oct 13 '22 17:10

Simon Briche


You can actually use query builder inside .where() like so:

.where((qb) => {condition == true ? do something if true : do something if false })

IMO @ItaiNoam answer should be correct with .modify()

like image 2
Merey Nurlan Avatar answered Oct 13 '22 17:10

Merey Nurlan