Integrate finish() and promptForSide().

vassalengine · Sep 9, 2023 · 4a7add4 · 4a7add4
1 parent e4ab000
commit 4a7add4
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/vassal-app/src/main/java/VASSAL/build/module/PlayerRoster.java b/vassal-app/src/main/java/VASSAL/build/module/PlayerRoster.java
@@ -772,9 +772,12 @@ protected String promptForSide(String newSide) {
     }
     else {
       fromWizard = true;
+      if (newSide == null) {
+        newSide = "";
+      }
     }
 
-    while (newSide != null && !newSide.isEmpty()) { // Loops until a valid side is found or op is canceled (repeats side check to minimuse race condition window)
+    while (newSide != null) { // Loops until a valid side is found or op is canceled (repeats side check to minimuse race condition window)
 
       for (final PlayerInfo p : players) {
         alreadyTaken.add(p.side);