X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=lmno.js;h=c254046ecb7ae7054a7a4f0b617acd96f46c936a;hb=756dc24a56a1da12d01025d7ebdef88ac7ea3471;hp=5d4ca3e7b554025a2b367cf8aca6dae71945ec1b;hpb=16564f558ee42c3fbc0ca5ee56c91407253747b5;p=empires-server

diff --git a/lmno.js b/lmno.js
index 5d4ca3e..c254046 100644
--- a/lmno.js
+++ b/lmno.js
@@ -305,12 +305,24 @@ for (let key in engines) {
 
   /* Further, add some routes conditionally depending on whether the
    * engine provides specific, necessary methods for the routes. */
-  if (engine.prototype.add_move) {
+
+  /* Note: We have to use hasOwnProperty here since the base Game
+   * class has a geeric add_move function, and we don't want that to
+   * have any influence on our decision. Only if the child has
+   * overridden that do we want to create a "/move" route. */
+  if (engine.prototype.hasOwnProperty("add_move")) {
     router.post('/move', (request, response) => {
       const game = request.game;
       const move = request.body.move;
+      const player = game.players_by_session[request.session.id];
+
+      /* Reject move if there is no player for this session. */
+      if (! player) {
+        response.json({legal: false, message: "No valid player from session"});
+        return;
+      }
 
-      const result = game.add_move(move);
+      const result = game.add_move(player, move);
 
       /* Feed move response back to the client. */
       response.json(result);