From 485014d28daf30f2309d7da4d9087c5716d4214f Mon Sep 17 00:00:00 2001
From: Martin Berg Alstad <600878@stud.hvl.no>
Date: Mon, 22 May 2023 22:13:21 +0200
Subject: [PATCH] Comments

---
 pac-man-board-game/ClientApp/src/components/gameBoard.tsx     | 4 ++--
 .../ClientApp/src/game/possibleMovesAlgorithm.ts              | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/pac-man-board-game/ClientApp/src/components/gameBoard.tsx b/pac-man-board-game/ClientApp/src/components/gameBoard.tsx
index b087736..2387dab 100644
--- a/pac-man-board-game/ClientApp/src/components/gameBoard.tsx
+++ b/pac-man-board-game/ClientApp/src/components/gameBoard.tsx
@@ -42,7 +42,7 @@ const Board: Component<BoardProps> = (
   const [tileSize, setTileSize] = useState(2);
   const [selectedCharacter, setSelectedCharacter] = useState<Character>();
   // TODO show the paths to the positions when hovering over a possible position (type Path = CharacterPosition[])
-  const [possiblePositions, setPossiblePositions] = useState<Position[]>([]);
+  const [possiblePositions, setPossiblePositions] = useState<Position[]>([]); // TODO reset when other client moves a character
 
   function handleSelectCharacter(character: Character): void {
     setSelectedCharacter(character);
@@ -178,5 +178,5 @@ const CharacterComponent: Component<CharacterComponentProps> = (
   }) => (
   <div className={`rounded-full w-4/5 h-4/5 cursor-pointer hover:border border-black ${className}`}
        style={{backgroundColor: `${character.color}`}}
-       onClick={onClick ? () => onClick(character) : undefined}/>
+       onClick={() => onClick?.(character)}/>
 );
diff --git a/pac-man-board-game/ClientApp/src/game/possibleMovesAlgorithm.ts b/pac-man-board-game/ClientApp/src/game/possibleMovesAlgorithm.ts
index e412107..ef31ed6 100644
--- a/pac-man-board-game/ClientApp/src/game/possibleMovesAlgorithm.ts
+++ b/pac-man-board-game/ClientApp/src/game/possibleMovesAlgorithm.ts
@@ -16,7 +16,7 @@ function findPossibleRecursive(board: number[][], currentPos: Position, steps: n
                                possibleList: Position[], visitedTiles: Position[]): Position | null {
   if (isOutsideBoard(currentPos, board.length)) {
     addTeleportationTiles(board, currentPos, steps, possibleList, visitedTiles);
-  } else if (visitedTiles.find(tile => tile.x === currentPos.x && tile.y === currentPos.y)) {
+  } else if (visitedTiles.find(tile => tile.x === currentPos.x && tile.y === currentPos.y)) { // TODO might be true when teleporting, when it shouldn't (1,5) and 6 steps
     return null;
   } else if (isWall(board, currentPos)) {
     return null;