From 52729e34318bed9a5001dea1f9aa74220b1a1f52 Mon Sep 17 00:00:00 2001 From: TheBestPessimist Date: Fri, 3 Mar 2023 14:12:44 +0200 Subject: [PATCH] cleanup --- .../SharedCommands/FilesFolders.cs | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/Flow.Launcher.Plugin/SharedCommands/FilesFolders.cs b/Flow.Launcher.Plugin/SharedCommands/FilesFolders.cs index bd8d32ff5..017dfa62f 100644 --- a/Flow.Launcher.Plugin/SharedCommands/FilesFolders.cs +++ b/Flow.Launcher.Plugin/SharedCommands/FilesFolders.cs @@ -14,8 +14,6 @@ namespace Flow.Launcher.Plugin.SharedCommands { private const string FileExplorerProgramName = "explorer"; - private const string FileExplorerProgramEXE = "explorer.exe"; - /// /// Copies the folder and all of its files and folders /// including subfolders to the target location @@ -145,13 +143,25 @@ namespace Flow.Launcher.Plugin.SharedCommands return File.Exists(filePath); } + /* + ❓ + todo does it make sense to remove Flow.Launcher.Plugin.IPublicAPI.OpenDirectory (used as Context.API.OpenDirectory(path)), + and replace with this function? + todo or the other way around: remove this function and use OpenDirectory + because i don't see the point of having 2 functions which do the same thing + */ /// /// Open a directory window (using the OS's default handler, usually explorer) /// /// public static void OpenPath(string fileOrFolderPath) { - var psi = new ProcessStartInfo { FileName = FileExplorerProgramName, UseShellExecute = true, Arguments = '"' + fileOrFolderPath + '"' }; + var psi = new ProcessStartInfo + { + FileName = FileExplorerProgramName, + UseShellExecute = true, + Arguments = '"' + fileOrFolderPath + '"' + }; try { if (LocationExists(fileOrFolderPath) || FileExists(fileOrFolderPath)) @@ -173,7 +183,7 @@ namespace Flow.Launcher.Plugin.SharedCommands /// public static void OpenContainingFolder(string path) { - Process.Start(FileExplorerProgramEXE, $" /select,\"{path}\""); + Process.Start(FileExplorerProgramName, $" /select,\"{path}\""); } ///