From e8ebf7091ddac501dc796c596ff6ae33a2b0b195 Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Fri, 18 Jan 2013 20:41:31 +0100
Subject: [PATCH] implement disable last commit for git browse command

---
 .../scm/repository/spi/GitBrowseCommand.java      | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/spi/GitBrowseCommand.java b/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/spi/GitBrowseCommand.java
index 3e1dad43ff..a4f040dc7d 100644
--- a/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/spi/GitBrowseCommand.java
+++ b/scm-plugins/scm-git-plugin/src/main/java/sonia/scm/repository/spi/GitBrowseCommand.java
@@ -143,7 +143,7 @@ public class GitBrowseCommand extends AbstractGitCommand
 
     if (revId != null)
     {
-      result = getResult(repo, revId, request.getPath());
+      result = getResult(repo, request, revId, request.getPath());
     }
     else
     {
@@ -240,7 +240,7 @@ public class GitBrowseCommand extends AbstractGitCommand
    *
    * @throws IOException
    */
-  private FileObject createFileObject(org.eclipse.jgit.lib.Repository repo,
+  private FileObject createFileObject(org.eclipse.jgit.lib.Repository repo, BrowseCommandRequest request,
     ObjectId revId, TreeWalk treeWalk)
     throws IOException
   {
@@ -261,8 +261,9 @@ public class GitBrowseCommand extends AbstractGitCommand
       file.setLength(loader.getSize());
 
       // don't show message and date for directories to improve performance
-      if (!file.isDirectory())
+      if (!file.isDirectory() && ! request.isDisableLastCommit())
       {
+        logger.trace("fetch last commit for {} at {}", path, revId.getName());
         RevCommit commit = getLatestCommit(repo, revId, path);
 
         if (commit != null)
@@ -345,8 +346,8 @@ public class GitBrowseCommand extends AbstractGitCommand
    * @throws IOException
    * @throws RepositoryException
    */
-  private BrowserResult getResult(org.eclipse.jgit.lib.Repository repo,
-    ObjectId revId, String path)
+  private BrowserResult getResult(org.eclipse.jgit.lib.Repository repo, 
+    BrowseCommandRequest request, ObjectId revId, String path)
     throws IOException, RepositoryException
   {
     BrowserResult result = null;
@@ -384,7 +385,7 @@ public class GitBrowseCommand extends AbstractGitCommand
       {
         while (treeWalk.next())
         {
-          FileObject fo = createFileObject(repo, revId, treeWalk);
+          FileObject fo = createFileObject(repo, request, revId, treeWalk);
 
           if (fo != null)
           {
@@ -408,7 +409,7 @@ public class GitBrowseCommand extends AbstractGitCommand
 
             if (p.split("/").length > limit)
             {
-              FileObject fo = createFileObject(repo, revId, treeWalk);
+              FileObject fo = createFileObject(repo, request, revId, treeWalk);
 
               if (fo != null)
               {