chore: whitespace fixes

Author: FabianStoll

Assets/Scripts/ConnectionManagement/ConnectionMethod.cs

@@ -9,8 +9,9 @@
 namespace Unity.BossRoom.ConnectionManagement
 {
     /// <summary>
-    /// ConnectionMethod contains all setup needed to setup NGO to be ready to start a connection, either host or client side.
-    /// Please override this abstract class to add a new transport or way of connecting.
+    ///     ConnectionMethod contains all setup needed to setup NGO to be ready to start a connection, either host or client
+    ///     side.
+    ///     Please override this abstract class to add a new transport or way of connecting.
     /// </summary>
     public abstract class ConnectionMethodBase
     {
@@ -19,23 +20,23 @@ public abstract class ConnectionMethodBase
         protected readonly string m_PlayerName;
 
         /// <summary>
-        /// Setup the host connection prior to starting the NetworkManager
+        ///     Setup the host connection prior to starting the NetworkManager
         /// </summary>
         /// <returns></returns>
         public abstract Task SetupHostConnectionAsync();
-        
+
         /// <summary>
-        /// Setup the client connection prior to starting the NetworkManager
+        ///     Setup the client connection prior to starting the NetworkManager
         /// </summary>
         /// <returns></returns>
         public abstract void SetupClientConnectionAsync();
 
         /// <summary>
-        /// Setup the client for reconnection prior to reconnecting
+        ///     Setup the client for reconnection prior to reconnecting
         /// </summary>
         /// <returns>
-        /// success = true if succeeded in setting up reconnection, false if failed.
-        /// shouldTryAgain = true if we should try again after failing, false if not.
+        ///     success = true if succeeded in setting up reconnection, false if failed.
+        ///     shouldTryAgain = true if we should try again after failing, false if not.
         /// </returns>
         public abstract Task<(bool success, bool shouldTryAgain)> SetupClientReconnectionAsync();
 
@@ -79,7 +80,7 @@ protected string GetPlayerId()
     }
 
     /// <summary>
-    /// Simple IP connection setup with UTP
+    ///     Simple IP connection setup with UTP
     /// </summary>
     class ConnectionMethodIP : ConnectionMethodBase
     {
@@ -116,7 +117,7 @@ public override async Task SetupHostConnectionAsync()
     }
 
     /// <summary>
-    /// UTP's Relay connection setup using the Session integration
+    ///     UTP's Relay connection setup using the Session integration
     /// </summary>
     class ConnectionMethodRelay : ConnectionMethodBase
     {

Assets/Scripts/ConnectionManagement/ConnectionState/ConnectionState.cs

@@ -7,7 +7,7 @@
 namespace Unity.BossRoom.ConnectionManagement
 {
     /// <summary>
-    /// Base class representing a connection state.
+    ///     Base class representing a connection state.
     /// </summary>
     abstract class ConnectionState
     {
@@ -27,11 +27,11 @@ public virtual void OnClientDisconnect(ulong clientId) { }
         public virtual void OnServerStarted() { }
 
         public virtual void StartClientIP(string playerName, string ipaddress, int port) { }
-        
+
         public virtual void StartClientSession(string playerName) { }
 
         public virtual void StartHostIP(string playerName, string ipaddress, int port) { }
-        
+
         public virtual void StartHostSession(string playerName) { }
 
         public virtual void OnUserRequestedShutdown() { }

Assets/Scripts/ConnectionManagement/ConnectionState/OfflineState.cs

@@ -10,8 +10,8 @@
 namespace UUnity.BossRoom.ConnectionManagement
 {
     /// <summary>
-    /// Connection state corresponding to when the NetworkManager is shut down. From this state we can transition to the
-    /// ClientConnecting sate, if starting as a client, or the StartingHost state, if starting as a host.
+    ///     Connection state corresponding to when the NetworkManager is shut down. From this state we can transition to the
+    ///     ClientConnecting sate, if starting as a client, or the StartingHost state, if starting as a host.
     /// </summary>
     class OfflineState : ConnectionState
     {
@@ -35,7 +35,7 @@ public override void Enter()
         }
 
         public override void Exit() { }
-        
+
         public override void OnClientConnected(ulong _)
         {
             m_ConnectStatusPublisher.Publish(ConnectStatus.Success);

Assets/Scripts/ConnectionManagement/ConnectionState/StartingHostState.cs

@@ -9,8 +9,8 @@
 namespace Unity.BossRoom.ConnectionManagement
 {
     /// <summary>
-    /// Connection state corresponding to a host starting up. Starts the host when entering the state. If successful,
-    /// transitions to the Hosting state, if not, transitions back to the Offline state.
+    ///     Connection state corresponding to a host starting up. Starts the host when entering the state. If successful,
+    ///     transitions to the Hosting state, if not, transitions back to the Offline state.
     /// </summary>
     class StartingHostState : OnlineState
     {
@@ -43,6 +43,7 @@ public override void ApprovalCheck(NetworkManager.ConnectionApprovalRequest requ
         {
             var connectionData = request.Payload;
             var clientId = request.ClientNetworkId;
+
             // This happens when starting as a host, before the end of the StartHost call. In that case, we simply approve ourselves.
             if (clientId == m_ConnectionManager.NetworkManager.LocalClientId)
             {
@@ -75,7 +76,7 @@ async void StartHost()
                     if (!m_ConnectionManager.NetworkManager.StartHost())
                     {
                         StartHostFailed();
-                    }   
+                    }
                 }
             }
             catch (Exception)

Assets/Scripts/Gameplay/GameState/ClientCharSelectState.cs

@@ -12,20 +12,23 @@
 namespace Unity.BossRoom.Gameplay.GameState
 {
     /// <summary>
-    /// Client specialization of the Character Select game state. Mainly controls the UI during character-select.
+    ///     Client specialization of the Character Select game state. Mainly controls the UI during character-select.
     /// </summary>
     [RequireComponent(typeof(NetcodeHooks))]
     public class ClientCharSelectState : GameStateBehaviour
     {
         /// <summary>
-        /// Reference to the scene's state object so that UI can access state
+        ///     Reference to the scene's state object so that UI can access state
         /// </summary>
         public static ClientCharSelectState Instance { get; private set; }
 
         [SerializeField]
         NetcodeHooks m_NetcodeHooks;
 
-        public override GameState ActiveState { get { return GameState.CharSelect; } }
+        public override GameState ActiveState
+        {
+            get { return GameState.CharSelect; }
+        }
 
         [SerializeField]
         NetworkCharSelection m_NetworkCharSelection;
@@ -49,6 +52,7 @@ public class ColorAndIndicator
             public Sprite Indicator;
             public Color Color;
         }
+
         [Tooltip("Representational information for each player")]
         public ColorAndIndicator[] m_IdentifiersForEachPlayerNumber;
 
@@ -68,7 +72,7 @@ public class ColorAndIndicator
         [SerializeField]
         [Tooltip("UI elements to turn on when the player has locked in their seat choice (and is now waiting for other players to do the same). Turned off otherwise!")]
         List<GameObject> m_UIElementsForSeatChosen;
-        
+
         [SerializeField]
         [Tooltip("UI elements to turn on when the session is closed (and game is about to start). Turned off otherwise!")]
         List<GameObject> m_UIElementsForSessionEnding;
@@ -95,10 +99,10 @@ public class ColorAndIndicator
         Dictionary<Guid, GameObject> m_SpawnedCharacterGraphics = new Dictionary<Guid, GameObject>();
 
         /// <summary>
-        /// Conceptual modes or stages that the session can be in. We don't actually
-        /// bother to keep track of what SessionMode we're in at any given time; it's just
-        /// an abstraction that makes it easier to configure which UI elements should
-        /// be enabled/disabled in each stage of the session.
+        ///     Conceptual modes or stages that the session can be in. We don't actually
+        ///     bother to keep track of what SessionMode we're in at any given time; it's just
+        ///     an abstraction that makes it easier to configure which UI elements should
+        ///     be enabled/disabled in each stage of the session.
         /// </summary>
         enum SessionMode
         {
@@ -175,7 +179,7 @@ void OnNetworkSpawn()
         }
 
         /// <summary>
-        /// Called when our PlayerNumber (e.g. P1, P2, etc.) has been assigned by the server
+        ///     Called when our PlayerNumber (e.g. P1, P2, etc.) has been assigned by the server
         /// </summary>
         /// <param name="playerNum"></param>
         void OnAssignedPlayerNumber(int playerNum)
@@ -191,7 +195,7 @@ void UpdatePlayerCount()
         }
 
         /// <summary>
-        /// Called by the server when any of the seats in the session have changed. (Including ours!)
+        ///     Called by the server when any of the seats in the session have changed. (Including ours!)
         /// </summary>
         void OnSessionPlayerStateChanged(NetworkListEvent<NetworkCharSelection.SessionPlayerState> changeEvent)
         {
@@ -219,6 +223,7 @@ void OnSessionPlayerStateChanged(NetworkListEvent<NetworkCharSelection.SessionPl
             {
                 // we haven't chosen a seat yet (or were kicked out of our seat by someone else)
                 UpdateCharacterSelection(NetworkCharSelection.SeatState.Inactive);
+
                 // make sure our player num is properly set in Session UI
                 OnAssignedPlayerNumber(m_NetworkCharSelection.sessionPlayers[localPlayerIdx].PlayerNumber);
             }
@@ -230,9 +235,9 @@ void OnSessionPlayerStateChanged(NetworkListEvent<NetworkCharSelection.SessionPl
         }
 
         /// <summary>
-        /// Internal utility that sets the character-graphics and class-info box based on
-        /// our chosen seat. It also triggers a SessionMode change when it notices that our seat-state
-        /// is LockedIn.
+        ///     Internal utility that sets the character-graphics and class-info box based on
+        ///     our chosen seat. It also triggers a SessionMode change when it notices that our seat-state
+        ///     is LockedIn.
         /// </summary>
         /// <param name="state">Our current seat state</param>
         /// <param name="seatIdx">Which seat we're sitting in, or -1 if SeatState is Inactive</param>
@@ -271,6 +276,7 @@ void UpdateCharacterSelection(NetworkCharSelection.SeatState state, int seatIdx
                         m_ClassInfoBox.ConfigureForClass(m_NetworkCharSelection.AvatarConfiguration[seatIdx].CharacterClass);
                     }
                 }
+
                 if (state == NetworkCharSelection.SeatState.LockedIn && !m_HasLocalPlayerLockedIn)
                 {
                     // the local player has locked in their seat choice! Rearrange the UI appropriately
@@ -297,7 +303,7 @@ void UpdateCharacterSelection(NetworkCharSelection.SeatState state, int seatIdx
         }
 
         /// <summary>
-        /// Internal utility that sets the graphics for the eight session-seats (based on their current networked state)
+        ///     Internal utility that sets the graphics for the eight session-seats (based on their current networked state)
         /// </summary>
         void UpdateSeats()
         {
@@ -326,7 +332,7 @@ void UpdateSeats()
         }
 
         /// <summary>
-        /// Called by the server when the session closes (because all players are seated and locked in)
+        ///     Called by the server when the session closes (because all players are seated and locked in)
         /// </summary>
         void OnSessionClosedChanged(bool wasSessionClosed, bool isSessionClosed)
         {
@@ -349,9 +355,9 @@ void OnSessionClosedChanged(bool wasSessionClosed, bool isSessionClosed)
         }
 
         /// <summary>
-        /// Turns on the UI elements for a specified "session mode", and turns off UI elements for all other modes.
-        /// It can also disable/enable the session seats and the "Ready" button if they are inappropriate for the
-        /// given mode.
+        ///     Turns on the UI elements for a specified "session mode", and turns off UI elements for all other modes.
+        ///     It can also disable/enable the session seats and the "Ready" button if they are inappropriate for the
+        ///     given mode.
         /// </summary>
         void ConfigureUIForSessionMode(SessionMode mode)
         {
@@ -380,8 +386,10 @@ void ConfigureUIForSessionMode(SessionMode mode)
                         {
                             m_CurrentCharacterGraphics.gameObject.SetActive(false);
                         }
+
                         m_ClassInfoBox.ConfigureForNoSelection();
                     }
+
                     m_ReadyButtonText.text = "READY!";
                     break;
                 case SessionMode.SeatChosen:
@@ -405,11 +413,10 @@ void ConfigureUIForSessionMode(SessionMode mode)
                 // disable interaction if seat is already locked or all seats disabled
                 seat.SetDisableInteraction(seat.IsLocked() || isSeatsDisabledInThisMode);
             }
-
         }
 
         /// <summary>
-        /// Called directly by UI elements!
+        ///     Called directly by UI elements!
         /// </summary>
         /// <param name="seatIdx"></param>
         public void OnPlayerClickedSeat(int seatIdx)
@@ -421,7 +428,7 @@ public void OnPlayerClickedSeat(int seatIdx)
         }
 
         /// <summary>
-        /// Called directly by UI elements!
+        ///     Called directly by UI elements!
         /// </summary>
         public void OnPlayerClickedReady()
         {
@@ -442,6 +449,5 @@ GameObject GetCharacterGraphics(Avatar avatar)
 
             return characterGraphics;
         }
-
     }
 }

Assets/Scripts/Gameplay/UI/Session/SessionListItemUI.cs

@@ -7,17 +7,20 @@
 namespace Unity.BossRoom.Gameplay.UI
 {
     /// <summary>
-    /// An individual Session UI in the list of available Sessions.
+    ///     An individual Session UI in the list of available Sessions.
     /// </summary>
     public class SessionListItemUI : MonoBehaviour
     {
-        [SerializeField] TextMeshProUGUI m_SessionNameText;
-        [SerializeField] TextMeshProUGUI m_SessionCountText;
+        [SerializeField]
+        TextMeshProUGUI m_SessionNameText;
+        [SerializeField]
+        TextMeshProUGUI m_SessionCountText;
 
-        [Inject] SessionUIMediator m_SessionUIMediator;
+        [Inject]
+        SessionUIMediator m_SessionUIMediator;
 
         ISessionInfo m_Data;
-        
+
         public void SetData(ISessionInfo data)
         {
             m_Data = data;