DotNetTools: Fix assembly menu ignoring FileBrowseExecutable settings

dmex · dmex · commit 973a76131ee5 · 2021-01-21T13:47:42.000+11:00
diff --git a/plugins/DotNetTools/asmpage.c b/plugins/DotNetTools/asmpage.c
@@ -524,7 +524,13 @@ VOID DotNetAsmShowContextMenu(
                 {
                     if (!PhIsNullOrEmptyString(node->PathText) && PhDoesFileExistsWin32(PhGetString(node->PathText)))
                     {
-                        PhShellExploreFile(Context->WindowHandle, node->PathText->Buffer);
+                        PhShellExecuteUserString(
+                            Context->WindowHandle,
+                            L"FileBrowseExecutable",
+                            node->PathText->Buffer,
+                            FALSE,
+                            L"Make sure the Explorer executable file is present."
+                            );
                     }
                 }
                 break;

Original file line number	Diff line number	Diff line change
`@@ -524,7 +524,13 @@ VOID DotNetAsmShowContextMenu(`
`524`	`524`	`{`
`525`	`525`	`if (!PhIsNullOrEmptyString(node->PathText) && PhDoesFileExistsWin32(PhGetString(node->PathText)))`
`526`	`526`	`{`
`527`		`- PhShellExploreFile(Context->WindowHandle, node->PathText->Buffer);`
	`527`	`+ PhShellExecuteUserString(`
	`528`	`+ Context->WindowHandle,`
	`529`	`+ L"FileBrowseExecutable",`
	`530`	`+ node->PathText->Buffer,`
	`531`	`+ FALSE,`
	`532`	`+ L"Make sure the Explorer executable file is present."`
	`533`	`+ );`
`528`	`534`	`}`
`529`	`535`	`}`
`530`	`536`	`break;`