From: Benjamin Auder <benjamin.auder@somewhere>
Date: Mon, 2 Dec 2019 09:14:44 +0000 (+0100)
Subject: Attempt to fix countdown for live games. Some issues with GameStorage.add now
X-Git-Url: https://git.auder.net/game/current/%7B%7B%20asset%28%27mixstore/css/static/home.css%27%29%20%7D%7D?a=commitdiff_plain;h=c0b2760655298f07eb403da70307636e57b2813a;p=vchess.git

Attempt to fix countdown for live games. Some issues with GameStorage.add now
---

diff --git a/client/src/utils/gameStorage.js b/client/src/utils/gameStorage.js
index 031f62f5..a8aa6edf 100644
--- a/client/src/utils/gameStorage.js
+++ b/client/src/utils/gameStorage.js
@@ -47,6 +47,7 @@ function dbOperation(callback)
 export const GameStorage =
 {
   // Optional callback to get error status
+  // TODO: this func called from Hall seems to not work now...
   add: function(game, callback)
   {
     dbOperation((db) => {
diff --git a/client/src/views/Game.vue b/client/src/views/Game.vue
index 6cb75f47..52e97c53 100644
--- a/client/src/views/Game.vue
+++ b/client/src/views/Game.vue
@@ -79,11 +79,16 @@ export default {
       }
     },
     "game.clocks": function(newState) {
-      this.virtualClocks = newState.map(s => ppt(s));
       const currentTurn = this.vr.turn;
       const colorIdx = ["w","b"].indexOf(currentTurn);
       let countdown = newState[colorIdx] -
         (Date.now() - this.game.initime[colorIdx])/1000;
+      this.virtualClocks = [0,1].map(i => {
+        const removeTime = i == colorIdx
+          ? (Date.now() - this.game.initime[colorIdx])/1000
+          : 0;
+        return ppt(newState[i] - removeTime);
+      });
       const myTurn = (currentTurn == this.game.mycolor);
       let clockUpdate = setInterval(() => {
         if (countdown <= 0 || this.vr.turn != currentTurn)
@@ -297,11 +302,15 @@ export default {
         const vname = (variantCell.length > 0 ? variantCell[0].name : "");
         if (!game.fen)
           game.fen = game.fenStart; //game wasn't started
-
-
-        // TODO: process rtime, clocks............ game.clocks doesn't exist anymore
-console.log(game);
-
+        const gtype = (game.timeControl.indexOf('d') >= 0 ? "corr" : "live");
+        if (gtype == "corr")
+        {
+          // corr game: needs to compute the clocks + initime
+          //if (game.players[i].rtime < 0) initime = Date.now(), else compute,
+          //also using move.played fields
+          game.clocks = [-1, -1];
+          game.initime = [0, 0];
+        }
         const tc = extractTime(game.timeControl);
         if (game.clocks[0] < 0) //game unstarted
         {
@@ -316,6 +325,7 @@ console.log(game);
           game,
           // NOTE: assign mycolor here, since BaseGame could also bs VS computer
           {
+            type: gtype,
             increment: tc.increment,
             vname: vname,
             mycolor: [undefined,"w","b"][myIdx+1],
diff --git a/client/src/views/Hall.vue b/client/src/views/Hall.vue
index b7526ccd..c22c1e17 100644
--- a/client/src/views/Hall.vue
+++ b/client/src/views/Hall.vue
@@ -561,6 +561,7 @@ export default {
       const game = Object.assign({}, gameInfo, {
         // (other) Game infos: constant
         fenStart: gameInfo.fen,
+        created: Date.now(),
         // Game state (including FEN): will be updated
         moves: [],
         clocks: [-1, -1], //-1 = unstarted
diff --git a/server/db/create.sql b/server/db/create.sql
index 56d4a247..0fb74564 100644
--- a/server/db/create.sql
+++ b/server/db/create.sql
@@ -43,6 +43,7 @@ create table Challenges (
   foreign key (vid) references Variants(id)
 );
 
+-- NOTE: no need for a "created" field, it's deduce from first move playing time
 create table Games (
   id integer primary key,
   vid integer,