Skip to content

Commit

Permalink
Fixed #149 (Config button may crash the game)
Browse files Browse the repository at this point in the history
  • Loading branch information
Mirsario committed Sep 11, 2022
1 parent d278d99 commit c50f8e6
Showing 1 changed file with 23 additions and 0 deletions.
23 changes: 23 additions & 0 deletions Common/MainMenuOverlays/ConfigurationMenuButton.cs
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
using System.Diagnostics;
using System.IO;
using System.Security.Policy;
using Terraria;
using TerrariaOverhaul.Core.Configuration;
using TerrariaOverhaul.Core.Localization;

Expand All @@ -10,9 +13,29 @@ public ConfigurationMenuButton(Text text) : base(text) { }

protected override void OnClicked()
{
/*
Process.Start(new ProcessStartInfo(ConfigSystem.ConfigPath) {
UseShellExecute = true,
Verb = "open",
});
*/

string configFilePath = ConfigSystem.ConfigPath;
string configDirectory = ConfigSystem.ConfigDirectory;
string configFileName = Path.GetFileName(configFilePath);

try {
Terraria.Utils.OpenToURL(configFilePath);
}
catch {
try {
Terraria.Utils.OpenToURL(configDirectory);
}
catch {
if (Main.menuMode != 888) {
Terraria.Utils.ShowFancyErrorMessage($"[c/FF7777:Unable to open Overhaul's configuration file for editing.]\r\nPlease manually navigate to [c/9fecf0:{configDirectory}] and modify [c/9fecf0:{configFileName}] with a text editor of your choosing.\r\n\r\nThe lack of a configuration GUI is temporary.", 0);
}
}
}
}
}

0 comments on commit c50f8e6

Please sign in to comment.