Merge pull request #146 from Flow-Launcher/dev

jjw24 · web-flow · commit 135f63a48485 · 2020-08-28T05:12:47.000+10:00
Release 1.2.1
diff --git a/Flow.Launcher.Test/Plugins/ExplorerTest.cs b/Flow.Launcher.Test/Plugins/ExplorerTest.cs
@@ -58,8 +58,8 @@ public void GivenWindowsIndexSearch_WhenProvidedFolderPath_ThenQueryWhereRestric
                 $"Actual: {result}{Environment.NewLine}");
         }
 
-        [TestCase("C:\\", "SELECT TOP 100 System.FileName, System.ItemPathDisplay, System.ItemType FROM SystemIndex WHERE directory='file:C:\\'")]
-        [TestCase("C:\\SomeFolder\\", "SELECT TOP 100 System.FileName, System.ItemPathDisplay, System.ItemType FROM SystemIndex WHERE directory='file:C:\\SomeFolder\\'")]
+        [TestCase("C:\\", "SELECT TOP 100 System.FileName, System.ItemUrl, System.ItemType FROM SystemIndex WHERE directory='file:C:\\'")]
+        [TestCase("C:\\SomeFolder\\", "SELECT TOP 100 System.FileName, System.ItemUrl, System.ItemType FROM SystemIndex WHERE directory='file:C:\\SomeFolder\\'")]
         public void GivenWindowsIndexSearch_WhenSearchTypeIsTopLevelDirectorySearch_ThenQueryShouldUseExpectedString(string folderPath, string expectedString)
         {
             // Given
@@ -74,7 +74,7 @@ public void GivenWindowsIndexSearch_WhenSearchTypeIsTopLevelDirectorySearch_Then
                 $"Actual string was: {queryString}{Environment.NewLine}");
         }
 
-        [TestCase("C:\\SomeFolder\\flow.launcher.sln", "SELECT TOP 100 System.FileName, System.ItemPathDisplay, System.ItemType " +
+        [TestCase("C:\\SomeFolder\\flow.launcher.sln", "SELECT TOP 100 System.FileName, System.ItemUrl, System.ItemType " +
             "FROM SystemIndex WHERE (System.FileName LIKE 'flow.launcher.sln%' " +
                                         "OR CONTAINS(System.FileName,'\"flow.launcher.sln*\"',1033))" +
                                         " AND directory='file:C:\\SomeFolder'")]
@@ -126,7 +126,7 @@ public void GivenWindowsIndexSearch_WhenSearchAllFoldersAndFiles_ThenQueryWhereR
                 $"Actual string was: {resultString}{Environment.NewLine}");
         }
 
-        [TestCase("flow.launcher.sln", "SELECT TOP 100 \"System.FileName\", \"System.ItemPathDisplay\", \"System.ItemType\" " +
+        [TestCase("flow.launcher.sln", "SELECT TOP 100 \"System.FileName\", \"System.ItemUrl\", \"System.ItemType\" " +
             "FROM \"SystemIndex\" WHERE (System.FileName LIKE 'flow.launcher.sln%' " +
                                         "OR CONTAINS(System.FileName,'\"flow.launcher.sln*\"',1033)) AND scope='file:'")]
         public void GivenWindowsIndexSearch_WhenSearchAllFoldersAndFiles_ThenQueryShouldUseExpectedString(
@@ -200,7 +200,7 @@ public void GivenWindowsIndexSearch_WhenQueryWhereRestrictionsIsForFileContentSe
                 $"Actual string was: {resultString}{Environment.NewLine}");
         }
 
-        [TestCase("some words", "SELECT TOP 100 System.FileName, System.ItemPathDisplay, System.ItemType " +
+        [TestCase("some words", "SELECT TOP 100 System.FileName, System.ItemUrl, System.ItemType " +
                     "FROM SystemIndex WHERE FREETEXT('some words') AND scope='file:'")]
         public void GivenWindowsIndexSearch_WhenSearchForFileContent_ThenQueryShouldUseExpectedString(
             string userSearchString, string expectedString)
diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/Search/EnvironmentVariables.cs b/Plugins/Flow.Launcher.Plugin.Explorer/Search/EnvironmentVariables.cs
@@ -22,11 +22,20 @@ internal static Dictionary<string, string> LoadEnvironmentStringPaths()
 
             foreach (DictionaryEntry special in Environment.GetEnvironmentVariables())
             {
-                if (Directory.Exists(special.Value.ToString()))
+                var path = special.Value.ToString();
+                if (Directory.Exists(path))
                 {
+                    // we add a trailing slash to the path to make sure drive paths become valid absolute paths.
+                    // for example, if %systemdrive% is C: we turn it to C:\
+                    path = path.TrimEnd(Path.DirectorySeparatorChar) + Path.DirectorySeparatorChar;
+
+                    // if we don't have an absolute path, we use Path.GetFullPath to get one.
+                    // for example, if %homepath% is \Users\John we turn it to C:\Users\John
+                    path = Path.IsPathFullyQualified(path) ? path : Path.GetFullPath(path);
+
                     // Variables are returned with a mixture of all upper/lower case. 
                     // Call ToLower() to make the results look consistent
-                    envStringPaths.Add(special.Key.ToString().ToLower(), special.Value.ToString());
+                    envStringPaths.Add(special.Key.ToString().ToLower(), path);
                 }
             }
 
diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/Search/WindowsIndex/IndexSearch.cs b/Plugins/Flow.Launcher.Plugin.Explorer/Search/WindowsIndex/IndexSearch.cs
@@ -51,17 +51,19 @@ internal List<Result> ExecuteWindowsIndexSearch(string indexQueryString, string
                                 {
                                     if (dataReaderResults.GetValue(0) != DBNull.Value && dataReaderResults.GetValue(1) != DBNull.Value)
                                     {
+                                        var path = new Uri(dataReaderResults.GetString(1)).LocalPath;
+
                                         if (dataReaderResults.GetString(2) == "Directory")
                                         {
                                             folderResults.Add(resultManager.CreateFolderResult(
                                                                                 dataReaderResults.GetString(0),
-                                                                                dataReaderResults.GetString(1), 
-                                                                                dataReaderResults.GetString(1), 
+                                                                                path,
+                                                                                path, 
                                                                                 query, true, true));
                                         }
                                         else
                                         {
-                                            fileResults.Add(resultManager.CreateFileResult(dataReaderResults.GetString(1), query, true, true));
+                                            fileResults.Add(resultManager.CreateFileResult(path, query, true, true));
                                         }
                                     }
                                 }
diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/Search/WindowsIndex/QueryConstructor.cs b/Plugins/Flow.Launcher.Plugin.Explorer/Search/WindowsIndex/QueryConstructor.cs
@@ -21,7 +21,7 @@ public CSearchQueryHelper CreateBaseQuery()
             baseQuery.QueryMaxResults = settings.MaxResult;
 
             // Set list of columns we want to display, getting the path presently
-            baseQuery.QuerySelectColumns = "System.FileName, System.ItemPathDisplay, System.ItemType";
+            baseQuery.QuerySelectColumns = "System.FileName, System.ItemUrl, System.ItemType";
 
             // Filter based on file name
             baseQuery.QueryContentProperties = "System.FileName";
diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/plugin.json b/Plugins/Flow.Launcher.Plugin.Explorer/plugin.json
@@ -7,7 +7,7 @@
   "Name": "Explorer",
   "Description": "Search and manage files and folders. Explorer utilises Windows Index Search",
   "Author": "Jeremy Wu",
-  "Version": "1.2.0",
+  "Version": "1.2.2",
   "Language": "csharp",
   "Website": "https://github.com/Flow-Launcher/Flow.Launcher",
   "ExecuteFileName": "Flow.Launcher.Plugin.Explorer.dll",
diff --git a/SolutionAssemblyInfo.cs b/SolutionAssemblyInfo.cs
@@ -16,6 +16,6 @@
 [assembly: AssemblyTrademark("")]
 [assembly: AssemblyCulture("")]
 [assembly: ComVisible(false)]
-[assembly: AssemblyVersion("1.2.0")]
-[assembly: AssemblyFileVersion("1.2.0")]
-[assembly: AssemblyInformationalVersion("1.2.0")]
+[assembly: AssemblyVersion("1.2.1")]
+[assembly: AssemblyFileVersion("1.2.1")]
+[assembly: AssemblyInformationalVersion("1.2.1")]
diff --git a/appveyor.yml b/appveyor.yml
@@ -1,4 +1,4 @@
-version: '1.2.0.{build}'
+version: '1.2.1.{build}'
 
 init:
 - ps: |

Original file line number	Diff line number	Diff line change
`@@ -22,11 +22,20 @@ internal static Dictionary<string, string> LoadEnvironmentStringPaths()`
`22`	`22`
`23`	`23`	`foreach (DictionaryEntry special in Environment.GetEnvironmentVariables())`
`24`	`24`	`{`
`25`		`- if (Directory.Exists(special.Value.ToString()))`
	`25`	`+ var path = special.Value.ToString();`
	`26`	`+ if (Directory.Exists(path))`
`26`	`27`	`{`
	`28`	`+ // we add a trailing slash to the path to make sure drive paths become valid absolute paths.`
	`29`	`+ // for example, if %systemdrive% is C: we turn it to C:\`
	`30`	`+ path = path.TrimEnd(Path.DirectorySeparatorChar) + Path.DirectorySeparatorChar;`
	`31`	`+`
	`32`	`+ // if we don't have an absolute path, we use Path.GetFullPath to get one.`
	`33`	`+ // for example, if %homepath% is \Users\John we turn it to C:\Users\John`
	`34`	`+ path = Path.IsPathFullyQualified(path) ? path : Path.GetFullPath(path);`
	`35`	`+`
`27`	`36`	`// Variables are returned with a mixture of all upper/lower case.`
`28`	`37`	`// Call ToLower() to make the results look consistent`
`29`		`- envStringPaths.Add(special.Key.ToString().ToLower(), special.Value.ToString());`
	`38`	`+ envStringPaths.Add(special.Key.ToString().ToLower(), path);`
`30`	`39`	`}`
`31`	`40`	`}`
`32`	`41`
Original file line number	Diff line number	Diff line change
`@@ -51,17 +51,19 @@ internal List<Result> ExecuteWindowsIndexSearch(string indexQueryString, string`
`51`	`51`	`{`
`52`	`52`	`if (dataReaderResults.GetValue(0) != DBNull.Value && dataReaderResults.GetValue(1) != DBNull.Value)`
`53`	`53`	`{`
	`54`	`+ var path = new Uri(dataReaderResults.GetString(1)).LocalPath;`
	`55`	`+`
`54`	`56`	`if (dataReaderResults.GetString(2) == "Directory")`
`55`	`57`	`{`
`56`	`58`	`folderResults.Add(resultManager.CreateFolderResult(`
`57`	`59`	`dataReaderResults.GetString(0),`
`58`		`- dataReaderResults.GetString(1),`
`59`		`- dataReaderResults.GetString(1),`
	`60`	`+ path,`
	`61`	`+ path,`
`60`	`62`	`query, true, true));`
`61`	`63`	`}`
`62`	`64`	`else`
`63`	`65`	`{`
`64`		`- fileResults.Add(resultManager.CreateFileResult(dataReaderResults.GetString(1), query, true, true));`
	`66`	`+ fileResults.Add(resultManager.CreateFileResult(path, query, true, true));`
`65`	`67`	`}`
`66`	`68`	`}`
`67`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-version: '1.2.0.{build}'`
	`1`	`+version: '1.2.1.{build}'`
`2`	`2`
`3`	`3`	`init:`
`4`	`4`	`- ps: \|`