Merge branch 'main' of https://github.com/viastolfi/maettleship into docker-deployment

ftech last features

README.md

@@ -66,6 +66,7 @@ The current state of maettleship is the following one
 - [X] End game (partially handled)
 - [X] Create an account and add it on the database
 - [X] Connection to your account
+- [X] Rematch 
 
 # TODO
 
@@ -73,7 +74,6 @@ Those point are the feature I still need to introduce
 
 - [ ] Security improvement (hashed password, sql injection handling)
 - [ ] Fix all the problems linked to my rooms implementation
-- [ ] Rematch 
 - [ ] Player historic (link to its account)
 - [ ] Global scoreboard
 - [ ] General error handling

index.js

@@ -202,11 +202,19 @@ io.on("connection", (socket) => {
 
     if (room == null) {
       callack({
-        status: false
+        status: false,
+        message: "No room for this code"
       })
       return 
     }
 
+    if (room.players.length >= 2) {
+      callack({
+        status: false,
+        message: "Room is full"
+      })
+    }
+
     callack({
       status: true
     })

public/scripts/app.js

@@ -177,9 +177,7 @@ function onJoinRoom() {
     
     socket.emit("ask for room", roomKey, socket.id, (response) => {
       if (response.status !== true) {
-        if (errorHolder.textContent == "") {
+        errorHolder.textContent = "Error : " + response.message
-          errorHolder.append("Error : Room Id don't exist")
-        }
       } else {
         loader.style.display = "none";
         roomkeyHolder.innerHTML += `Your room key is : <strong>` + roomId + `</strong>`;