From 58aedcd1f49272a6f0dfb7adeeb7650d7daeb75e Mon Sep 17 00:00:00 2001 From: Benjamin Auder Date: Sat, 9 May 2020 16:04:55 +0200 Subject: [PATCH] Unused code, style (cosmetics) --- client/src/store.js | 3 +-- client/src/translations/en.js | 2 -- client/src/translations/es.js | 2 -- client/src/translations/fr.js | 2 -- client/src/views/Game.vue | 3 ++- server/models/Game.js | 12 ++++++++---- server/routes/users.js | 18 +++++++++--------- server/routes/variants.js | 2 +- server/sockets.js | 3 --- 9 files changed, 21 insertions(+), 26 deletions(-) diff --git a/client/src/store.js b/client/src/store.js index fa363a38..0bfeeebb 100644 --- a/client/src/store.js +++ b/client/src/store.js @@ -12,7 +12,6 @@ export const store = { settings: {}, lang: "" }, - socketCloseListener: null, initialize() { const headers = { "Content-Type": "application/json;charset=UTF-8", @@ -97,7 +96,7 @@ export const store = { // Default: random asymmetric this.state.settings.randomness = 2; const supportedLangs = ["en", "es", "fr"]; - const navLanguage = navigator.language.substr(0,2); + const navLanguage = navigator.language.substr(0, 2); this.state.lang = localStorage["lang"] || (supportedLangs.includes(navLanguage) ? navLanguage : "en"); diff --git a/client/src/translations/en.js b/client/src/translations/en.js index 0d1f0b94..285b698a 100644 --- a/client/src/translations/en.js +++ b/client/src/translations/en.js @@ -36,7 +36,6 @@ export const translations = { "Database error: stop private browsing, or update your browser": "Database error: stop private browsing, or update your browser", Delete: "Delete", Deterministic: "Deterministic", - "Discord invitation": "Discord invitation", Download: "Download", Draw: "Draw", "Draw offer only in your turn": "Draw offer only in your turn", @@ -79,7 +78,6 @@ export const translations = { "Name or Email": "Name or Email", Next_p: "Next", Next_g: "Next", - Next_n: "Next", "New connexion detected: tab now offline": "New connexion detected: tab now offline", "New correspondance game:": "New correspondance game:", "New game": "New game", diff --git a/client/src/translations/es.js b/client/src/translations/es.js index d7fef9b1..67d6a527 100644 --- a/client/src/translations/es.js +++ b/client/src/translations/es.js @@ -36,7 +36,6 @@ export const translations = { "Database error: stop private browsing, or update your browser": "Error de la base de datos: detener la navegación privada, o actualizar su navegador", Delete: "Borrar", Deterministic: "Determinista", - "Discord invitation": "Invitación Discord", Download: "Descargar", Draw: "Tablas", "Draw offer only in your turn": "Oferta de tablas solo en tu turno", @@ -79,7 +78,6 @@ export const translations = { "Name or Email": "Nombre o Email", Next_p: "Siguiente", Next_g: "Siguiente", - Next_n: "Siguiente", "New connexion detected: tab now offline": "Nueva conexión detectada: pestaña ahora desconectada", "New correspondance game:": "Nueva partida por correspondencia:", "New game": "Nueva partida", diff --git a/client/src/translations/fr.js b/client/src/translations/fr.js index d66ba0b4..73ff2526 100644 --- a/client/src/translations/fr.js +++ b/client/src/translations/fr.js @@ -36,7 +36,6 @@ export const translations = { "Database error: stop private browsing, or update your browser": "Erreur de base de données : arrêtez la navigation privée, ou mettez à jour votre navigateur", Delete: "Supprimer", Deterministic: "Déterministe", - "Discord invitation": "Invitation Discord", Download: "Télécharger", Draw: "Nulle", "Draw offer only in your turn": "Proposition de nulle seulement sur votre temps", @@ -79,7 +78,6 @@ export const translations = { "Name or Email": "Nom ou Email", Next_p: "Suivant", Next_g: "Suivante", - Next_n: "Suivante", "New connexion detected: tab now offline": "Nouvelle connexion détectée : onglet désormais hors ligne", "New correspondance game:": "Nouvelle partie par corespondance :", "New game": "Nouvelle partie", diff --git a/client/src/views/Game.vue b/client/src/views/Game.vue index 075919db..4afac12d 100644 --- a/client/src/views/Game.vue +++ b/client/src/views/Game.vue @@ -256,7 +256,8 @@ export default { // In case of incomplete information variant: boardDiv.style.visibility = "hidden"; this.atCreation(); - } else + } + else // Same game ID this.nextIds = JSON.parse(this.$route.query["next"] || "[]"); } diff --git a/server/models/Game.js b/server/models/Game.js index 1d2cd684..0d59d6e3 100644 --- a/server/models/Game.js +++ b/server/models/Game.js @@ -381,8 +381,10 @@ const GameModel = "(" + id + ",?," + Date.now() + "," + obj.move.idx + ")"; db.run(query, JSON.stringify(obj.move.squares)); finishAndSendQuery(); - } else cb({ errmsg: "Wrong move index" }); - } else { + } + else cb({ errmsg: "Wrong move index" }); + } + else { if (ret.maxIdx < 2) cb({ errmsg: "Time not over" }); else { // We also need the game cadence @@ -399,14 +401,16 @@ const GameModel = } } }); - } else finishAndSendQuery(); + } + else finishAndSendQuery(); // NOTE: chat and delchat are mutually exclusive if (!!obj.chat) { const 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) { + } + else if (obj.delchat) { const query = "DELETE " + "FROM Chats " + diff --git a/server/routes/users.js b/server/routes/users.js index a3fa7061..7d886b2c 100644 --- a/server/routes/users.js +++ b/server/routes/users.js @@ -34,19 +34,19 @@ router.post('/register', access.unlogged, access.ajax, (req,res) => { const name = req.body.name; const email = req.body.email; const notify = !!req.body.notify; - if (UserModel.checkNameEmail({name: name, email: email})) { + if (UserModel.checkNameEmail({ name: name, email: email })) { UserModel.create(name, email, notify, (err, ret) => { if (!!err) { const msg = err.code == "SQLITE_CONSTRAINT" ? "User name or email already in use" : "User creation failed. Try again"; - res.json({errmsg: msg}); + res.json({ errmsg: msg }); } else { const user = { id: ret.id, name: name, - email: email, + email: email }; setAndSendLoginToken("Welcome to " + params.siteURL, user, res); res.json({}); @@ -85,7 +85,7 @@ router.get("/users", access.ajax, (req,res) => { // NOTE: slightly too permissive RegExp if (ids.match(/^([0-9]+,?)+$/)) { UserModel.getByIds(ids, (err, users) => { - res.json({ users:users }); + res.json({ users: users }); }); } }); @@ -93,7 +93,7 @@ router.get("/users", access.ajax, (req,res) => { router.put('/update', access.logged, access.ajax, (req,res) => { const name = req.body.name; const email = req.body.email; - if (UserModel.checkNameEmail({name: name, email: email})) { + if (UserModel.checkNameEmail({ name: name, email: email })) { const user = { id: req.userId, name: name, @@ -125,7 +125,7 @@ function setAndSendLoginToken(subject, to, res) { router.get('/sendtoken', access.unlogged, access.ajax, (req,res) => { const nameOrEmail = decodeURIComponent(req.query.nameOrEmail); const type = (nameOrEmail.indexOf('@') >= 0 ? "email" : "name"); - if (UserModel.checkNameEmail({[type]: nameOrEmail})) { + if (UserModel.checkNameEmail({ [type]: nameOrEmail })) { UserModel.getOne(type, nameOrEmail, (err,user) => { access.checkRequest(res, err, user, "Unknown user", () => { setAndSendLoginToken("Token for " + params.siteURL, user, res); @@ -137,12 +137,12 @@ router.get('/sendtoken', access.unlogged, access.ajax, (req,res) => { router.get('/authenticate', access.unlogged, access.ajax, (req,res) => { if (!req.query.token.match(/^[a-z0-9]+$/)) - return res.json({errmsg: "Bad token"}); + return res.json({ errmsg: "Bad token" }); UserModel.getOne("loginToken", req.query.token, (err,user) => { access.checkRequest(res, err, user, "Invalid token", () => { // If token older than params.tokenExpire, do nothing if (Date.now() > user.loginTime + params.token.expire) - res.json({errmsg: "Token expired"}); + res.json({ errmsg: "Token expired" }); else { // Generate session token (if not exists) + destroy login token UserModel.trySetSessionToken(user.id, (token) => { @@ -155,7 +155,7 @@ router.get('/authenticate', access.unlogged, access.ajax, (req,res) => { id: user.id, name: user.name, email: user.email, - notify: user.notify, + notify: user.notify }); }); } diff --git a/server/routes/variants.js b/server/routes/variants.js index 3f7417e3..6e0e3af1 100644 --- a/server/routes/variants.js +++ b/server/routes/variants.js @@ -6,7 +6,7 @@ const access = require("../utils/access"); router.get('/variants', access.ajax, function(req, res) { VariantModel.getAll((err,variants) => { - res.json(err || {variantArray:variants}); + res.json(err || { variantArray:variants }); }); }); diff --git a/server/sockets.js b/server/sockets.js index 3a776d1d..9fa09bce 100644 --- a/server/sockets.js +++ b/server/sockets.js @@ -1,7 +1,4 @@ -const url = require('url'); - // Node version in Ubuntu 16.04 does not know about URL class -// NOTE: url is already transformed, without ?xxx=yyy... parts function getJsonFromUrl(url) { const query = url.substr(2); //starts with "/?" let result = {}; -- 2.44.0