Changed enum names to follow upper snake case convention

Nitrox.Test/Model/StatusCodeTests.cs

@@ -29,7 +29,7 @@ public void ThrowStatusCodesNonFatal()
     public void ThrowStatusCodesLongException()
     {
         // Test if the text will wrap or if the messageBox will expand to accomodate for a long exception message
-        DisplayStatusCode(StatusCode.fileSystemErr, false, "This is a testing exception that is super long to test if text will wrap to fit the message box space, or potentially expand the message box as needed. Did you know that you are wasting your time reading this? Like actually there is nothing here you can stop reading, just run the test and you will see this text there. Alright this is probably long enough hopefully it works :) and btw Crabsnake is the best mod.");
+        DisplayStatusCode(StatusCoe.FILE_SYSTEM_ERR, false, "This is a testing exception that is super long to test if text will wrap to fit the message box space, or potentially expand the message box as needed. Did you know that you are wasting your time reading this? Like actually there is nothing here you can stop reading, just run the test and you will see this text there. Alright this is probably long enough hopefully it works :) and btw Crabsnake is the best mod.");
     }
 
 }

NitroxClient/Communication/Exceptions/ClientConnectionFailedException.cs

@@ -6,12 +6,12 @@ public class ClientConnectionFailedException : Exception
     {
         public ClientConnectionFailedException(string message) : base(message)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Client failed to connect to the server and has been disconnected" + message);
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Client failed to connect to the server and has been disconnected" + message);
         }
 
         public ClientConnectionFailedException(string message, Exception innerException) : base(message, innerException)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Client failed to connect to the server and has been disconnected" + message);
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Client failed to connect to the server and has been disconnected" + message);
         }
     }
 }

NitroxClient/Communication/MultiplayerSession/ConnectionState/AwaitingReservationCredentials.cs

@@ -56,7 +56,7 @@ private static void ClientIsConnected(IMultiplayerSessionConnectionContext sessi
         {
             if (!sessionConnectionContext.Client.IsConnected)
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "The client is not connected.");
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The client is not connected.");
             }
         }
 
@@ -68,7 +68,7 @@ private static void PlayerSettingsIsNotNull(IMultiplayerSessionConnectionContext
             }
             catch (ArgumentNullException ex)
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "The context does not contain player settings." + ex);
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The context does not contain player settings." + ex);
             }
         }
 
@@ -80,7 +80,7 @@ private static void AuthenticationContextIsNotNull(IMultiplayerSessionConnection
             }
             catch (ArgumentNullException ex)
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "The context does not contain an authentication context." + ex);
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The context does not contain an authentication context." + ex);
             }
         }
     }

NitroxClient/Communication/MultiplayerSession/ConnectionState/AwaitingSessionReservation.cs

@@ -59,15 +59,15 @@ private static void ReservationIsNotNull(IMultiplayerSessionConnectionContext se
             }
             catch (ArgumentNullException ex)
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "The context does not have a reservation." + ex.ToString());
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The context does not have a reservation." + ex.ToString());
             }
         }
 
         private void ReservationPacketIsCorrelated(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
             if (!reservationCorrelationId.Equals(sessionConnectionContext.Reservation.CorrelationId))
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, sessionConnectionContext.Reservation.ToString() + reservationCorrelationId.ToString());
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, sessionConnectionContext.Reservation.ToString() + reservationCorrelationId.ToString());
                 throw new UncorrelatedPacketException(sessionConnectionContext.Reservation, reservationCorrelationId);
             }
         }

NitroxClient/Communication/MultiplayerSession/ConnectionState/CommunicatingState.cs

@@ -15,7 +15,7 @@ public virtual void Disconnect(IMultiplayerSessionConnectionContext sessionConne
         {
             sessionConnectionContext.ClearSessionState();
             sessionConnectionContext.Client.Stop();
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Client was disconnected.");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Client was disconnected.");
             Disconnected newConnectionState = new Disconnected();
             sessionConnectionContext.UpdateConnectionState(newConnectionState);
         }

NitroxClient/Communication/MultiplayerSession/ConnectionState/ConnectionNegotiatedState.cs

@@ -8,7 +8,7 @@ public abstract class ConnectionNegotiatedState : CommunicatingState
     {
         public override Task NegotiateReservationAsync(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Unable to negotiate a session connection in the current state.");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Unable to negotiate a session connection in the current state.");
             throw new Exception();
         }
     }

NitroxClient/Communication/MultiplayerSession/ConnectionState/ConnectionNegotiatingState.cs

@@ -7,7 +7,7 @@ public abstract class ConnectionNegotiatingState : CommunicatingState
     {
         public override void JoinSession(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Cannot join a session until a reservation has been negotiated with the server.");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Cannot join a session until a reservation has been negotiated with the server.");
         }
     }
 }

NitroxClient/Communication/MultiplayerSession/ConnectionState/Disconnected.cs

@@ -32,7 +32,7 @@ private void ValidateState(IMultiplayerSessionConnectionContext sessionConnectio
             }
             catch (ArgumentNullException ex)
             {
-                DisplayStatusCode(StatusCode.connectionFailClient, false, "The context is missing an IP address." + ex.ToString());
+                DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "The context is missing an IP address." + ex.ToString());
             }
         }
 
@@ -44,7 +44,7 @@ private static void ValidateClient(IMultiplayerSessionConnectionContext sessionC
             }
             catch (ArgumentNullException ex)
             {
-                DisplayStatusCode(StatusCode.connectionFailClient, false, "The client must be set on the connection context before trying to negotiate a session reservation." + ex.ToString());
+                DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "The client must be set on the connection context before trying to negotiate a session reservation." + ex.ToString());
             }
         }
 
@@ -56,7 +56,7 @@ private static async Task StartClientAsync(string ipAddress, IClient client, int
 
                 if (!client.IsConnected)
                 {
-                    DisplayStatusCode(StatusCode.connectionFailClient, false, "The client failed to connect without providing a reason why.");
+                    DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "The client failed to connect without providing a reason why.");
                 }
             }
         }
@@ -74,12 +74,12 @@ private static void EstablishSessionPolicy(IMultiplayerSessionConnectionContext
 
         public void JoinSession(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Cannot join a session until a reservation has been negotiated with the server.");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Cannot join a session until a reservation has been negotiated with the server.");
         }
 
         public void Disconnect(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "Not connected to a multiplayer server.");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Not connected to a multiplayer server.");
         }
     }
 }

NitroxClient/Communication/MultiplayerSession/ConnectionState/EstablishingSessionPolicy.cs

@@ -47,15 +47,15 @@ private static void SessionPolicyIsNotNull(IMultiplayerSessionConnectionContext
             }
             catch (ArgumentNullException ex)
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "The context is missing a session policy." + ex.ToString());
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The context is missing a session policy." + ex.ToString());
             }
         }
 
         private void SessionPolicyPacketCorrelation(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
             if (!policyRequestCorrelationId.Equals(sessionConnectionContext.SessionPolicy.CorrelationId))
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "The packet was uncorrelated to the function of this processor: ");
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The packet was uncorrelated to the function of this processor: ");
             }
         }
 

NitroxClient/Communication/MultiplayerSession/ConnectionState/SessionJoined.cs

@@ -9,7 +9,7 @@ public class SessionJoined : ConnectionNegotiatedState
 
         public override void JoinSession(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
-            DisplayStatusCode(StatusCode.outboundConnectionAlreadyOpen, false, "The session is already in progress.");
+            DisplayStatusCode(StatusCode.OUTBOUND_CONNECTION_ALREADY_OPEN, false, "The session is already in progress.");
         }
     }
 }

NitroxClient/Communication/MultiplayerSession/ConnectionState/SessionReservationRejected.cs

@@ -9,7 +9,7 @@ public class SessionReservationRejected : ConnectionNegotiatedState
 
         public override void JoinSession(IMultiplayerSessionConnectionContext sessionConnectionContext)
         {
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "The session has rejected the reserveration request.");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "The session has rejected the reserveration request.");
         }
     }
 }

NitroxClient/Communication/MultiplayerSession/ConnectionState/SessionReserved.cs

@@ -28,7 +28,7 @@ private static void ValidateState(IMultiplayerSessionConnectionContext sessionCo
         {
             if (!sessionConnectionContext.Client.IsConnected)
             {
-                DisplayStatusCode(StatusCode.connectionFailClient, false, "The client is not connected.");
+                DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "The client is not connected.");
             }
         }
 

NitroxClient/Communication/MultiplayerSession/MultiplayerSessionManager.cs

@@ -68,15 +68,15 @@ public void ProcessSessionPolicy(MultiplayerSessionPolicy policy)
                                            .Replace("{serverVersion}", SessionPolicy.NitroxVersionAllowed.ToString())
                                            .Replace("{localVersion}", localVersion.ToString()));
                     CurrentState.Disconnect(this);
-                    DisplayStatusCode(StatusCode.versionMismatch, false, $"Client is out of date. Server: {SessionPolicy.NitroxVersionAllowed}, Client: {localVersion}");
+                    DisplayStatusCode(StatusCode.VERSION_MISMATCH, false, $"Client is out of date. Server: {SessionPolicy.NitroxVersionAllowed}, Client: {localVersion}");
                     return;
                 case 1:
                     Log.Error($"Server is out of date. Server: {SessionPolicy.NitroxVersionAllowed}, Client: {localVersion}");
                     Log.InGame(Language.main.Get("Nitrox_OutOfDateServer")
                                            .Replace("{serverVersion}", SessionPolicy.NitroxVersionAllowed.ToString())
                                            .Replace("{localVersion}", localVersion.ToString()));
                     CurrentState.Disconnect(this);
-                    DisplayStatusCode(StatusCode.versionMismatch, false, $"Server is out of date. Server: {SessionPolicy.NitroxVersionAllowed}, Client: {localVersion}");
+                    DisplayStatusCode(StatusCode.VERSION_MISMATCH, false, $"Server is out of date. Server: {SessionPolicy.NitroxVersionAllowed}, Client: {localVersion}");
                     return;
             }
 
@@ -121,7 +121,7 @@ public void Disconnect()
             if (CurrentState.CurrentStage != MultiplayerSessionConnectionStage.DISCONNECTED)
             {
                 CurrentState.Disconnect(this);
-                DisplayStatusCode(StatusCode.connectionFailClient, false, "Client has been disconnected");
+                DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "Client has been disconnected");
             }
         }
 

NitroxClient/Communication/NetworkingLayer/LiteNetLib/LiteNetLibClient.cs

@@ -112,7 +112,7 @@ private void Disconnected(NetPeer peer, DisconnectInfo disconnectInfo)
         if (Multiplayer.Active)
         {
             Modal.Get<LostConnectionModal>()?.Show();
-            DisplayStatusCode(StatusCode.connectionFailClient, false, "The client was disconnected from the server");
+            DisplayStatusCode(StatusCode.CONNECTION_FAIL_CLIENT, false, "The client was disconnected from the server");
         }
 
         IsConnected = false;

NitroxClient/Communication/Packets/Processors/BuildingResyncProcessor.cs

@@ -46,12 +46,12 @@ public IEnumerator ResyncBuildingEntities(Dictionary<BuildEntity, int> buildEnti
         Stopwatch stopwatch = Stopwatch.StartNew();
         BuildingHandler.Main.StartResync(buildEntities);
         yield return UpdateEntities<Base, BuildEntity>(buildEntities.Keys.ToList(), OverwriteBase, IsInCloseProximity).OnYieldError((exception) => {
-            DisplayStatusCode(StatusCode.syncFail, false, exception.ToString() + $"Encountered an exception while resyncing BuildEntities");
+            DisplayStatusCode(StatusCode.SYNC_FAIL, false, exception.ToString() + $"Encountered an exception while resyncing BuildEntities");
         });
 
         BuildingHandler.Main.StartResync(moduleEntities);
         yield return UpdateEntities<Constructable, ModuleEntity>(moduleEntities.Keys.ToList(), OverwriteModule, IsInCloseProximity).OnYieldError( (exception)=> {
-            DisplayStatusCode(StatusCode.syncFail, false, exception.ToString() + $"Encountered an exception while resyncing ModuleEntities");
+            DisplayStatusCode(StatusCode.SYNC_FAIL, false, exception.ToString() + $"Encountered an exception while resyncing ModuleEntities");
         });
         BuildingHandler.Main.StopResync();
 
@@ -120,7 +120,7 @@ public IEnumerator ResyncBuildingEntities(Dictionary<BuildEntity, int> buildEnti
             Log.Info($"[{typeof(E)} RESYNC] spawning entity {entity.Id}");
             yield return entities.SpawnEntityAsync(entity).OnYieldError((ex) => {
                 Log.Error(ex);
-                DisplayStatusCode(StatusCode.syncFail, false, "Error while trying to sync: " + ex.ToString());
+                DisplayStatusCode(StatusCode.SYNC_FAIL, false, "Error while trying to sync: " + ex.ToString());
             }
             );
         }

NitroxClient/Communication/Packets/Processors/ChatMessageProcessor.cs

@@ -56,7 +56,7 @@ private void LogClientMessage(ChatMessage message)
                 if (!remotePlayer.HasValue)
                 {
                     string playerTableFormatted = string.Join("\n", remotePlayerManager.GetAll().Select(ply => $"Name: '{ply.PlayerName}', Id: {ply.PlayerId}"));
-                    DisplayStatusCode(StatusCode.remotePlayerErr, true, $"Tried to add chat message for remote player that could not be found with id '${message.PlayerId}' and message: '{message.Text}'.\nAll remote players right now:\n{playerTableFormatted}");
+                    DisplayStatusCode(StatusCode.REMOTE_PLAYER_ERR, true, $"Tried to add chat message for remote player that could not be found with id '${message.PlayerId}' and message: '{message.Text}'.\nAll remote players right now:\n{playerTableFormatted}");
                 }
 
                 playerName = remotePlayer.Value.PlayerName;

NitroxClient/Communication/Packets/Processors/CyclopsDamageProcessor.cs

@@ -112,7 +112,7 @@ private void SetActiveDamagePoints(SubRoot cyclops, int[] damagePointIndexes)
                 // Looks like the list came in unordered. I've uttered "That shouldn't happen" enough to do sanity checks for what should be impossible.
                 if (packetDamagePointsIndex < damagePointIndexes.Length)
                 {
-                    DisplayStatusCode(StatusCode.invalidVariableVal, false, $"[CyclopsDamageProcessor packet.DamagePointIds did not fully iterate! Id: {damagePointIndexes[packetDamagePointsIndex]} had no matching Id in damageManager.damagePoints, or the order is incorrect!]");
+                    DisplayStatusCode(StatusCode.INVALID_VARIABLE_VAL, false, $"[CyclopsDamageProcessor packet.DamagePointIds did not fully iterate! Id: {damagePointIndexes[packetDamagePointsIndex]} had no matching Id in damageManager.damagePoints, or the order is incorrect!]");
                 }
             }
             else

NitroxClient/Communication/Packets/Processors/EntityDestroyedProcessor.cs

@@ -23,7 +23,7 @@ public override void Process(EntityDestroyed packet)
         entities.RemoveEntity(packet.Id);
         if (!NitroxEntity.TryGetObjectFrom(packet.Id, out GameObject gameObject))
         {
-            DisplayStatusCode(StatusCode.invalidPacket, false, $"[{nameof(EntityDestroyedProcessor)}] Could not find entity with id: {packet.Id} to destroy.");
+            DisplayStatusCode(StatusCode.INVALID_PACKET, false, $"[{nameof(EntityDestroyedProcessor)}] Could not find entity with id: {packet.Id} to destroy.");
             return;
         }
 

NitroxClient/Communication/Packets/Processors/EntityReparentedProcessor.cs

@@ -30,7 +30,7 @@ public override void Process(EntityReparented packet)
             // In some cases, the affected entity may be pending spawning or out of range.
             // we only require the parent (in this case, the visible entity is undergoing
             // some change that must be shown, and if not is an error).
-            DisplayStatusCode(StatusCode.invalidPacket, false, "The entity this process was trying to process was null");
+            DisplayStatusCode(StatusCode.INVALID_PACKET, false, "The entity this process was trying to process was null");
             return;
         }
         // Get the soon-to-be new parent of the gameObject from the packet
@@ -79,7 +79,7 @@ private void InventoryItemReparented(GameObject entity, GameObject newParent)
 
         if (!opContainer.HasValue)
         {
-            DisplayStatusCode(StatusCode.invalidVariableVal, false, $"Could not find container field on GameObject {newParent.GetFullHierarchyPath()}");
+            DisplayStatusCode(StatusCode.INVALID_VARIABLE_VAL, false, $"Could not find container field on GameObject {newParent.GetFullHierarchyPath()}");
             return;
         }
 

NitroxClient/Communication/Packets/Processors/EntityTransformUpdatesProcessor.cs

@@ -22,7 +22,7 @@ public override void Process(EntityTransformUpdates packet)
                 continue;
             } else
             {
-                DisplayStatusCode(StatusCode.invalidPacket, false, "GameObject provided to the packet processor was null");
+                DisplayStatusCode(StatusCode.INVALID_PACKET, false, "GameObject provided to the packet processor was null");
             }
 
             RemotelyControlled remotelyControlled = opGameObject.Value.GetComponent<RemotelyControlled>();

NitroxClient/Communication/Packets/Processors/EscapePodChangedProcessor.cs

@@ -30,7 +30,7 @@ public override void Process(EscapePodChanged packet)
                     escapePod = sub.GetComponent<EscapePod>();
                 } else
                 {
-                    DisplayStatusCode(StatusCode.invalidPacket, false, "EscapePodId was null");
+                    DisplayStatusCode(StatusCode.INVALID_PACKET, false, "EscapePodId was null");
                 }
 
                 remotePlayer.Value.SetEscapePod(escapePod);