Use TRIGGER statement in Web SQL Database

[imiaou]

I use Web SQL Database to store data in my extension.
What I want is changing the modified time whenever a row in my table
been UPDATE(d).
Here is my code:

tx.executeSql(
'CREATE TABLE items ('
+'id INTEGER PRIMARY KEY,'
+'title TEXT NOT NULL,'
+'desc TEXT,'
+'url TEXT,'
+'list TEXT,'
+'tag TEXT,'
+'updated DATETIME DEFAULT CURRENT_TIMESTAMP ,'
+'created DATETIME DEFAULT CURRENT_TIMESTAMP);'

+'CREATE TRIGGER update_modified_time UPDATE ON items'
+'BEGIN'
+ 'UPDATE items SET updated = NOW();'
+'END;'
);

Whenever I run it, there's a "not an error" error. Quite weird.

[PhistucK]

If I am not mistaking, triggers are not supported within Web SQL Databases.

Anyway, the error sounds like really cryptic.

Search crbug.com for an existing issue and star it. If you cannot find one, file a new issue at new.crbug.com.

Thank you.

☆PhistucK

--
You received this message because you are subscribed to the Google Groups "Chromium HTML5" group.
To post to this group, send email to chromiu...@chromium.org.
To unsubscribe from this group, send email to chromium-html...@chromium.org.
For more options, visit this group at http://groups.google.com/a/chromium.org/group/chromium-html5/?hl=en.

[Jeremy Orlow]

Agreed.

[Dumitru Daniliuc]

i'll take a look at this.

[Dumitru Daniliuc]

so the problem is that you're trying to execute 2 statements in the same call to executeSql(). when you try to do that, we return SQLITE_ERROR, which apparently has an error message of "not an error"... i'm not sure that there's a better error to return: http://www.sqlite.org/c3ref/c_abort.html.

if you move your CREATE TRIGGER statement into a separate executeSql() call, you should be fine. we do allow triggers. you will want to add some spaces between 'UPDATE ON items', 'BEGIN', 'UPDATE items', etc. in your statement though; otherwise it's equivalent to: CREATE TRIGGER update_modified_time UPDATE ON itemsBEGINUPDATE items SET updated = NOW();END;. also, you might want to consider changing "CREATE TABLE items" to "CREATE TABLE IF NOT EXISTS items", and do the same thing for the trigger.

dumi

[imiaou]

Thanks. I'll try it.

[imiaou]

It works!
Many thanks, dumi.

I have another question:
Does Chrome 7 stable implement all the sqlite feature so I don't worry
about if the feature is supported when I want to use it? Or is there a
status page I can check the implementation?

[PhistucK]

I think it does. There have not been many changes\new features since Chrome 5 or so, I believe, in this area.

☆PhistucK

[Dumitru Daniliuc]

this is correct. we should have support for all sqlite features, except a few of them that either didn't make much sense in the WebSQLDBs context (ATTACH or VACUUM (which btw, we do automatically for you)), or were considered a security concern (RANDOMBLOB, for example).

dumi