fixed igcmd set/remove-default-viewer cmd does not follow show ui param

#1835
d2phap · Feb 1, 2024 · 24f4237 · 24f4237
1 parent 5c9b42c
commit 24f4237
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 11 deletions.
diff --git a/Source/igcmd/Functions.cs b/Source/igcmd/Functions.cs
@@ -68,12 +68,7 @@ public static IgExitCode SetDesktopWallpaper(string imgPath, string styleStr)
     /// </summary>
     /// <param name="enable"></param>
     /// <param name="exts">Extensions to proceed. Example: <c>.png;.jpg;</c></param>
-    public static IgExitCode SetAppExtensions(
-        bool enable,
-        string exts = "",
-        bool perMachine = false,
-        bool showUi = false,
-        bool hideAdminRequiredErrorUi = false)
+    public static IgExitCode SetAppExtensions(bool enable, string exts = "", bool perMachine = false)
     {
         var langPath = enable
             ? "FrmMain.MnuSetDefaultPhotoViewer"
@@ -91,9 +86,12 @@ public static IgExitCode SetAppExtensions(
                 : ExplorerApi.UnregisterAppAndExtensions(exts, perMachine);
             if (error != null) throw error;
 
-            _ = Config.ShowInfo(null,
-                title: Config.Language[langPath],
-                heading: Config.Language[$"{langPath}._Success"]);
+            if (Program.ShowUi)
+            {
+                _ = Config.ShowInfo(null,
+                    title: Config.Language[langPath],
+                    heading: Config.Language[$"{langPath}._Success"]);
+            }
         }, (error) =>
         {
             _ = Config.ShowError(null,

diff --git a/Source/igcmd/Program.cs b/Source/igcmd/Program.cs
@@ -131,7 +131,7 @@ private static int Main(string[] args)
                 exts = CmdArgs[1];
             }
 
-            return (int)Functions.SetAppExtensions(true, exts, PerMachine, ShowUi, HideAdminRequiredErrorUi);
+            return (int)Functions.SetAppExtensions(true, exts, PerMachine);
         }
         #endregion
 
@@ -145,7 +145,7 @@ private static int Main(string[] args)
                 exts = CmdArgs[1];
             }
 
-            return (int)Functions.SetAppExtensions(false, exts, PerMachine, ShowUi, HideAdminRequiredErrorUi);
+            return (int)Functions.SetAppExtensions(false, exts, PerMachine);
         }
         #endregion