12542 - Mitigate race condition when selecting player sides

vassal-app/src/main/java/VASSAL/build/module/PlayerRoster.java

@@ -759,50 +759,59 @@ public List<String> getAvailableSides() {
   protected String promptForSide() {
     final ArrayList<String> availableSides = new ArrayList<>(sides);
     final ArrayList<String> alreadyTaken = new ArrayList<>();
+    String newSide = "";
 
-    for (final PlayerInfo p : players) {
-      alreadyTaken.add(p.side);
-    }
+    while (newSide != null) { // Loops until a valid side is found or op is canceled (repeats side check to minimuse race condition window)
 
-    availableSides.removeAll(alreadyTaken);
+      for (final PlayerInfo p : players) {
+        alreadyTaken.add(p.side);
+      }
 
-    // If a "real" player side is available, we want to offer "the next one" as the default, rather than observer.
-    // Thus hotseat players can easily cycle through the player positions as they will appear successively as the default.
-    // Common names for Solitaire players (Solitaire, Solo, Referee) do not count as "real" player sides, and will be skipped.
-    // If we have no "next" side available to offer, we stay with the observer side as our default offering.
-    boolean found = false;       // If we find a usable side
-    final String mySide = getMySide(); // Get our own side, so we can find the "next" one
-    final int myidx = (mySide != null) ? sides.indexOf(mySide) : -1; // See if we have a current non-observe side.
-    int i = (myidx >= 0) ? ((myidx + 1) % sides.size()) : 0;   // If we do, start looking in the "next" slot, otherwise start at beginning.
-    for (int tries = 0; i != myidx && tries < sides.size(); i = (i + 1) % sides.size(), tries++) { // Wrap-around search of sides
-      final String s = sides.get(i);
-      if (!alreadyTaken.contains(s) && !isSoloSide(s)) {
-        found = true; // Found an available slot that's not our current one and not a "solo" slot.
+      if (!newSide.isEmpty() && !alreadyTaken.contains(newSide)) {
+        // found side is still available
+        // sides must always be stored internally in English.
+        newSide = untranslateSide(newSide);
         break;
       }
-    }
 
-    final String nextChoice = found ? sides.get(i) : translatedObserver; // This will be our defaulted choice for the dropdown.
+      availableSides.removeAll(alreadyTaken);
+
+      // If a "real" player side is available, we want to offer "the next one" as the default, rather than observer.
+      // Thus hotseat players can easily cycle through the player positions as they will appear successively as the default.
+      // Common names for Solitaire players (Solitaire, Solo, Referee) do not count as "real" player sides, and will be skipped.
+      // If we have no "next" side available to offer, we stay with the observer side as our default offering.
+      final String mySide = getMySide(); // Get our own side, so we can find the "next" one
+      final int myidx = (mySide != null) ? sides.indexOf(mySide) : -1; // See if we have a current non-observe side.
+      boolean found = false;       // Set when we find a usable side
+      int i = (myidx >= 0) ? ((myidx + 1) % sides.size()) : 0;   // If we do, start looking in the "next" slot, otherwise start at beginning.
+      for (int tries = 0; i != myidx && tries < sides.size(); i = (i + 1) % sides.size(), tries++) { // Wrap-around search of sides
+        final String s = sides.get(i);
+        if (!alreadyTaken.contains(s) && !isSoloSide(s)) {
+          found = true; // Found an available slot that's not our current one and not a "solo" slot.
+          break;
+        }
+      }
 
-    availableSides.add(0, translatedObserver);
+      final String nextChoice = found ? sides.get(i) : translatedObserver; // This will be our defaulted choice for the dropdown.
 
-    final GameModule g = GameModule.getGameModule();
-    String newSide = (String) JOptionPane.showInputDialog(
-      g.getPlayerWindow(),
-      Resources.getString("PlayerRoster.switch_sides", getMyLocalizedSide()), //$NON-NLS-1$
-      Resources.getString("PlayerRoster.choose_side"), //$NON-NLS-1$
-      JOptionPane.QUESTION_MESSAGE,
-      null,
-      availableSides.toArray(new String[0]),
-      nextChoice // Offer calculated most likely "next side" as the default
-    );
+      availableSides.add(0, translatedObserver);
 
-    // sides must always be stored internally in English.
-    if (translatedObserver.equals(newSide)) {
-      newSide = OBSERVER;
-    }
-    else {
-      newSide = untranslateSide(newSide);
+      final GameModule g = GameModule.getGameModule();
+
+      newSide = (String) JOptionPane.showInputDialog(
+              g.getPlayerWindow(),
+              newSide.isEmpty() ? Resources.getString("PlayerRoster.switch_sides", getMyLocalizedSide()) : Resources.getString("PlayerRoster.switch_sides2", newSide, getMyLocalizedSide()), //$NON-NLS-1$
+              Resources.getString("PlayerRoster.choose_side"), //$NON-NLS-1$
+              JOptionPane.QUESTION_MESSAGE,
+              null,
+              availableSides.toArray(new String[0]),
+              nextChoice // Offer calculated most likely "next side" as the default
+      );
+
+      if (translatedObserver.equals(newSide)) { // Observer returns here, other returns are checked once more.
+        newSide = OBSERVER;
+        break;
+      }
     }
     return newSide;
   }

vassal-app/src/main/resources/VASSAL/i18n/VASSAL.properties

@@ -1025,6 +1025,7 @@ PlayerRoster.join_another_side=Join another side
 PlayerRoster.give_up_position=Give up your position as %1$s?
 PlayerRoster.join_game_as=Join game as which side?  
 PlayerRoster.switch_sides=Your current side is %1$s. Switch to which side?
+PlayerRoster.switch_sides2=%1$s is now unavailable. Your current side is %2$s. Switch to which side?
 PlayerRoster.choose_side=Choose side
 PlayerRoster.observer=<observer>
 PlayerRoster.solitaire=Solitaire

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_da.properties

@@ -1023,6 +1023,7 @@ PlayerRoster.join_another_side=Skift til en anden side
 PlayerRoster.give_up_position=Vil du opgive din position som %1$s?
 PlayerRoster.join_game_as=Join spillet på hvilken side?
 PlayerRoster.switch_sides=Din nuværende side er %1$s. Skift til hvilken side?
+PlayerRoster.switch_sides2=%1$s er nu ikke tilgængelig. Din nuværende side er %2$s. Skift til hvilken side?
 PlayerRoster.choose_side=Vælg side
 PlayerRoster.observer=<tilskuer>
 PlayerRoster.solitaire=Solitaire

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_de.properties

@@ -547,6 +547,8 @@ PlayerRoster.become_observer=Werde zum Beobachter
 PlayerRoster.join_another_side=Wechsle zu anderer Seite
 PlayerRoster.give_up_position=Nicht mehr weiterspielen als %1$s?
 PlayerRoster.join_game_as=Spiele auf welcher Seite?
+PlayerRoster.switch_sides=Ihre aktuelle Seite ist %1$s. Auf welche Seite wechseln?
+PlayerRoster.switch_sides2=%1$s ist jetzt nicht verf\u00FCgbar. Ihre aktuelle Seite ist %2$s. Auf welche Seite wechseln?
 PlayerRoster.choose_side=Wähle Seite
 PlayerRoster.observer=<Beobachter>
 

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_es.properties

@@ -704,6 +704,7 @@ PlayerRoster.join_another_side=Unirse a otro bando
 PlayerRoster.give_up_position=¿Dejar su lugar como %1$s?
 PlayerRoster.join_game_as=Unirse a la partida como... 
 PlayerRoster.switch_sides=Tu bando actual es %1$s. ¿Cambiar de bando?
+PlayerRoster.switch_sides2=%1$s ahora no est� disponible. Tu bando actual es %2$s. ¿Cambiar de bando?
 PlayerRoster.choose_side=Seleccione su bando
 PlayerRoster.observer=<Observador>
 PlayerRoster.solitaire=Solitario

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_fr.properties

@@ -782,6 +782,7 @@ PlayerRoster.join_another_side=Changer de camp
 PlayerRoster.give_up_position=Abandonner sa position comme %1$s
 PlayerRoster.join_game_as=Rejoindre la partie en prenant quel camp ?
 PlayerRoster.switch_sides=Votre camp actuel est %1$s. Basculer vers quel camp?
+PlayerRoster.switch_sides2=%1$s n?est pas disponible. Votre camp actuel est %2$s. Basculer vers quel camp?
 PlayerRoster.choose_side=Choisissez votre camp
 PlayerRoster.observer=<Observateur>
 PlayerRoster.solitaire=Solitaire

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_it.properties

@@ -601,6 +601,8 @@ PlayerRoster.become_observer=Diventa osservatore
 PlayerRoster.join_another_side=Cambia fazione
 PlayerRoster.give_up_position=Abbandoni la fazione di %1$s?
 PlayerRoster.join_game_as=Scelta fazione  
+PlayerRoster.switch_sides=Il tuo lato attuale � %1$s. Passare da che parte?
+PlayerRoster.switch_sides2=%1$s ora non � disponibile. Il tuo lato attuale � %2$s. Passare da che parte?
 PlayerRoster.choose_side=Scegli la fazione
 PlayerRoster.observer=<osservatore>
 PlayerRoster.referee=Arbitro

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_ja.properties

@@ -522,6 +522,8 @@ PlayerRoster.become_observer=見学者になる
 PlayerRoster.join_another_side=別な陣営で参加
 PlayerRoster.give_up_position=%1$s の立場を放棄しますか？
 PlayerRoster.join_game_as=どの陣営で参加しますか？
+PlayerRoster.switch_sides=Your current side is %1$s. Switch to which side?
+PlayerRoster.switch_sides2=%1$s is now unavailable. Your current side is %2$s. Switch to which side?
 PlayerRoster.choose_side=陣営を選んで下さい
 PlayerRoster.observer=<見学者>
 

vassal-app/src/main/resources/VASSAL/i18n/VASSAL_nl.properties

@@ -527,6 +527,8 @@ PlayerRoster.become_observer=Wordt kijker
 PlayerRoster.join_another_side=Kies een andere kant
 PlayerRoster.give_up_position=Geef uw positie op als %1$s?
 PlayerRoster.join_game_as=Doe mee aan welke kant?  
+PlayerRoster.switch_sides=Je huidige kant is %1$s. Naar welke kant overstappen?
+PlayerRoster.switch_sides2=%1$s is nu niet beschikbaar. Je huidige kant is %2$s. Naar welke kant overstappen?
 PlayerRoster.choose_side=Kies kant
 PlayerRoster.observer=<kijker>