From dd10eb93984e629c4ee5a07cd3b875fdb20f0460 Mon Sep 17 00:00:00 2001 From: Benjamin Auder Date: Fri, 1 May 2020 20:31:02 +0200 Subject: [PATCH] Draft or user bio support (ready in Hall, TODO in Game) --- TODO | 4 +- client/src/components/UserBio.vue | 90 +++++++++++++++++++++++++++++++ client/src/store.js | 3 -- client/src/translations/en.js | 1 - client/src/translations/es.js | 1 - client/src/translations/fr.js | 1 - client/src/variants/Rampage.js | 10 ++++ client/src/views/Hall.vue | 4 +- server/db/create.sql | 2 +- server/models/User.js | 18 +++++-- server/routes/users.js | 32 ++++++----- 11 files changed, 140 insertions(+), 26 deletions(-) create mode 100644 client/src/components/UserBio.vue create mode 100644 client/src/variants/Rampage.js diff --git a/TODO b/TODO index cd695996..c2981cee 100644 --- a/TODO +++ b/TODO @@ -7,10 +7,10 @@ https://www.chessvariants.com/winning.dir/kinglet.html you need to win all the pawns to win the game kings are not royal -https://www.chessvariants.com/diffmove.dir/balaklava.html - Rampage Chess: Any Unit May move to any empty square that is guarded. A King cannot make a special move to get out of check. +https://www.chessvariants.com/rules/pacifist-chess (1 et 2, utilisant somme values ou nb pièces) + Swap chess: Friendly pieces may swap places https://www.chessvariants.com/crossover.dir/koopachess.html diff --git a/client/src/components/UserBio.vue b/client/src/components/UserBio.vue new file mode 100644 index 00000000..c954f3d8 --- /dev/null +++ b/client/src/components/UserBio.vue @@ -0,0 +1,90 @@ + + + + + diff --git a/client/src/store.js b/client/src/store.js index 33d03121..fa363a38 100644 --- a/client/src/store.js +++ b/client/src/store.js @@ -46,7 +46,6 @@ export const store = { name: localStorage.getItem("myname") || "", //"" for "anonymous" email: "", //unknown yet notify: false, //email notifications - newsRead: localStorage.getItem("newsRead") || 0, sid: mysid }; // Slow verification through the server: @@ -79,8 +78,6 @@ export const store = { localStorage.removeItem("myname"); this.state.user.email = json.email; this.state.user.notify = json.notify; - if (!!json.newsRead && json.newsRead > this.state.user.newsRead) - this.state.user.newsRead = json.newsRead; }); // Settings initialized with values from localStorage const getItemDefaultTrue = (item) => { diff --git a/client/src/translations/en.js b/client/src/translations/en.js index 91667a74..a72cf612 100644 --- a/client/src/translations/en.js +++ b/client/src/translations/en.js @@ -154,7 +154,6 @@ export const translations = { "Who's there?": "Who's there?", With: "With", with: "with", - "Write news": "Write news", "Wrong time control": "Wrong time control", "Your message": "Your message", diff --git a/client/src/translations/es.js b/client/src/translations/es.js index daf09aef..10e5422b 100644 --- a/client/src/translations/es.js +++ b/client/src/translations/es.js @@ -154,7 +154,6 @@ export const translations = { "Who's there?": "¿Quién está ahí?", With: "Con", with: "con", - "Write news": "Escribir una news", "Wrong time control": "Cadencia errónea", "Your message": "Tu mensaje", diff --git a/client/src/translations/fr.js b/client/src/translations/fr.js index 6b57c329..c4de1448 100644 --- a/client/src/translations/fr.js +++ b/client/src/translations/fr.js @@ -154,7 +154,6 @@ export const translations = { "Who's there?": "Qui est là ?", With: "Avec", with: "avec", - "Write news": "Écrire une news", "Wrong time control": "Cadence erronée", "Your message": "Votre message", diff --git a/client/src/variants/Rampage.js b/client/src/variants/Rampage.js new file mode 100644 index 00000000..31d4f359 --- /dev/null +++ b/client/src/variants/Rampage.js @@ -0,0 +1,10 @@ +import { ChessRules } from "@/base_rules"; + +// Plan : garder intact isAttacked, +// ajouter "guarded" qui somme les attaques blanches et soustrait les attaques noires sur une case +// (regarder toutes directions, OK) +// --> boulot à faire sur chaque case vide, après chaque getPotentialMoves() +// --> sauf si le roi est en échec (avant de jouer). + +export class RampageRules extends ChessRules { +}; diff --git a/client/src/views/Hall.vue b/client/src/views/Hall.vue index 0e2c1c60..48af22ee 100644 --- a/client/src/views/Hall.vue +++ b/client/src/views/Hall.vue @@ -113,7 +113,7 @@ main v-for="sid in Object.keys(people)" v-if="!!people[sid].name" ) - span {{ people[sid].name }} + UserBio(:id="people[sid].id" :name="people[sid].name") button.player-action( v-if="isGamer(sid)" @click="watchGame(sid)" @@ -212,6 +212,7 @@ import params from "@/parameters"; import { getRandString, shuffle, randInt } from "@/utils/alea"; import { getDiagram } from "@/utils/printDiagram"; import Chat from "@/components/Chat.vue"; +import UserBio from "@/components/UserBio.vue"; import GameList from "@/components/GameList.vue"; import ChallengeList from "@/components/ChallengeList.vue"; import { GameStorage } from "@/utils/gameStorage"; @@ -220,6 +221,7 @@ export default { name: "my-hall", components: { Chat, + UserBio, GameList, ChallengeList }, diff --git a/server/db/create.sql b/server/db/create.sql index 84b72b99..15417b19 100644 --- a/server/db/create.sql +++ b/server/db/create.sql @@ -16,7 +16,7 @@ create table Users ( sessionToken varchar, created datetime, notify boolean, - newsRead datetime + bio text default '' ); create table Problems ( diff --git a/server/models/User.js b/server/models/User.js index edb3c06e..8a97d572 100644 --- a/server/models/User.js +++ b/server/models/User.js @@ -13,7 +13,7 @@ const sendEmail = require('../utils/mailer'); * sessionToken: token in cookies for authentication * notify: boolean (send email notifications for corr games) * created: datetime - * newsRead: datetime + * bio: text */ const UserModel = { @@ -58,6 +58,16 @@ const UserModel = { }); }, + getBio: function(id) { + db.serialize(function() { + const query = + "SELECT bio " + + "FROM Users " + + "WHERE id = " + id; + db.get(query, cb); + }); + }, + ///////// // MODIFY @@ -71,13 +81,13 @@ const UserModel = { }); }, - setNewsRead: function(id) { + setBio: function(id, bio) { db.serialize(function() { const query = "UPDATE Users " + - "SET newsRead = " + Date.now() + " " + + "SET bio = ? " + "WHERE id = " + id; - db.run(query); + db.run(query, bio); }); }, diff --git a/server/routes/users.js b/server/routes/users.js index 7898ac8a..a9adc162 100644 --- a/server/routes/users.js +++ b/server/routes/users.js @@ -4,6 +4,22 @@ const sendEmail = require('../utils/mailer'); const genToken = require("../utils/tokenGenerator"); const access = require("../utils/access"); const params = require("../config/parameters"); +const sanitizeHtml = require('sanitize-html'); + +router.get("/userbio", access.ajax, (req,res) => { + const uid = req.query["id"]; + if (!!(uid.toString().match(/^[0-9]+$/))) { + UserModel.getBio(uid, (err, bio) => { + res.json(bio); + }); + } +}); + +router.put('/userbio', access.logged, access.ajax, (req,res) => { + const bio = sanitizeHtml(req.body.bio); + UserModel.setBio(req.userId, bio); + res.json({}); +}); router.post('/register', access.unlogged, access.ajax, (req,res) => { const name = req.body.name; @@ -36,16 +52,14 @@ router.get("/whoami", access.ajax, (req,res) => { name: user.name, email: user.email, id: user.id, - notify: user.notify, - newsRead: user.newsRead + notify: user.notify }); }; const anonymous = { name: "", email: "", id: 0, - notify: false, - newsRead: 0 + notify: false }; if (!req.cookies.token) callback(anonymous); else if (req.cookies.token.match(/^[a-z0-9]+$/)) { @@ -60,8 +74,8 @@ router.get("/users", access.ajax, (req,res) => { const ids = req.query["ids"]; // NOTE: slightly too permissive RegExp if (ids.match(/^([0-9]+,?)+$/)) { - UserModel.getByIds(ids, (err,users) => { - res.json({users:users}); + UserModel.getByIds(ids, (err, users) => { + res.json({ users:users }); }); } }); @@ -81,12 +95,6 @@ router.put('/update', access.logged, access.ajax, (req,res) => { } }); -// Special route to update newsRead timestamp: -router.put('/newsread', access.logged, access.ajax, (req,res) => { - UserModel.setNewsRead(req.userId); - res.json({}); -}); - // Authentication-related methods: // to: object user (to who we send an email) -- 2.44.0