From 056be018d7ebe5badfdad7a8712b4986707fa748 Mon Sep 17 00:00:00 2001 From: riverwanderer <58135975+riverwanderer@users.noreply.github.com> Date: Wed, 6 Sep 2023 14:45:00 +0100 Subject: [PATCH] Translate current side before checking for "next side". --- vassal-app/src/main/java/VASSAL/build/module/PlayerRoster.java | 2 +- 1 file changed, 1 insertion(+), 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 index 5f5ebeff72..554bab0df4 100644 --- a/vassal-app/src/main/java/VASSAL/build/module/PlayerRoster.java +++ b/vassal-app/src/main/java/VASSAL/build/module/PlayerRoster.java @@ -771,7 +771,7 @@ protected String promptForSide() { // 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 String mySide = translateSide(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