Warn if user tries to open solution file that isn't checked out

Stop user from opening solution files that aren't related to the target PR diff file.
github · Jan 12, 2018 · 4ae78f8 · 4ae78f8
1 parent 6415049
commit 4ae78f8
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/src/GitHub.VisualStudio/Views/GitHubPane/PullRequestDetailView.xaml.cs b/src/GitHub.VisualStudio/Views/GitHubPane/PullRequestDetailView.xaml.cs
@@ -132,6 +132,12 @@ async Task DoOpenLiveFile(IPullRequestFileNode file)
         {
             try
             {
+                if (!ViewModel.IsCheckedOut)
+                {
+                    ShowInfoMessage("Checkout PR branch before opening file in solution.");
+                    return;
+                }
+
                 var fullPath = ViewModel.GetLocalFilePath(file);
 
                 var activeView = NavigationService.FindActiveView();
@@ -152,6 +158,13 @@ async Task DoOpenLiveFile(IPullRequestFileNode file)
             }
         }
 
+        static void ShowInfoMessage(string message)
+        {
+            ErrorHandler.ThrowOnFailure(VsShellUtilities.ShowMessageBox(
+                Services.GitHubServiceProvider, message, null,
+                OLEMSGICON.OLEMSGICON_INFO, OLEMSGBUTTON.OLEMSGBUTTON_OK, OLEMSGDEFBUTTON.OLEMSGDEFBUTTON_FIRST));
+        }
+
         async Task DoDiffFile(IPullRequestFileNode file, bool workingDirectory)
         {
             try