Fixed: now at every action is sent all playersList

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvent.java

@@ -8,6 +8,7 @@ import it.polimi.ingsw.gc14.Model.Player;
 import it.polimi.ingsw.gc14.Model.Slot;
 
 import java.io.Serializable;
+import java.util.List;
 import java.util.Map;
 
 /**
@@ -55,9 +56,9 @@ public abstract class NetworkEvent implements Serializable {
     protected CurrentState currentState;
 
     /**
-     * Player directly involved in the event, when required.
+     * List of all players.
      */
-    protected Player player;
+    protected List<Player> playerList;
 
     /**
      * Sets the game data associated with this network event.
@@ -65,16 +66,16 @@ public abstract class NetworkEvent implements Serializable {
      * @param slotPlayerMap the map associating each slot with the player occupying it.
      * @param orderLogicCard the order logic card used to manage turn order.
      * @param currentState the current state of the game.
-     * @param player the player directly associated with the event.
+     * @param playerList the list of the actual state of the players
      */
     public void setData(Map<Slot, Player> slotPlayerMap,
                         OrderLogicCard orderLogicCard,
                         CurrentState currentState,
-                        Player player) {
+                        List<Player> playerList) {
         this.slotPlayerMap = slotPlayerMap;
         this.orderLogicCard = orderLogicCard;
         this.currentState = currentState;
-        this.player = player;
+        this.playerList = playerList;
     }
 
     /**

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/AddPlayer.java

@@ -48,7 +48,7 @@ public class AddPlayer extends NetworkEvent implements  Serializable {
     public boolean apply(MiniModel miniModel){
         if(isError)
             return false;
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/DisconnectedPlayer.java

@@ -37,7 +37,7 @@ public class DisconnectedPlayer extends NetworkEvent implements  Serializable{
             return false;
         if(miniModel!=null)
         {
-            miniModel.setPlayer(player);
+            miniModel.setPlayers(playerList);
             miniModel.setOrderLogicCard(orderLogicCard);
             miniModel.setCurrentState(currentState);
             miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/DrawLowerBuildingCard.java

@@ -43,7 +43,7 @@ public class DrawLowerBuildingCard extends NetworkEvent implements  Serializable
             return false;
 
         miniModel.lowerListBuildingCards.remove(pos);
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/DrawLowerTribeCard.java

@@ -42,7 +42,7 @@ public class DrawLowerTribeCard extends NetworkEvent implements  Serializable{
             return false;
 
         miniModel.lowerListTribeCards.remove(pos);
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/DrawUpperBuildingCard.java

@@ -43,7 +43,7 @@ public class DrawUpperBuildingCard extends NetworkEvent implements  Serializable
             return false;
 
         miniModel.upperListBuildingCards.remove(pos);
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/DrawUpperTribeCard.java

@@ -41,7 +41,7 @@ public class DrawUpperTribeCard extends NetworkEvent implements  Serializable{
         if(isError)
             return false;
         miniModel.upperListTribeCards.remove(pos);
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/ReconnectPlayer.java

@@ -49,7 +49,7 @@ public class ReconnectPlayer extends NetworkEvent implements Serializable {
             return false;
 
         if (miniModel != null) {
-            miniModel.setPlayer(player);
+            miniModel.setPlayers(playerList);
             miniModel.setOrderLogicCard(orderLogicCard);
             miniModel.setCurrentState(currentState);
             miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/SkipTurn.java

@@ -35,7 +35,7 @@ public class SkipTurn extends NetworkEvent implements  Serializable{
     public boolean apply(MiniModel miniModel){
         if(isError)
             return false;
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/SlotChoice.java

@@ -41,7 +41,7 @@ public class SlotChoice extends NetworkEvent implements  Serializable {
     public boolean apply(MiniModel miniModel){
         if(isError)
             return false;
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/Network/NetworkEvents/TotemChoice.java

@@ -72,7 +72,7 @@ public class TotemChoice extends NetworkEvent implements Serializable {
         if (isError)
             return false;
 
-        miniModel.setPlayer(player);
+        miniModel.setPlayers(playerList);
         miniModel.setOrderLogicCard(orderLogicCard);
         miniModel.setCurrentState(currentState);
         miniModel.setSlotPlayerMap(slotPlayerMap);

src/main/java/it/polimi/ingsw/gc14/ServerLauncher.java

@@ -129,7 +129,7 @@ public class ServerLauncher {
                     {
                         playerList.remove(event.getUsername());
                     }else {
-                        event.setData(game.getSlotMap(), game.orderLogicCard, game.getCurrentState(), game.getPlayerByUsername(event.getUsername()));
+                        event.setData(game.getSlotMap(), game.orderLogicCard, game.getCurrentState(), game.getPlayers());
                         if (event.getEventType().equals(EventType.TOTEM_CHOICE)) {
                             ((TotemChoice) event).setAvailableTotems(gameController.getModel().getAvailableTotems());
                         } else if (event.getEventType().equals(EventType.DISCONNECTED_PLAYER) && game.getCurrentState().equals(GameStages.WAITING)) {