930408 - Remove affectedRows and lastInsertRowID from Sqlite.jsm documentation

Status: NEW

(Toolkit :: Storage, defect, P3)

Description

[Geoff Lankow (:darktrojan)]

These properties are listed on the MDN page[1], possibly by mistake, or maybe they did once exist, but they don't exist now. It would be useful to have them.

[1] https://developer.mozilla.org/en-US/docs/Mozilla/JavaScript_code_modules/Sqlite.jsm

Comment 1

[Gregory Szorc [:gps]]

There is some context in bug 813833 on why these aren't implemented. Mainly, these APIs and async programming don't go well together. They are traps users can fall into very easily. I'll update the docs.

Comment 2

[Marco Bonardo [:mak]]

I think now it's even worse than just traps, Sqlite added support for WITHOUT ROWID, for these tables lastInsertRowid doesn't make any sense.