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 5e47302 commit 63733bc
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;

namespace TerrariaOverhaul.Common.MainMenuOverlays
Expand All @@ -9,10 +12,30 @@ public ConfigurationMenuButton(string 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 63733bc

Please sign in to comment.