Two people cant choose the same animal, even if one is selected before another peson connects. Life is now lost/deducted once at the end of the round instead of one per move (causing multiple lifes lost if in the pit) when lives reach zero players die

Assets/Scripts/Character.cs

@@ -69,7 +69,14 @@ public class Character : MonoBehaviour
 
         characterAnimator = GetComponentInChildren<Animator>();
     }
-
+    private void Update()
+    {
+        if(lives < 1)
+        {
+            this.enabled = false;
+            gameObject.SetActive(false);
+        }
+    }
 
 
     #endregion Unity Functions

Assets/Scripts/GameMode/ColorGameMode/ColorGameMode.cs

@@ -170,15 +170,15 @@ public class ColorGameMode : GameMode
         {
             character.stuck = true;
         }
-
+        Debug.Log("inWater = " + character.inWater + ", inPit = " + character.inPit + ", stuck = " + character.stuck);
+    }
+    protected override void OnPlayerKilled(Character character, ClientData client)
+    {
         if (character.inPit)
         {
             character.lives -= 1;
             character.ClientLink.Lives = character.lives;
         }
-
-        Debug.Log("inWater = " + character.inWater + ", inPit = " + character.inPit + ", stuck = " + character.stuck);
-
     }
 
     private IEnumerator AnimateBlock(Material mat, float time)

Assets/Scripts/GameMode/ColorGameMode/RacetrackGameMode.cs

@@ -187,15 +187,16 @@ public class RacetrackGameMode : GameMode
         {
             character.stuck = true;
         }
+        Debug.Log("inWater = " + character.inWater + ", inPit = " + character.inPit + ", stuck = " + character.stuck);
+    }
 
+    protected override void OnPlayerKilled(Character character, ClientData client)
+    {
         if (character.inPit)
         {
             character.lives -= 1;
             character.ClientLink.Lives = character.lives;
         }
-
-        Debug.Log("inWater = " + character.inWater + ", inPit = " + character.inPit + ", stuck = " + character.stuck);
-
     }
 
     private IEnumerator AnimateBlock(Material mat, float time)

Assets/Scripts/GameMode/GameMode.cs

@@ -67,6 +67,9 @@ public abstract class GameMode : ScriptableObject
     /// <param name="currentBlock">Block which the character finished on</param>
     protected virtual void OnPlayerMoved(Character character, ClientData client, Block currentBlock) { }
 
+
+    protected virtual void OnPlayerKilled(Character character, ClientData client) { }
+
     /// <summary>
     /// Called once after every player has finished one move
     /// </summary>
@@ -150,6 +153,11 @@ public abstract class GameMode : ScriptableObject
         OnPlayerMoved(player.character, player.client, player.character.CurrentBlock);
     }
 
+    public void PlayerKilled(PlayerData player)
+    {
+        OnPlayerKilled(player.character, player.client);
+    }
+
     public void AllPlayersMoved(PlayerData[] allPlayers)
     {
         OnAllPlayersMoved(allPlayers);

Assets/Scripts/Managers/GameManager.cs

@@ -126,6 +126,12 @@ public class GameManager : MonoBehaviour
 
             //Let Gamemode know all players have moved
             gameMode.AllPlayersMoved(playerDataAsArray.ToArray());
+            //decrease lives here!!
+            foreach(PlayerData player in playerDataAsArray)
+            {
+                gameMode.PlayerKilled(player);
+            }
+
             playerDataAsArray.ForEach(p => p.client.SendLives()); //Update the players score
 
             //if Game is over break out of loop

Assets/Scripts/Networking/Client/ClientLoginManager.cs

@@ -47,6 +47,7 @@ namespace Networking.Client
         [Tooltip("Send prefilled Player settings after connecting")]
         private bool IdentifyOnConnect = false;
 
+        ConnectedClients clients;
         #endregion Inspector Fields
 
         #region Events

Assets/Scripts/UI/Client/LoginUIManager.cs

@@ -8,7 +8,7 @@ public class LoginUIManager : MonoBehaviour
 {
     public int conn;
     public int maxPlayers = 2;
-    ConnectedClients clients;
+    public ConnectedClients clients;
     [Header("References")]
     [SerializeField]
     private ClientLoginManager clientManager;
@@ -54,6 +54,15 @@ public class LoginUIManager : MonoBehaviour
     private void Update()
     {
         conn = GetComponent<animalSelection>().count;
+        clients = GetComponent<animalSelection>().Clients;
+        foreach (ClientData cli in clients.AllClients)
+        {
+            if (playerAnimal == cli.characterAnimal)
+            {
+                playerAnimal = null;
+                animalSelected = false;
+            }
+        }
     }
     private void OnEnable()
     {
@@ -78,7 +87,6 @@ public class LoginUIManager : MonoBehaviour
         OnLoginFail();
     }
 
-
     #endregion Unity Functions
 
 
@@ -151,6 +159,7 @@ public class LoginUIManager : MonoBehaviour
         {
             if (colorSelected == true && animalSelected == true && playerName != "")
             {
+
                 clientManager.SendPlayerDetails(playerName, playerColor, playerAnimal);
                 PlayerPrefs.SetString("LastUsedName", playerName);
             }