X-Git-Url: https://git.auder.net/?a=blobdiff_plain;f=server%2Froutes%2Fgames.js;h=3e2122d6d46ef4a7c6151da620031a62bb83a6a9;hb=1ad003fffabb4c95b9fd2d76a2e7ae1f20302cda;hp=b5f59b0c380f6d5b9ad8f9238972128ece5671c7;hpb=945a61650f0e496e1bea84d0b06657cf45cf7edf;p=vchess.git

diff --git a/server/routes/games.js b/server/routes/games.js
index b5f59b0c..3e2122d6 100644
--- a/server/routes/games.js
+++ b/server/routes/games.js
@@ -32,7 +32,7 @@ router.post("/games", access.logged, access.ajax, (req,res) => {
         const oppIdx = (gameInfo.players[0].id == req.userId ? 1 : 0);
         const oppId = gameInfo.players[oppIdx].id;
         UserModel.tryNotify(oppId,
-          "New game: " + params.siteURL + "/game/" + ret.gid);
+          "Game started: " + params.siteURL + "/game/" + ret.gid);
         res.json({gameId: ret.gid});
       });
     }
@@ -77,28 +77,25 @@ router.put("/games", access.logged, access.ajax, (req,res) => {
   error = GameModel.checkGameUpdate(obj);
   if (!!error)
     return res.json({errmsg: error});
-  GameModel.update(gid, obj, (err) => {
-    if (!!err)
-      return res.json(err);
-    if (!!obj.move || !!obj.score)
-    {
-      // Notify opponent if he enabled notifications:
-      GameModel.getPlayers(gid, (err2,players) => {
-        if (!err2)
-        {
-          const oppid = (players[0].id == req.userId
-            ? players[1].id
-            : players[0].id);
-          const messagePrefix = (!!obj.move
-            ? "New move in game: "
-            : "Game ended: ");
-          UserModel.tryNotify(oppid,
-            messagePrefix + params.siteURL + "/game/" + gid);
-        }
-      });
-    }
-    res.json({});
-  });
+  GameModel.update(gid, obj); //no callback here (several operations)
+  if (!!obj.move || !!obj.score)
+  {
+    // Notify opponent if he enabled notifications:
+    GameModel.getPlayers(gid, (err,players) => {
+      if (!err)
+      {
+        const oppid = (players[0].uid == req.userId
+          ? players[1].uid
+          : players[0].uid);
+        const messagePrefix = (!!obj.move
+          ? "New move in game: "
+          : "Game ended: ");
+        UserModel.tryNotify(oppid,
+          messagePrefix + params.siteURL + "/game/" + gid);
+      }
+    });
+  }
+  res.json({}); //TODO: what if some update action fails?
 });
 
 module.exports = router;