-var db = require("../utils/database");
+const db = require("../utils/database");
const UserModel = require("./User");
/*
* vid: integer (variant id)
* fenStart: varchar (initial position)
* fen: varchar (current position)
- * timeControl: string
+ * cadence: string
* score: varchar (result)
* scoreMsg: varchar ("Time", "Mutual agreement"...)
* created: datetime
- * drawOffer: boolean
+ * drawOffer: char ('w','b' or '' for none)
*
* Structure table Players:
* gid: ref game id
const GameModel =
{
checkGameInfo: function(g) {
- if (!g.id.toString().match(/^[0-9]+$/))
- return "Wrong game ID";
- if (!g.vid.toString().match(/^[0-9]+$/))
- return "Wrong variant ID";
- if (!g.vname.match(/^[a-zA-Z0-9]+$/))
- return "Wrong variant name";
- if (!g.timeControl.match(/^[0-9dhms +]+$/))
- return "Wrong characters in time control";
- if (!g.fen.match(/^[a-zA-Z0-9, /-]*$/))
- return "Bad FEN string";
- if (g.players.length != 2)
- return "Need exactly 2 players";
- if (g.players.some(p => !p.id.toString().match(/^[0-9]+$/)))
- return "Wrong characters in player ID";
- return "";
+ return (
+ g.vid.toString().match(/^[0-9]+$/) &&
+ g.cadence.match(/^[0-9dhms +]+$/) &&
+ g.fen.match(/^[a-zA-Z0-9, /-]*$/) &&
+ g.players.length == 2 &&
+ g.players.every(p => p.id.toString().match(/^[0-9]+$/))
+ );
},
- create: function(vid, fen, timeControl, players, cb)
- {
- db.serialize(function() {
- let query =
- "INSERT INTO Games"
- + " (vid, fenStart, fen, score, timeControl, created, drawOffer)"
- + " VALUES (" + vid + ",'" + fen + "','" + fen + "','*','"
- + timeControl + "'," + Date.now() + "," + false + ")";
+ create: function(vid, fen, cadence, players, cb)
+ {
+ db.serialize(function() {
+ let query =
+ "INSERT INTO Games " +
+ "(vid, fenStart, fen, score, cadence, created, drawOffer) " +
+ "VALUES " +
+ "(" + vid + ",'" + fen + "','" + fen + "','*','" + cadence + "'," + Date.now() + ",'')";
db.run(query, function(err) {
- if (!!err)
- return cb(err);
- players.forEach((p,idx) => {
- const color = (idx==0 ? "w" : "b");
- query =
- "INSERT INTO Players VALUES " +
- "(" + this.lastID + "," + p.id + ",'" + color + "')";
- db.run(query);
- });
- cb(null, {gid: this.lastID});
- });
- });
- },
+ if (err)
+ cb(err)
+ else
+ {
+ players.forEach((p,idx) => {
+ const color = (idx==0 ? "w" : "b");
+ query =
+ "INSERT INTO Players VALUES " +
+ "(" + this.lastID + "," + p.id + ",'" + color + "')";
+ db.run(query);
+ });
+ cb(null, {gid: this.lastID});
+ }
+ });
+ });
+ },
- // TODO: queries here could be async, and wait for all to complete
- getOne: function(id, cb)
- {
- db.serialize(function() {
- // TODO: optimize queries?
- let query =
- // NOTE: g.scoreMsg can be NULL
- // (in this case score = "*" and no reason to look at it)
- "SELECT g.id, g.vid, g.fen, g.fenStart, g.timeControl, g.score, " +
- "g.scoreMsg, v.name AS vname " +
- "FROM Games g " +
+ // TODO: some queries here could be async
+ getOne: function(id, light, cb)
+ {
+ // NOTE: ignoring errors (shouldn't happen at this stage)
+ db.serialize(function() {
+ let query =
+ "SELECT g.id, g.vid, g.fen, g.fenStart, g.cadence, g.created, g.score, " +
+ "g.scoreMsg, g.drawOffer, v.name AS vname " +
+ "FROM Games g " +
"JOIN Variants v " +
" ON g.vid = v.id " +
- "WHERE g.id = " + id;
- db.get(query, (err,gameInfo) => {
- if (!!err)
- return cb(err);
- query =
- "SELECT p.uid, p.color, u.name " +
- "FROM Players p " +
+ "WHERE g.id = " + id;
+ db.get(query, (err,gameInfo) => {
+ query =
+ "SELECT p.uid, p.color, u.name " +
+ "FROM Players p " +
"JOIN Users u " +
" ON p.uid = u.id " +
- "WHERE p.gid = " + id;
- db.all(query, (err2,players) => {
- if (!!err2)
- return cb(err2);
- query =
- "SELECT squares, played, idx " +
- "FROM Moves " +
- "WHERE gid = " + id;
- db.all(query, (err3,moves) => {
- if (!!err3)
- return cb(err3);
- query =
- "SELECT msg, name, added " +
- "FROM Chats " +
+ "WHERE p.gid = " + id;
+ db.all(query, (err2,players) => {
+ if (light)
+ {
+ query =
+ "SELECT COUNT(*) AS nbMoves " +
+ "FROM Moves " +
"WHERE gid = " + id;
- db.all(query, (err4,chats) => {
- if (!!err4)
- return cb(err4);
- const game = Object.assign({},
+ db.get(query, (err,ret) => {
+ const game = Object.assign({},
gameInfo,
- {
- players: players,
- moves: moves,
- chats: chats,
- }
+ {players: players},
+ {movesCount: ret.nbMoves}
);
- return cb(null, game);
+ cb(null, game);
+ });
+ }
+ else
+ {
+ // Full game requested:
+ query =
+ "SELECT squares, played, idx " +
+ "FROM Moves " +
+ "WHERE gid = " + id;
+ db.all(query, (err3,moves) => {
+ query =
+ "SELECT msg, name, added " +
+ "FROM Chats " +
+ "WHERE gid = " + id;
+ db.all(query, (err4,chats) => {
+ const game = Object.assign({},
+ gameInfo,
+ {
+ players: players,
+ moves: moves,
+ chats: chats,
+ }
+ );
+ cb(null, game);
+ });
});
- });
- });
- });
- });
- },
+ }
+ });
+ });
+ });
+ },
- getByUser: function(uid, excluded, cb)
- {
- db.serialize(function() {
- // Next query is fine because a player appear at most once in a game
- const query =
- "SELECT gid " +
- "FROM Players " +
- "WHERE uid " + (excluded ? "<>" : "=") + " " + uid;
- db.all(query, (err,gameIds) => {
- if (!!err)
- return cb(err);
- gameIds = gameIds || []; //might be empty
- let gameArray = [];
- for (let i=0; i<gameIds.length; i++)
- {
- GameModel.getOne(gameIds[i]["gid"], (err2,game) => {
- if (!!err2)
- return cb(err2);
- gameArray.push(game);
- // Call callback function only when gameArray is complete:
- if (i == gameIds.length - 1)
- return cb(null, gameArray);
- });
- }
- });
- });
- },
+ // For display on MyGames or Hall: no need for moves or chats
+ getByUser: function(uid, excluded, cb)
+ {
+ db.serialize(function() {
+ let query = "";
+ if (uid == 0)
+ {
+ // Special case anonymous user: show all games
+ query =
+ "SELECT id AS gid " +
+ "FROM Games";
+ }
+ else
+ {
+ // Registered user:
+ query =
+ "SELECT gid " +
+ "FROM Players " +
+ "GROUP BY gid " +
+ "HAVING COUNT(uid = " + uid + " OR NULL) " +
+ (excluded ? " = 0" : " > 0");
+ }
+ db.all(query, (err,gameIds) => {
+ if (err || gameIds.length == 0)
+ cb(err, []);
+ else
+ {
+ let gameArray = [];
+ let gCounter = 0;
+ for (let i=0; i<gameIds.length; i++)
+ {
+ GameModel.getOne(gameIds[i]["gid"], true, (err2,game) => {
+ gameArray.push(game);
+ gCounter++; //TODO: let's hope this is atomic?!
+ // Call callback function only when gameArray is complete:
+ if (gCounter == gameIds.length)
+ cb(null, gameArray);
+ });
+ }
+ }
+ });
+ });
+ },
getPlayers: function(id, cb)
{
db.serialize(function() {
const query =
- "SELECT id " +
+ "SELECT uid " +
"FROM Players " +
"WHERE gid = " + id;
db.all(query, (err,players) => {
checkGameUpdate: function(obj)
{
// Check all that is possible (required) in obj:
- if (!!obj.move)
- {
- if (!obj.move.played.toString().match(/^[0-9]+$/))
- return "Wrong move played time";
- if (!obj.move.idx.toString().match(/^[0-9]+$/))
- return "Wrong move index";
- }
- if (!!obj.fen && !obj.fen.match(/^[a-zA-Z0-9, /-]*$/))
- return "Wrong FEN string";
- if (!!obj.score && !obj.score.match(/^[012?*\/-]+$/))
- return "Wrong characters in score";
- if (!!obj.scoreMsg && !obj.scoreMsg.match(/^[a-zA-Z ]+$/))
- return "Wrong characters in score message";
- if (!!obj.chat)
- return UserModel.checkNameEmail({name: obj.chat.name});
- return "";
+ return (
+ (
+ !obj.move || (
+ !!(obj.move.played.toString().match(/^[0-9]+$/)) &&
+ !!(obj.move.idx.toString().match(/^[0-9]+$/))
+ )
+ ) && (
+ !obj.drawOffer || !!(obj.drawOffer.match(/^[wbtn]$/))
+ ) && (
+ !obj.fen || !!(obj.fen.match(/^[a-zA-Z0-9, /-]*$/))
+ ) && (
+ !obj.score || !!(obj.score.match(/^[012?*\/-]+$/))
+ ) && (
+ !obj.scoreMsg || !!(obj.scoreMsg.match(/^[a-zA-Z ]+$/))
+ ) && (
+ !obj.chat || UserModel.checkNameEmail({name: obj.chat.name})
+ )
+ );
},
- // obj can have fields move, chat, fen, drawOffer and/or score
- update: function(id, obj)
+ // obj can have fields move, chat, fen, drawOffer and/or score + message
+ update: function(id, obj, cb)
{
- db.parallelize(function() {
+ db.parallelize(function() {
let query =
"UPDATE Games " +
"SET ";
let modifs = "";
- if (!!obj.message)
- modifs += "message = message || ' ' || '" + obj.message + "',";
- if ([true,false].includes(obj.drawOffer))
- modifs += "drawOffer = " + obj.drawOffer + ",";
- if (!!obj.fen)
+ // NOTE: if drawOffer is set, we should check that it's player's turn
+ // A bit overcomplicated. Let's trust the client on that for now...
+ if (obj.drawOffer)
+ {
+ if (obj.drawOffer == "n") //Special "None" update
+ obj.drawOffer = "";
+ modifs += "drawOffer = '" + obj.drawOffer + "',";
+ }
+ if (obj.fen)
modifs += "fen = '" + obj.fen + "',";
- if (!!obj.score)
+ if (obj.score)
modifs += "score = '" + obj.score + "',";
- if (!!obj.scoreMsg)
+ if (obj.scoreMsg)
modifs += "scoreMsg = '" + obj.scoreMsg + "',";
modifs = modifs.slice(0,-1); //remove last comma
if (modifs.length > 0)
query += modifs + " WHERE id = " + id;
db.run(query);
}
+ // NOTE: move, chat and delchat are mutually exclusive
if (!!obj.move)
{
- const m = obj.move;
+ // Security: only update moves if index is right
query =
- "INSERT INTO Moves (gid, squares, played, idx) VALUES " +
- "(" + id + ",?," + m.played + "," + m.idx + ")";
- db.run(query, JSON.stringify(m.squares));
+ "SELECT MAX(idx) AS maxIdx " +
+ "FROM Moves " +
+ "WHERE gid = " + id;
+ db.get(query, (err,ret) => {
+ const m = obj.move;
+ if (!ret.maxIdx || ret.maxIdx + 1 == m.idx) {
+ query =
+ "INSERT INTO Moves (gid, squares, played, idx) VALUES " +
+ "(" + id + ",?," + m.played + "," + m.idx + ")";
+ db.run(query, JSON.stringify(m.squares));
+ cb(null);
+ }
+ else cb({errmsg:"Wrong move index"});
+ });
}
- if (!!obj.chat)
+ else cb(null);
+ if (obj.chat)
{
- query =
- "INSERT INTO Chats (gid, msg, name, added) VALUES ("
- + id + ",?,'" + obj.chat.name + "'," + "," + Date.now() + ")";
+ query =
+ "INSERT INTO Chats (gid, msg, name, added) VALUES ("
+ + id + ",?,'" + obj.chat.name + "'," + Date.now() + ")";
db.run(query, obj.chat.msg);
}
+ else if (obj.delchat)
+ {
+ query =
+ "DELETE " +
+ "FROM Chats " +
+ "WHERE gid = " + id;
+ db.run(query);
+ }
});
},
- remove: function(id)
- {
- db.parallelize(function() {
- let query =
- "DELETE FROM Games " +
- "WHERE id = " + id;
- db.run(query);
- query =
- "DELETE FROM Players " +
- "WHERE gid = " + id;
- db.run(query);
- query =
- "DELETE FROM Moves " +
- "WHERE gid = " + id;
- db.run(query);
- query =
- "DELETE FROM Chats " +
- "WHERE gid = " + id;
- db.run(query);
- });
- },
+ remove: function(id)
+ {
+ db.parallelize(function() {
+ let query =
+ "DELETE FROM Games " +
+ "WHERE id = " + id;
+ db.run(query);
+ query =
+ "DELETE FROM Players " +
+ "WHERE gid = " + id;
+ db.run(query);
+ query =
+ "DELETE FROM Moves " +
+ "WHERE gid = " + id;
+ db.run(query);
+ query =
+ "DELETE FROM Chats " +
+ "WHERE gid = " + id;
+ db.run(query);
+ });
+ },
cleanGamesDb: function()
{
const day = 86400000;
db.serialize(function() {
let query =
- "SELECT id,score " +
+ "SELECT id, created " +
"FROM Games ";
db.all(query, (err,games) => {
games.forEach(g => {
query =
- "SELECT max(played) AS lastMaj " +
+ "SELECT count(*) as nbMoves, max(played) AS lastMaj " +
"FROM Moves " +
"WHERE gid = " + g.id;
- db.get(query, (err2,updated) => {
- if (!updated && tsNow - g.created > 7*day)
- return GameModel.remove(g.id);
- const lastMaj = updated.lastMaj;
- if (g.score != "*" && tsNow - lastMaj > 7*day ||
- g.score == "*" && tsNow - lastMaj > 91*day)
+ db.get(query, (err2,mstats) => {
+ // Remove games still not really started,
+ // with no action in the last 3 months:
+ if ((mstats.nbMoves == 0 && tsNow - g.created > 91*day) ||
+ (mstats.nbMoves == 1 && tsNow - mstats.lastMaj > 91*day))
{
GameModel.remove(g.id);
}