Merge pull request #8 from ahmetoozcan/powershell-path-corrupt-fix

Closes #7

Author: HorridModz

Main.cs

@@ -55,7 +55,7 @@ private static bool _IsValidPath(string path)
         {
             int c = path[i];
 
-            if (c == '<' || c == '>' || c == '|' || c == '*' || c == '?' || c < 32)
+            if (c == '<' || c == '>' || c == '|' || c == '*' || c == '?' || c < 32 || c == '/')
                 return false;
         }
 
@@ -210,6 +210,9 @@ public List<Result> Query(Query query)
                         Context.API.ShowMsgError("Failed to add to PATH", $"Failed to add {$"\"{folderPath}\""} to {(system ? "system" : "user")} PATH - {ex.Message}");
                         return true;
                     }
+                    if (folderPath.Contains(';')) {
+                        Context.API.ShowMsgError("Warning to Powershell users", "This path won't work in Powershell because it contains a semicolon. For more information, visit https://github.com/HorridModz/Flow.Launcher.Plugin.Add2Path/pull/8");
+                    } 
                     Context.API.ShowMsg("Successfully added to PATH", $"Successfully added {$"\"{folderPath}\""} to {(system ? "system" : "user")} PATH.");
                     return true;
                 },

PathUtils.cs

@@ -93,8 +93,7 @@ public static string _EncodeParameterArgument(string original)
         if (string.IsNullOrEmpty(original))
             return original;
         string value = Regex.Replace(original, @"(\\*)" + "\"", @"$1\$0");
-        value = Regex.Replace(value, @"^(.*\s.*?)(\\*)$", "\"$1$2$2\"");
-        return value;
+        return Regex.Replace(value, @"^(.*\s.*?)(\\*)$", "\"$1$2$2\"");
     }
 
     private static string _GetRegistryKey(bool system = false)
@@ -148,8 +147,13 @@ public static void SetFullString(string value, bool system = false)
 
     public static void Set(List<string> folderPaths, bool system = false)
     {
-        // Wrap folder paths in quotes
-        folderPaths = (from folderPath in folderPaths select $"\"{folderPath}\"").ToList();
+        // ?: Windows wraps paths with semicolon (;) as default, 
+        // ?: this way CMD can read path value by discarding path seperator character 
+        // ?: (which is semicolon for in this case) which is between double quotes 
+        // !: But in the other hand path with semicolons won't be working on Powershell because Powershell
+        // !: doesn't trims any double quotes and seperates paths only by checking path seperator character
+        // ?: You can check all the details from https://github.com/HorridModz/Flow.Launcher.Plugin.Add2Path/pull/8 
+        folderPaths = (from folderPath in folderPaths select folderPath.Contains(';') ? $"\"{folderPath}\"" : folderPath).ToList();
         PathUtils.SetFullString(String.Join(";", folderPaths), system);
     }