fix(storage): bad defaults in options. Fixes #647

This commit is contained in:
Max Lynch
2015-12-04 12:51:44 -06:00
parent 83bf7632f2
commit fa295f9bbe

View File

@ -22,7 +22,7 @@ const DB_NAME = '__ionicstorage';
* }); * });
* *
* // Sql storage also exposes the full engine underneath * // Sql storage also exposes the full engine underneath
* storage.query('insert into projects(name, data) values('Cool Project', 'blah');' * storage.query('insert into projects(name, data) values('Cool Project', 'blah')');
* storage.query('select * from projects').then((resp) => {}) * storage.query('select * from projects').then((resp) => {})
* ``` * ```
* *
@ -39,7 +39,7 @@ export class SqlStorage extends StorageEngine {
static BACKUP_LIBRARY = 1 static BACKUP_LIBRARY = 1
static BACKUP_DOCUMENTS = 0 static BACKUP_DOCUMENTS = 0
constructor(options) { constructor(options={}) {
super(); super();
let dbOptions = util.defaults(options, { let dbOptions = util.defaults(options, {
@ -98,6 +98,7 @@ export class SqlStorage extends StorageEngine {
*/ */
query(query, ...params) { query(query, ...params) {
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
try {
this._db.transaction((tx) => { this._db.transaction((tx) => {
tx.executeSql(query, params, (tx, res) => { tx.executeSql(query, params, (tx, res) => {
resolve({ resolve({
@ -110,7 +111,12 @@ export class SqlStorage extends StorageEngine {
err: err err: err
}); });
}) })
}, err => {
reject(err);
}); });
} catch(e) {
reject(e);
}
}) })
} }