Still need to unescape / on macOs & Linux

rkeithhill · rkeithhill · commit 6fa07db2f916 · 2019-04-09T22:21:11.000-06:00
diff --git a/src/PowerShellEditorServices/Workspace/Workspace.cs b/src/PowerShellEditorServices/Workspace/Workspace.cs
@@ -709,8 +709,9 @@ public static string ConvertPathToDocumentUri(string path)
             }
             else
             {
-                // On non-Windows system, simply append the escaped path.
-                docUriStrBld.Append(escapedPath);
+                // On non-Windows systems, append the escapedPath and undo the over-aggressive
+                // escaping of / done by Uri.EscapeDataString.
+                docUriStrBld.Append(escapedPath).Replace("%2F", "/");
             }
 
 #if !CoreCLR

Original file line number	Diff line number	Diff line change
`@@ -709,8 +709,9 @@ public static string ConvertPathToDocumentUri(string path)`
`709`	`709`	`}`
`710`	`710`	`else`
`711`	`711`	`{`
`712`		`- // On non-Windows system, simply append the escaped path.`
`713`		`- docUriStrBld.Append(escapedPath);`
	`712`	`+ // On non-Windows systems, append the escapedPath and undo the over-aggressive`
	`713`	`+ // escaping of / done by Uri.EscapeDataString.`
	`714`	`+ docUriStrBld.Append(escapedPath).Replace("%2F", "/");`
`714`	`715`	`}`
`715`	`716`
`716`	`717`	`#if !CoreCLR`