github: fix 'open in browser' url

/tree/ - for directories
/blob/ - fol files
GitHub automatically converts one to another

Except one case: root folder. /blob/master -> 404, not into /tree/master

diff --git a/plugins/github/src/org/jetbrains/plugins/github/GithubOpenInBrowserAction.java b/plugins/github/src/org/jetbrains/plugins/github/GithubOpenInBrowserAction.java
index f587d44c8cb6352e607e9d89f6c5ee97158feb76..cec5ba26972167303758956de392861f1e585332 100644 (file)
--- a/plugins/github/src/org/jetbrains/plugins/github/GithubOpenInBrowserAction.java
+++ b/plugins/github/src/org/jetbrains/plugins/github/GithubOpenInBrowserAction.java
@@ -22,6 +22,7 @@ import com.intellij.openapi.editor.Editor;
 import com.intellij.openapi.editor.SelectionModel;
 import com.intellij.openapi.project.DumbAwareAction;
 import com.intellij.openapi.project.Project;
+import com.intellij.openapi.util.text.StringUtil;
 import com.intellij.openapi.vcs.changes.Change;
 import com.intellij.openapi.vcs.changes.ChangeListManager;
 import com.intellij.openapi.vfs.VirtualFile;
@@ -156,7 +157,12 @@ public class GithubOpenInBrowserAction extends DumbAwareAction {
     if (githubRepoUrl == null) {
       return null;
     }
-    builder.append(githubRepoUrl).append("/blob/").append(branch).append(relativePath);
+    if (StringUtil.isEmptyOrSpaces(relativePath)) {
+      builder.append(githubRepoUrl).append("/tree/").append(branch);
+    }
+    else {
+      builder.append(githubRepoUrl).append("/blob/").append(branch).append(relativePath);
+    }
 
     if (editor != null && editor.getDocument().getLineCount() >= 1) {
       // lines are counted internally from 0, but from 1 on github