From: Carl Worth <cworth@cworth.org>
Date: Fri, 5 Jun 2020 20:54:28 +0000 (-0700)
Subject: Rename "next_player" property to "team_to_play"
X-Git-Url: https://git.cworth.org/git?p=lmno-server;a=commitdiff_plain;h=fbb338a161c79e0f889a30d8f08058c16583944b

Rename "next_player" property to "team_to_play"

The game object already has two distinct notions of players and
teams. And this property will only be compared to team values, not
player values, so this name is more accurate and less confusing.
---

diff --git a/tictactoe.js b/tictactoe.js
index f919843..256afec 100644
--- a/tictactoe.js
+++ b/tictactoe.js
@@ -7,7 +7,7 @@ class TicTacToe extends Game {
     this.state = {
       moves: [],
       board: Array(9).fill(""),
-      next_player: "X",
+      team_to_play: "X",
     };
     this.teams = ["X", "O"];
   }
@@ -23,7 +23,7 @@ class TicTacToe extends Game {
     }
 
     /* Cannot move if it's not this player's team's turn. */
-    if (player.team !== this.state.next_player)
+    if (player.team !== this.state.team_to_play)
     {
       return { legal: false,
                message: "It's not your turn to move" };
@@ -36,13 +36,13 @@ class TicTacToe extends Game {
                message: "Square is already occupied" };
     }
 
-    this.state.board[square] = this.state.next_player;
+    this.state.board[square] = this.state.team_to_play;
     this.state.moves.push(square);
 
-    if (this.state.next_player === "X")
-      this.state.next_player = "O";
+    if (this.state.team_to_play === "X")
+      this.state.team_to_play = "O";
     else
-      this.state.next_player = "X";
+      this.state.team_to_play = "X";
 
     return { legal: true };
   }