X-Git-Url: https://git.auder.net/?a=blobdiff_plain;f=client%2Fsrc%2Fviews%2FHall.vue;h=e76139a183cc74c91d61380b317952cb72318034;hb=e64c6f67185b45b3b1205069532362c1bf9680db;hp=16ce9bc7544ce62f17d669a9c576f16b42aac05f;hpb=a6bddfc684bd3cc4defd7d49cc15bf59f2d9a9e9;p=vchess.git

diff --git a/client/src/views/Hall.vue b/client/src/views/Hall.vue
index 16ce9bc7..e76139a1 100644
--- a/client/src/views/Hall.vue
+++ b/client/src/views/Hall.vue
@@ -78,9 +78,10 @@ import { NbPlayers } from "@/data/nbPlayers";
 import { checkChallenge } from "@/data/challengeCheck";
 import { ArrayFun } from "@/utils/array";
 import { ajax } from "@/utils/ajax";
-import { getRandString, shuffle } from "@/utils/alea";
+import { getRandString } from "@/utils/alea";
 import GameList from "@/components/GameList.vue";
 import ChallengeList from "@/components/ChallengeList.vue";
+import { GameStorage } from "@/utils/storage";
 export default {
   name: "my-hall",
   components: {
@@ -100,7 +101,7 @@ export default {
         fen: "",
         vid: 0,
         nbPlayers: 0,
-        to: ["", "", ""], //name of challenged players
+        to: ["", "", ""], //name(s) of challenged player(s)
         timeControl: "", //"2m+2s" ...etc
       },
     };
@@ -124,23 +125,30 @@ export default {
   created: function() {
     // Always add myself to players' list
     this.players.push(this.st.user);
+    if (this.st.user.id > 0)
+    {
     // Ask server for current corr games (all but mines)
 //    ajax(
-//      "",
+//      "/games",
 //      "GET",
+//      {excluded: this.st.user.id},
 //      response => {
-//
+//        this.games = this.games.concat(response.games);
 //      }
 //    );
-//    // Also ask for corr challenges (all)
-//    ajax(
-//      "",
-//      "GET",
-//      response => {
-//
-//      }
-//    );
-    // 0.1] Ask server for for room composition:
+    // Also ask for corr challenges (open + sent to me)
+      ajax(
+        "/challenges",
+        "GET",
+        {uid: this.st.user.id},
+        response => {
+          console.log(response.challenges);
+          // TODO: post-treatment on challenges ?
+          this.challenges = this.challenges.concat(response.challenges);
+        }
+      );
+    }
+    // 0.1] Ask server for room composition:
     const socketOpenListener = () => {
       this.st.conn.send(JSON.stringify({code:"pollclients"}));
     };
@@ -164,9 +172,9 @@ export default {
     filterGames: function(type) {
       return this.games.filter(c => c.type == type);
     },
-    classifyChallenge: function(c) {
+    classifyObject: function(o) { //challenge or game
       // Heuristic: should work for most cases... (TODO)
-      return (c.timeControl.indexOf('d') === -1 ? "live" : "corr");
+      return (o.timeControl.indexOf('d') === -1 ? "live" : "corr");
     },
     possibleNbplayers: function(nbp) {
       if (this.newchallenge.vid == 0)
@@ -175,11 +183,16 @@ export default {
         this.st.variants.findIndex(v => v.id == this.newchallenge.vid);
       return NbPlayers[this.st.variants[idxInVariants].name].includes(nbp);
     },
-    showGame: function(game) {
-      // NOTE: if we are an observer, the game will be found in main games list
-      // (sent by connected remote players)
-      // TODO: game path ? /vname/gameId seems better
-      this.$router.push("/" + game.id);
+    showGame: function(g) {
+      // NOTE: we are an observer, since only games I don't play are shown here
+      // ==> Moves sent by connected remote player(s) if live game
+      let url = "/" + g.id;
+      if (g.type == "live")
+      {
+        const sids = g.players.map(p => p.sid).join(",");
+        url += "?sids=" + sids;
+      }
+      this.$router.push(url);
     },
     getVname: function(vid) {
       const vIdx = this.st.variants.findIndex(v => v.id == vid);
@@ -189,6 +202,10 @@ export default {
       const pIdx = this.players.findIndex(pl => pl.name == pname);
       return (pIdx === -1 ? null : this.players[pIdx].sid);
     },
+    getPname: function(sid) {
+      const pIdx = this.players.findIndex(pl => pl.sid == sid);
+      return (pIdx === -1 ? null : this.players[pIdx].name);
+    },
     sendSomethingTo: function(to, code, obj, warnDisconnected) {
       const doSend = (code, obj, sid) => {
         this.st.conn.send(JSON.stringify(Object.assign(
@@ -198,7 +215,7 @@ export default {
           {target: sid}
         )));
       };
-      else if (!!to[0])
+      if (!!to[0])
       {
         to.forEach(pname => {
           // Challenge with targeted players
@@ -236,7 +253,7 @@ export default {
             this.players.push({sid:sid, id:0, name:""});
             // Ask identity, challenges and game(s)
             this.st.conn.send(JSON.stringify({code:"askidentity", target:sid}));
-            this.st.conn.send(JSON.stringify({code:"askchallenges", target:sid}));
+            this.st.conn.send(JSON.stringify({code:"askchallenge", target:sid}));
             this.st.conn.send(JSON.stringify({code:"askgame", target:sid}));
           });
           break;
@@ -262,6 +279,7 @@ export default {
             const myChallenge =
             {
               // Minimal challenge informations: (from not required)
+              id: c.id,
               to: c.to,
               fen: c.fen,
               vid: c.vid,
@@ -274,7 +292,20 @@ export default {
         }
         case "askgame":
         {
-          // TODO: Send my current live game (if any): variant, players, movesCount
+          // Send my current live game (if any)
+          if (!!localStorage["gid"])
+          {
+            const myGame =
+            {
+              // Minimal game informations: (fen+clock not required)
+              id: localStorage["gid"],
+              players: JSON.parse(localStorage["players"]), //array sid+id+name
+              vname: localStorage["vname"],
+              timeControl: localStorage["timeControl"],
+            };
+            this.st.conn.send(JSON.stringify({code:"game",
+              game:myGame, target:data.from}));
+          }
           break;
         }
         case "identity":
@@ -288,7 +319,7 @@ export default {
         {
           // Receive challenge from some player (+sid)
           let newChall = data.chall;
-          newChall.type = this.classifyChallenge(data.chall);
+          newChall.type = this.classifyObject(data.chall);
           const pIdx = this.players.findIndex(p => p.sid == data.from);
           newChall.from = this.players[pIdx]; //may be anonymous
           newChall.added = Date.now();
@@ -299,18 +330,21 @@ export default {
         case "game":
         {
           // Receive game from some player (+sid)
-          // TODO: receive game summary (update, count moves)
-          // (just players names, time control, and ID + player ID)
           // NOTE: it may be correspondance (if newgame while we are connected)
+          let newGame = data.game;
+          newGame.type = this.classifyObject(data.game);
+          newGame.vname = newGame.vname;
+          this.games.push(newGame);
           break;
         }
 // *  - receive "new game": if live, store locally + redirect to game
 // *    If corr: notify "new game has started", give link, but do not redirect
         case "newgame":
         {
-          // TODO: new game just started: data contain all informations
-          // (id, players, time control, fenStart ...)
-          // + cid to remove challenge from list
+          // Delete corresponding challenge:
+          ArrayFun.remove(this.challenges, c => c.id == data.cid);
+          // New game just started: data contain all informations
+          this.newGame(data.gameInfo);
           break;
         }
 // *  - receive "accept/withdraw/cancel challenge": apply action to challenges list
@@ -318,7 +352,6 @@ export default {
         case "acceptchallenge":
         {
           // Someone accept an open (or targeted) challenge
-          // TODO: keep SIDs, since we need them to notify newgame after chall is complete
           const cIdx = this.challenges.findIndex(c => c.id == data.cid);
           let c = this.challenges[cIdx];
           if (!c.seats)
@@ -351,8 +384,8 @@ export default {
         }
         case "refusechallenge":
         {
-          // TODO: show "player XXX refused challenge", and
-          // remove challenge from list.
+          alert(this.getPname(data.from) + " refused your challenge");
+          ArrayFun.remove(this.challenges, c => c.id == data.cid);
           break;
         }
         case "deletechallenge":
@@ -360,28 +393,23 @@ export default {
           ArrayFun.remove(this.challenges, c => c.id == data.cid);
           break;
         }
-        // TODO: distinguish hallConnect and gameConnect?
-        // Or (better) global variable players + game variable: "observers"
         case "connect":
-// *  - receive "player connect": send our current challenge (to him or global)
-// *    Also send all our games (live - max 1 - and corr) [in web worker ?]
         {
           this.players.push({name:"", id:0, sid:data.sid});
           this.st.conn.send(JSON.stringify({code:"askidentity", target:data.sid}));
+          this.st.conn.send(JSON.stringify({code:"askchallenge", target:data.sid}));
+          this.st.conn.send(JSON.stringify({code:"askgame", target:data.sid}));
           break;
         }
-// *  - receive "player disconnect": remove from players list
         case "disconnect":
         {
           ArrayFun.remove(this.players, p => p.sid == data.sid);
           // Also remove all challenges sent by this player:
-          for (let cIdx = this.challenges.length-1; cIdx >= 0; cIdx--)
-          {
-            if (this.challenges[cIdx].from.sid == data.sid)
-              this.challenges.splice(cIdx, 1);
-          }
-          // and all live games where he plays and no other opponent is online
-          // TODO
+          ArrayFun.remove(this.challenges, c => c.from.sid == data.sid);
+          // And all live games where he plays and no other opponent is online
+          ArrayFun.remove(this.games, g =>
+            g.type == "live" && (g.players.every(p => p.sid == data.sid
+              || !this.players.some(pl => pl.sid == p.sid))), "all");
           break;
         }
       }
@@ -400,8 +428,8 @@ export default {
       const error = checkChallenge(this.newchallenge);
       if (!!error)
         return alert(error);
-      const ctype = this.classifyChallenge(this.newchallenge);
-      const cto = this.newchallenge.to.slice(0, this.newchallenge.nbPlayers);
+      const ctype = this.classifyObject(this.newchallenge);
+      const cto = this.newchallenge.to.slice(0, this.newchallenge.nbPlayers - 1);
       // NOTE: "from" information is not required here
       let chall =
       {
@@ -410,10 +438,10 @@ export default {
         timeControl: this.newchallenge.timeControl,
         vid: this.newchallenge.vid,
       };
-      const finishAddChallenge = (cid) => {
+      const finishAddChallenge = (cid,warnDisconnected) => {
         chall.id = cid || "c" + getRandString();
-        // Send challenge to peers
-        this.sendSomethingTo(cto, "challenge", {chall:chall}, "warnDisconnected");
+        // Send challenge to peers (if connected)
+        this.sendSomethingTo(cto, "challenge", {chall:chall}, !!warnDisconnected);
         chall.added = Date.now();
         chall.type = ctype;
         chall.vname = vname;
@@ -441,7 +469,7 @@ export default {
       if (ctype == "live")
       {
         // Live challenges have a random ID
-        finishAddChallenge();
+        finishAddChallenge(null, "warnDisconnected");
       }
       else
       {
@@ -462,18 +490,38 @@ export default {
 // *  - prepare and start new game (if challenge is full after acceptation)
 // *    --> include challenge ID (so that opponents can delete the challenge too)
     clickChallenge: function(c) {
-      // TODO: also correspondance case (send to server)
+
+      console.log("click challenge");
+      console.log(c);
+
       if (!!c.accepted)
       {
-        // It's a multiplayer challenge I accepted: withdraw
         this.st.conn.send(JSON.stringify({code: "withdrawchallenge",
           cid: c.id, target: c.from.sid}));
+        if (c.type == "corr")
+        {
+          ajax(
+            "/challenges",
+            "PUT",
+            {action:"withdraw", id: this.challenges[cIdx].id}
+          );
+        }
         c.accepted = false;
       }
-      else if (c.from.sid == this.st.user.sid) //it's my challenge: cancel it
+      else if (c.from.sid == this.st.user.sid
+        || (this.st.user.id > 0 && c.from.id == this.st.user.id))
       {
+        // It's my challenge: cancel it
         this.sendSomethingTo(c.to, "deletechallenge", {cid:c.id});
         ArrayFun.remove(this.challenges, ch => ch.id == c.id);
+        if (c.type == "corr")
+        {
+          ajax(
+            "/challenges",
+            "DELETE",
+            {id: this.challenges[cIdx].id}
+          );
+        }
       }
       else //accept (or refuse) a challenge
       {
@@ -486,46 +534,65 @@ export default {
         this.st.conn.send(JSON.stringify({
           code: (c.accepted ? "accept" : "refuse") + "challenge",
           cid: c.id, target: c.from.sid}));
+        if (c.type == "corr" && c.accepted)
+        {
+          ajax(
+            "/challenges",
+            "PUT",
+            {action: "accept", id: this.challenges[cIdx].id}
+          );
+        }
         if (!c.accepted)
+        {
           ArrayFun.remove(this.challenges, ch => ch.id == c.id);
+          if (c.type == "corr")
+          {
+            ajax(
+              "/challenges",
+              "DELETE",
+              {id: this.challenges[cIdx].id}
+            );
+          }
+        }
       }
     },
-    launchGame: function(c) {
+    // NOTE: for live games only (corr games are launched on server)
+    launchGame: async function(c) {
       // Just assign colors and pass the message
       const vname = this.getVname(c.vid);
       const vModule = await import("@/variants/" + vname + ".js");
       window.V = vModule.VariantRules;
       let players = [c.from];
       Array.prototype.push.apply(players, c.seats);
-      c.type == corr alors use id...sinon sid (figés)
       let gameInfo =
       {
-        cid: c.id, //required to remove challenge
         fen: c.fen || V.GenRandInitFen(),
-        // Shuffle players order (white then black then other colors).
-        // Players' names are not required
-        players: shuffle(players).map(p => {id:p.id, sid:p.sid},
+        // Players' names may be required if game start when a player is offline
+        players: players.map(p => { return {name:p.name, sid:p.sid} }),
         vid: c.vid,
         timeControl: c.timeControl,
       };
       c.seats.forEach(s => {
+        // NOTE: cid required to remove challenge
         this.st.conn.send(JSON.stringify({code:"newgame",
-          gameInfo:gameInfo, target:s.sid}));
+          gameInfo:gameInfo, cid:c.id, target:s.sid}));
       });
+      // Delete corresponding challenge:
+      ArrayFun.remove(this.challenges, ch => ch.id == c.id);
       this.newGame(gameInfo); //also!
     },
+    // NOTE: for live games only (corr games are launched on server)
     newGame: function(gameInfo) {
-      // Extract times (in [milli?]seconds), set clocks,
-      // store in localStorage if live (on server otherwise)
-//      const fen = chall.fen || V.GenRandInitFen();
-//      const game = {}; //TODO: fen, players, time ...
-//      //setStorage(game); //TODO
-//      game.players.forEach(p => { //...even if game is by corr (could be played live, why not...)
-//        this.conn.send(
-//          JSON.stringify({code:"newgame", oppid:p.id, game:game}));
-//      });
-//      if (this.settings.sound >= 1)
-//        new Audio("/sounds/newgame.mp3").play().catch(err => {});
+      GameStorage.init({
+        gameId: getRandString(),
+        vname: this.getVname(gameInfo.vid),
+        fenStart: gameInfo.fen,
+        players: gameInfo.players,
+        timeControl: gameInfo.timeControl,
+      });
+      if (this.st.settings.sound >= 1)
+        new Audio("/sounds/newgame.mp3").play().catch(err => {});
+      // TODO: redirect to game
     },
   },
 };