Cannot update with timestamps set to true

[lpc921]

description

Using the update method on a model with primary or unique field, and timestamps: true option results in exception.

steps to reproduce

1. Create a model with timestamps option to true
2. Update an item to the table using .update

source

const { connect } = require("trilogy");

async function test() {
  const db = connect(":memory:");

  const model = await db.model(
    "tests",
    {
      id: { type: Number, primary: true },
      item: String,
    },
    { timestamps: true }
  );
  await model.create({ id: 1, item: "test1" });
  await model.update({ id: 1 }, { item: "test2" });
}

test();

error

node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[Error: update `tests` set `item` = 'test2' where `id` = 1 - SQLITE_ERROR: no such column: current_timestamp] {
  errno: 1,
  code: 'SQLITE_ERROR'
}

expected

Update should work

actual

Exception was thrown

[haltcase]

Hey @lpc921, nice find! This was a bug in the timestamp update trigger that escaped current_timestamp as if it were a column.

 const TimestampTriggerTemplate = `
 create trigger if not exists :name: after update on :modelName: begin
-  update :modelName: set :column: = \`current_timestamp\`
+  update :modelName: set :column: = current_timestamp
   where :key: = \`old\`.:key:;
 end
 `

I've pushed a patch already but it looks like there are some dependency issues to work out before a release can happen.