Chained update with increment?

Sorry if I’m missing something, but I can’t seem to be able to use .increment() along with updating other fields on the table.

For instance, I’d want to do:

knex('products')
  .whereIn('id', ids)
  .update({'updated_at': new Date()})
  .increment('count', 1);

But running this I get:

Possibly unhandled Error: The query type has already been set to update

I understand why it happens, I just think it would be nice to be able to add other fields to the update.

Alternatively, maybe we could remove .increment() altogether and somehow in the .update() itself be able to reference the table’s columns. Maybe something like:

knex('products')
  .whereIn('id', ids)
  .update({
    'updated_at': new Date(),
    'count': function() {
      return this + 1;
    }
  });

Yeah, not sure about the “this + 1” part, but maybe have another way of saying “the existing column value”. This would avoid the need for .increment() and .decrement().

I realize I could write this using .raw(), but it would be nice to have it in the library.

What do you think?

[imiric]

Oh, my bad, I realized I can do this:

knex('products')
  .whereIn('id', ids)
  .update({
    'updated_at': new Date(),
    'count': knex.raw('count + 1')
  });

Sorry, disregard this issue, and thanks for the awesome lib! 😃

[boutell]

In addition, this would be nice:

.increment({
  'thumbs_up': 1,
  'thumbs_down': -1
})

And it would continue to be a handy option after the more difficult issue of making increment truly chainable is resolved.