add forUpdate expression

[khomyakov42]

SELECT * FROM tableName WHERE ... LIMIT .. FOR UPDATE

[tgriesser]

So since forUpdate would only be applicable in the context of a transaction, would it make sense that this method (and potentially others relating to transactions forShare) be dynamically made available after calling the .transacting(trx) on the query builder chain?

[khomyakov42]

Yes

[tgriesser]

@glasscube42 - take a look, should be added & documented now.

[gx0r]

I seem to be having trouble with forUpdate against postgresql with knex 0.5.7. Code is..

knex.transaction(function(t) {
    return knex('users')
      .transacting(t)
      .forUpdate()
      .then(function (s) {
        console.log(s);
        return s;
      });
})
.then(function(resp) {
  console.log('TX complete! ' + JSON.stringify(resp));
  return resp;
})
.catch(function(err) {
    console.log('TX error! ' + err.message);
});

Outputs: 'TX error! Cannot call method 'then' of undefined'

[tgriesser]

Fixing now.

[tgriesser]

@llambda thanks for pointing this out... should be fixed, let me know if it's not working for you.

[gx0r]

@tgriesser Works good now. Thank you for fixing that.