link directly to modified files in side by side view

Author: Kraig Amador

src/main/java/hudson/plugins/git/browser/GitoriousWeb.java

@@ -40,7 +40,7 @@ public URL getChangeSetLink(GitChangeSet changeSet) throws IOException {
     /**
      * Creates a link to the commit diff.
      * 
-     * https://[Gitorious URL]/commit/a9182a07750c9a0dfd89a8461adf72ef5ef0885b
+     * https://[Gitorious URL]/commit/a9182a07750c9a0dfd89a8461adf72ef5ef0885b/diffs?diffmode=sidebyside&fragment=1#[path to file]
      * 
      * @param path
      * @return diff link
@@ -49,8 +49,7 @@ public URL getChangeSetLink(GitChangeSet changeSet) throws IOException {
     @Override
     public URL getDiffLink(Path path) throws IOException {
         final GitChangeSet changeSet = path.getChangeSet();
-        // gitorious does not show diffs on single files so we create a link to the commit
-        return new URL(url, "commit/" + changeSet.getId().toString());    
+        return new URL(url, "commit/" + changeSet.getId().toString() + "/diffs?diffmode=sidebyside&fragment=1#" + path.getPath());
     }
 
     /**

src/test/java/hudson/plugins/git/browser/GitoriousWebTest.java

@@ -67,10 +67,10 @@ public void testGetChangeSetLinkGitChangeSet() throws IOException, SAXException
     public void testGetDiffLinkPath() throws IOException, SAXException {
         final HashMap<String, Path> pathMap = createPathMap("rawchangelog");
         final Path modified1 = pathMap.get("src/main/java/hudson/plugins/git/browser/GithubWeb.java");
-        assertEquals(GITORIOUS_URL + "/commit/396fc230a3db05c427737aa5c2eb7856ba72b05d", gitoriousWeb.getDiffLink(modified1).toString());
+        assertEquals(GITORIOUS_URL + "/commit/396fc230a3db05c427737aa5c2eb7856ba72b05d/diffs?diffmode=sidebyside&fragment=1#src/main/java/hudson/plugins/git/browser/GithubWeb.java", gitoriousWeb.getDiffLink(modified1).toString());
         // For added files returns a link to the commit.
         final Path added = pathMap.get("src/test/resources/hudson/plugins/git/browser/rawchangelog-with-deleted-file");
-        assertEquals(GITORIOUS_URL + "/commit/396fc230a3db05c427737aa5c2eb7856ba72b05d", gitoriousWeb.getDiffLink(added).toString());
+        assertEquals(GITORIOUS_URL + "/commit/396fc230a3db05c427737aa5c2eb7856ba72b05d/diffs?diffmode=sidebyside&fragment=1#src/test/resources/hudson/plugins/git/browser/rawchangelog-with-deleted-file", gitoriousWeb.getDiffLink(added).toString());
     }
 
     /**
@@ -94,7 +94,7 @@ public void testGetFileLinkPathForDeletedFile() throws IOException, SAXException
         final HashMap<String,Path> pathMap = createPathMap("rawchangelog-with-deleted-file");
         final Path path = pathMap.get("bar");
         final URL fileLink = gitoriousWeb.getFileLink(path);
-        assertEquals(GITORIOUS_URL + "/commit/fc029da233f161c65eb06d0f1ed4f36ae81d1f4f", String.valueOf(fileLink));
+        assertEquals(GITORIOUS_URL + "/commit/fc029da233f161c65eb06d0f1ed4f36ae81d1f4f/diffs?diffmode=sidebyside&fragment=1#bar", String.valueOf(fileLink));
     }
 
     private GitChangeSet createChangeSet(String rawchangelogpath) throws IOException, SAXException {