projects
/
vchess.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Update TODO
[vchess.git]
/
client
/
src
/
utils
/
gameStorage.js
diff --git
a/client/src/utils/gameStorage.js
b/client/src/utils/gameStorage.js
index
5c094dc
..
1298667
100644
(file)
--- a/
client/src/utils/gameStorage.js
+++ b/
client/src/utils/gameStorage.js
@@
-43,7
+43,7
@@
export const GameStorage = {
// Optional callback to get error status
add: function(game, callback) {
dbOperation((err,db) => {
// Optional callback to get error status
add: function(game, callback) {
dbOperation((err,db) => {
- if (err) {
+ if (
!!
err) {
callback("error");
return;
}
callback("error");
return;
}
@@
-51,6
+51,9
@@
export const GameStorage = {
transaction.oncomplete = function() {
callback(); //everything's fine
};
transaction.oncomplete = function() {
callback(); //everything's fine
};
+ transaction.onerror = function(err) {
+ callback(err); //duplicate key error (most likely)
+ };
let objectStore = transaction.objectStore("games");
objectStore.add(game);
});
let objectStore = transaction.objectStore("games");
objectStore.add(game);
});
@@
-80,8
+83,7
@@
export const GameStorage = {
},
// Retrieve all local games (running, completed, imported...)
},
// Retrieve all local games (running, completed, imported...)
- // light: do not retrieve moves or clocks (TODO: this is the only usage)
- getAll: function(light, callback) {
+ getAll: function(callback) {
dbOperation((err,db) => {
let objectStore = db.transaction("games").objectStore("games");
let games = [];
dbOperation((err,db) => {
let objectStore = db.transaction("games").objectStore("games");
let games = [];
@@
-90,12
+92,11
@@
export const GameStorage = {
// if there is still another cursor to go, keep running this code
if (cursor) {
let g = cursor.value;
// if there is still another cursor to go, keep running this code
if (cursor) {
let g = cursor.value;
- if (light) {
- g.movesCount = g.moves.length;
- delete g.moves;
- delete g.clocks;
- delete g.initime;
- }
+ // Do not retrieve moves or clocks (unused in list mode)
+ g.movesCount = g.moves.length;
+ delete g.moves;
+ delete g.clocks;
+ delete g.initime;
games.push(g);
cursor.continue();
} else callback(games);
games.push(g);
cursor.continue();
} else callback(games);
@@
-110,8
+111,8
@@
export const GameStorage = {
dbOperation((err,db) => {
let objectStore = db.transaction("games").objectStore("games");
objectStore.get(gameId).onsuccess = function(event) {
dbOperation((err,db) => {
let objectStore = db.transaction("games").objectStore("games");
objectStore.get(gameId).onsuccess = function(event) {
- if (event.target.result)
-
callback(event.target.result);
+ // event.target.result is null if game not found
+ callback(event.target.result);
};
});
},
};
});
},
@@
-122,7
+123,7
@@
export const GameStorage = {
if (!err) {
let transaction = db.transaction(["games"], "readwrite");
transaction.oncomplete = function() {
if (!err) {
let transaction = db.transaction(["games"], "readwrite");
transaction.oncomplete = function() {
- callback(
{}
); //everything's fine
+ callback(); //everything's fine
};
transaction.objectStore("games").delete(gameId);
}
};
transaction.objectStore("games").delete(gameId);
}