Move some checks from TicTacToe.add_move to Game.add_move

[empires-server] / lmno.js

diff --git a/lmno.js b/lmno.js
index 18e60efd947349be35218a643ac6ba7a9756971d..baa96f82a5469d66ccd51916005182e4361e0c74 100644 (file)
--- a/lmno.js
+++ b/lmno.js
@@ -305,7 +305,12 @@ 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;