From b7cbbda10f16c2419f9a4d49e219fcface5e5658 Mon Sep 17 00:00:00 2001 From: Benjamin Auder Date: Thu, 23 Jan 2020 19:17:50 +0100 Subject: [PATCH] Finish corr implementation for draw offers (untested) --- TODO | 8 ++--- client/src/utils/gameStorage.js | 1 + client/src/views/Game.vue | 55 ++++++++++++++++++++------------- server/db/create.sql | 1 + server/models/Game.js | 9 ++++-- 5 files changed, 44 insertions(+), 30 deletions(-) diff --git a/TODO b/TODO index 8961f060..354e91d6 100644 --- a/TODO +++ b/TODO @@ -1,6 +1,2 @@ -1) abort / draw / resign: pass to opponent if online, store in game if corr -2) styling: connection indicator, names, put movesList + chat in better positions -3) complete translations, stylesheets, variants rules ... ---> publish -Later: implement analyze from position, multiple connections per SID ("double" games, simul games...) - (to allow problems links in forum) +styling: connection indicator, names, put movesList + chat in better positions +complete translations, stylesheets, variants rules ... diff --git a/client/src/utils/gameStorage.js b/client/src/utils/gameStorage.js index 6b446321..668a2306 100644 --- a/client/src/utils/gameStorage.js +++ b/client/src/utils/gameStorage.js @@ -82,6 +82,7 @@ export const GameStorage = move: obj.move, //may be undefined... fen: obj.fen, score: obj.score, + drawOffer: obj.drawOffer, } } ); diff --git a/client/src/views/Game.vue b/client/src/views/Game.vue index 56c98d27..2751b56b 100644 --- a/client/src/views/Game.vue +++ b/client/src/views/Game.vue @@ -52,9 +52,9 @@ export default { this.loadGame(); }, "game.clocks": function(newState) { - if (this.game.moves.length < 2) + if (this.game.moves.length < 2 || this.game.score != "*") { - // 1st move not completed yet: freeze time + // 1st move not completed yet, or game over: freeze time this.virtualClocks = newState.map(s => ppt(s)); return; } @@ -261,7 +261,6 @@ export default { } }, offerDraw: function() { - // TODO: also for corr games if (this.drawOffer == "received") { if (!confirm("Accept draw?")) @@ -273,7 +272,11 @@ export default { this.$refs["basegame"].endGame("1/2", "Mutual agreement"); } else if (this.drawOffer == "sent") + { this.drawOffer = ""; + if (this.game.type == "corr") + GameStorage.update(this.gameRef.id, {drawOffer: false}); + } else { if (!confirm("Offer draw?")) @@ -283,6 +286,8 @@ export default { if (p.sid != this.st.user.sid) this.st.conn.send(JSON.stringify({code:"drawoffer", target:p.sid})); }); + if (this.game.type == "corr") + GameStorage.update(this.gameRef.id, {drawOffer: true}); } }, abortGame: function() { @@ -336,22 +341,27 @@ export default { // corr game: needs to compute the clocks + initime // NOTE: clocks in seconds, initime in milliseconds game.clocks = [tc.mainTime, tc.mainTime]; - game.initime = [0, 0]; - const L = game.moves.length; game.moves.sort((m1,m2) => m1.idx - m2.idx); //in case of - if (L >= 3) + if (game.score == "*") //otherwise no need to bother with time { - let addTime = [0, 0]; - for (let i=2; i= 3) { - addTime[i%2] += tc.increment - - (game.moves[i].played - game.moves[i-1].played) / 1000; + let addTime = [0, 0]; + for (let i=2; i= 1) + game.initime[L%2] = game.moves[L-1].played; + if (game.drawOffer) + this.drawOffer = "received"; } - if (L >= 1) - game.initime[L%2] = game.moves[L-1].played; // Now that we used idx and played, re-format moves as for live games game.moves = game.moves.map( (m) => { const s = m.squares; @@ -370,15 +380,18 @@ export default { if (gtype == "live" && game.clocks[0] < 0) //game unstarted { game.clocks = [tc.mainTime, tc.mainTime]; - game.initime[0] = Date.now(); - if (myIdx >= 0) + if (game.score == "*") { - // I play in this live game; corr games don't have clocks+initime - GameStorage.update(game.id, + game.initime[0] = Date.now(); + if (myIdx >= 0) { - clocks: game.clocks, - initime: game.initime, - }); + // I play in this live game; corr games don't have clocks+initime + GameStorage.update(game.id, + { + clocks: game.clocks, + initime: game.initime, + }); + } } } this.game = Object.assign({}, diff --git a/server/db/create.sql b/server/db/create.sql index 9a8f785c..82c8659d 100644 --- a/server/db/create.sql +++ b/server/db/create.sql @@ -41,6 +41,7 @@ create table Games ( score varchar, timeControl varchar, created datetime, --used only for DB cleaning + drawOffer boolean, foreign key (vid) references Variants(id) ); diff --git a/server/models/Game.js b/server/models/Game.js index bc4ebdca..fa4aea02 100644 --- a/server/models/Game.js +++ b/server/models/Game.js @@ -29,9 +29,10 @@ const GameModel = { db.serialize(function() { let query = - "INSERT INTO Games (vid, fenStart, fen, score, timeControl, created)" + "INSERT INTO Games" + + " (vid, fenStart, fen, score, timeControl, created, drawOffer)" + " VALUES (" + vid + ",'" + fen + "','" + fen + "','*','" - + timeControl + "'," + Date.now() + ")"; + + timeControl + "'," + Date.now() + "," + false + ")"; db.run(query, function(err) { if (!!err) return cb(err); @@ -133,13 +134,15 @@ const GameModel = }); }, - // obj can have fields move, fen and/or score + // obj can have fields move, fen, drawOffer and/or score update: function(id, obj) { db.parallelize(function() { let query = "UPDATE Games " + "SET "; + if (!!obj.drawOffer) + query += "drawOffer = " + obj.drawOffer + ","; if (!!obj.fen) query += "fen = '" + obj.fen + "',"; if (!!obj.score) -- 2.44.0