refactor: load external tool's icon on startup

Author: love-linger

src/App.axaml.cs

@@ -266,6 +266,7 @@ public override void OnFrameworkInitializationCompleted()
             if (ApplicationLifetime is IClassicDesktopStyleApplicationLifetime desktop)
             {
                 BindingPlugins.DataValidators.RemoveAt(0);
+                Native.OS.SetupEnternalTools();
 
                 var launcher = new Views.Launcher();
                 _notificationReceiver = launcher;

src/Models/ExternalTool.cs

@@ -12,34 +12,23 @@ namespace SourceGit.Models
 {
     public class ExternalTool
     {
-        public string Name { get; set; } = string.Empty;
-        public string Icon { get; set; } = string.Empty;
-        public string Executable { get; set; } = string.Empty;
-        public string OpenCmdArgs { get; set; } = string.Empty;
+        public string Name { get; private set; } = string.Empty;
+        public string Executable { get; private set; } = string.Empty;
+        public string OpenCmdArgs { get; private set; } = string.Empty;
+        public Bitmap IconImage { get; private set; } = null;
 
-        public Bitmap IconImage
+        public ExternalTool(string name, string icon, string executable, string openCmdArgs)
         {
-            get
-            {
-                if (_isFirstTimeGetIcon)
-                {
-                    _isFirstTimeGetIcon = false;
-
-                    if (!string.IsNullOrWhiteSpace(Icon))
-                    {
-                        try
-                        {
-                            var icon = AssetLoader.Open(new Uri($"avares://SourceGit/Resources/ExternalToolIcons/{Icon}.png", UriKind.RelativeOrAbsolute));
-                            _iconImage = new Bitmap(icon);
-                        }
-                        catch
-                        {
-                        }
-                    }
-                }
+            Name = name;
+            Executable = executable;
+            OpenCmdArgs = openCmdArgs;
 
-                return _iconImage;
+            try
+            {
+                var asset = AssetLoader.Open(new Uri($"avares://SourceGit/Resources/ExternalToolIcons/{icon}.png", UriKind.RelativeOrAbsolute));
+                IconImage = new Bitmap(asset);
             }
+            catch { }
         }
 
         public void Open(string repo)
@@ -52,9 +41,6 @@ public void Open(string repo)
                 UseShellExecute = false,
             });
         }
-
-        private bool _isFirstTimeGetIcon = true;
-        private Bitmap _iconImage = null;
     }
 
     public class JetBrainsState
@@ -107,13 +93,7 @@ public void TryAdd(string name, string icon, string args, string env, Func<strin
                     return;
             }
 
-            Founded.Add(new ExternalTool
-            {
-                Name = name,
-                Icon = icon,
-                OpenCmdArgs = args,
-                Executable = path
-            });
+            Founded.Add(new ExternalTool(name, icon, path, args));
         }
 
         public void VSCode(Func<string> platformFinder)
@@ -154,13 +134,11 @@ public void FindJetBrainsFromToolbox(Func<string> platformFinder)
                     if (exclude.Contains(tool.ToolId.ToLowerInvariant()))
                         continue;
 
-                    Founded.Add(new ExternalTool
-                    {
-                        Name = $"{tool.DisplayName} {tool.DisplayVersion}",
-                        Icon = supported_icons.Contains(tool.ProductCode) ? $"JetBrains/{tool.ProductCode}" : $"JetBrains/JB",
-                        OpenCmdArgs = "\"{0}\"",
-                        Executable = Path.Combine(tool.InstallLocation, tool.LaunchCommand),
-                    });
+                    Founded.Add(new ExternalTool(
+                        $"{tool.DisplayName} {tool.DisplayVersion}",
+                        supported_icons.Contains(tool.ProductCode) ? $"JetBrains/{tool.ProductCode}" : "JetBrains/JB",
+                        Path.Combine(tool.InstallLocation, tool.LaunchCommand),
+                        "\"{0}\""));
                 }
             }
         }

src/Native/OS.cs

@@ -41,8 +41,6 @@ static OS()
             {
                 throw new Exception("Platform unsupported!!!");
             }
-
-            ExternalTools = _backend.FindExternalTools();
         }
 
         public static Models.Shell GetShell()
@@ -77,6 +75,11 @@ public static void SetupApp(AppBuilder builder)
             _backend.SetupApp(builder);
         }
 
+        public static void SetupEnternalTools()
+        {
+            ExternalTools = _backend.FindExternalTools();
+        }
+
         public static string FindGitExecutable()
         {
             return _backend.FindGitExecutable();