Allows cfg to be updated with any changes rather than always using de…

…fault properties for those missing (#1545) * Allows cfg to be updated with any changes rather than always using default properties for those missing * Changed Update to handle null actions
SubnauticaNitrox · Jul 25, 2021 · d3670ea · d3670ea
1 parent 4a35596
commit d3670ea
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/NitroxModel/Serialization/NitroxConfig.cs b/NitroxModel/Serialization/NitroxConfig.cs
@@ -131,12 +131,12 @@ public void Serialize()
         ///     Ensures updates are properly persisted to the backing config file without overwriting user edits.
         /// </summary>
         /// <param name="config"></param>
-        public void Update(Action<T> config)
+        public void Update(Action<T> config = null)
         {
             try
             {
                 Deserialize();
-                config(this as T);
+                config?.Invoke(this as T);
             }
             finally
             {

diff --git a/NitroxServer/Serialization/ServerConfig.cs b/NitroxServer/Serialization/ServerConfig.cs
@@ -109,7 +109,7 @@ public string SaveName
         public static ServerConfig Load()
         {
             ServerConfig config = new();
-            config.Deserialize();
+            config.Update();
             return config;
         }
     }

diff --git a/NitroxServer/Server.cs b/NitroxServer/Server.cs
@@ -133,10 +133,6 @@ public void Stop(bool shouldSave = true)
             if (shouldSave)
             {
                 Save();
-                if (!serverConfig.ConfigFileExists)
-                {
-                    serverConfig.Serialize();
-                }
             }
 
             server.Stop();