esoko/mapguesser
esoko/mapguesser
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 18 Wiki Activity

added error handling when requesting to join non existing room
All checks were successful
default-pipeline default-pipeline #181
Details

Browse Source
This commit is contained in:
Balázs Vigh 2021-05-08 18:03:17 +02:00
parent 430b32d0c6
commit 01a4fdf5e5
2 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
public/static/js/game.js
View File

@ -410,6 +410,10 @@
MapGuesser.showModalWithContent('Error', 'This game is already started, you cannot join.'); MapGuesser.showModalWithContent('Error', 'This game is already started, you cannot join.');
break; break;
case 'game_not_found':
MapGuesser.showModalWithContent('Error', 'The game room was not found by this ID. Please check the link.');
break;
default: default:
MapGuesser.showModalWithContent('Error', 'Error code: \'' + error + '\''); MapGuesser.showModalWithContent('Error', 'Error code: \'' + error + '\'');
break break

5
src/Controller/GameController.php
View File

@ -114,6 +114,11 @@ class GameController
} }
$room = $this->multiRoomRepository->getByRoomId($roomId); $room = $this->multiRoomRepository->getByRoomId($roomId);
if(!isset($room)) {
return new JsonContent(['error' => 'game_not_found']);
}
$state = $room->getStateArray(); $state = $room->getStateArray();
$map = $this->mapRepository->getById($state['mapId']); $map = $this->mapRepository->getById($state['mapId']);
$token = $this->getMultiToken($roomId); $token = $this->getMultiToken($roomId);