use program config from appdata/roaming instead of binary file (#34)

OpenLoco · Feb 14, 2024 · d936223 · d936223
1 parent d6efaae
commit d936223
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 48 deletions.
diff --git a/Gui/GuiSettings.cs b/Gui/GuiSettings.cs
@@ -1,6 +1,3 @@
-using System.Runtime.Serialization;
-using System.Text.Json.Serialization;
-
 namespace OpenLoco.ObjectEditor.Gui
 {
 	public class GuiSettings

diff --git a/Gui/MainForm.Designer.cs b/Gui/MainForm.Designer.cs
diff --git a/Gui/MainForm.cs b/Gui/MainForm.cs
@@ -78,7 +78,9 @@ int CurrentUIImagePageNumber
 
 		const int ImagesPerPage = 50;
 
-		const string SettingsFile = "./settings.json";
+		const string ApplicationName = "OpenLoco Object Editor";
+		string SettingsPath => Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData), ApplicationName);
+		string SettingsFile => Path.Combine(SettingsPath, "settings.json");
 
 		public MainForm()
 		{
@@ -107,7 +109,7 @@ public MainForm()
 			{
 				var buf = new byte[5];
 				var arr = stream!.Read(buf);
-				Text = $"OpenLoco Object Editor - {Encoding.ASCII.GetString(buf)}";
+				Text = $"{ApplicationName} - {Encoding.ASCII.GetString(buf)}";
 			}
 		}
 

diff --git a/Gui/MainFormModel.cs b/Gui/MainFormModel.cs
@@ -23,43 +23,6 @@ class MainFormModel
 
 		//public OpenLocoObjectEditor.ObjectManager ObjectManager { get; private set; } = new();
 
-		//public string PaletteFile
-		//{
-		//	get => Settings.PaletteFile;
-		//	set
-		//	{
-		//		Settings.PaletteFile = value;
-		//		LoadPalette();
-		//	}
-		//}
-
-		//private void LoadPalette()
-		//{
-		//	//if (G1 == null)
-		//	{
-		//		try
-		//		{
-		//			var paletteBitmap = new Bitmap(Settings.PaletteFile);
-		//			Palette = PaletteHelpers.PaletteFromBitmap(paletteBitmap);
-		//			SaveSettings();
-		//			logger.Debug($"Successfully loaded palette file {Settings.PaletteFile}");
-		//		}
-		//		catch (ArgumentException ex)
-		//		{
-		//			logger.Error(ex);
-		//		}
-		//	}
-		//	//else
-		//	//{
-		//	//	var g1PaletteElement = G1.G1Elements[ImageIds.MainPalette];
-		//	//	Palette = g1PaletteElement.ImageData
-		//	//		.Take(256 * 3)
-		//	//		.Chunk(3)
-		//	//		.Select(x => Color.FromArgb(x[2], x[1], x[0]))
-		//	//		.ToArray();
-		//	//}
-		//}
-
 		public PaletteMap PaletteMap { get; set; }
 
 		public G1Dat? G1 { get; set; }
@@ -166,13 +129,16 @@ static bool ValidateSettings(GuiSettings settings, ILogger logger)
 
 		public void SaveSettings()
 		{
-			//if (Settings.HasChanges)
+			var options = GetOptions();
+			var text = JsonSerializer.Serialize(Settings, options);
+
+			var parentDir = Path.GetDirectoryName(SettingsFile);
+			if (!Directory.Exists(parentDir))
 			{
-				var options = GetOptions();
-				var text = JsonSerializer.Serialize(Settings, options);
-				File.WriteAllText(SettingsFile, text);
-				//Settings.HasChanges = false;
+				_ = Directory.CreateDirectory(parentDir);
 			}
+
+			File.WriteAllText(SettingsFile, text);
 		}
 
 		// this method loads every single object entirely. it takes a long time to run