search

axemclion / jquery-indexeddb

An IndexedDB Plugin for Jquery.
Other
195 stars 71 forks source link

"delete" is a keyword #27

Closed stimg closed 11 years ago

stimg commented 11 years ago

In the code line like

transaction.objectStore('store_name').delete(id)

javascript keyword "delete" on this position will be recognized in some IDE like an error (at least in Eclipse and WebStorm). May be it would be better to use "erase" | "drop" | "clear" etc. instead.

axemclion commented 11 years ago

I wanted to use delete so that it resembles the IndexedDB Spec better. We could alias the 'delete' keyword to something like 'erase'. Note that a clear could mean clear the database of all contents.

You could send a pull request and I could add the feature.

axemclion commented 11 years ago

Done in e49ff3c