Add DiffComparator argument to all DiffUtils calls, default to ignore all w...

James Moger

2015-02-26 cff35236ba16e2c376c59feeff310f62554a4140

 src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java

@@ -31,11 +31,13 @@
import org.eclipse.jgit.revwalk.RevCommit;

import com.gitblit.Constants;
import com.gitblit.Keys;
import com.gitblit.models.GitNote;
import com.gitblit.models.PathModel.PathChangeModel;
import com.gitblit.models.SubmoduleModel;
import com.gitblit.servlet.RawServlet;
import com.gitblit.utils.DiffUtils;
import com.gitblit.utils.DiffUtils.DiffComparator;
import com.gitblit.utils.DiffUtils.DiffOutput;
import com.gitblit.utils.DiffUtils.DiffOutputType;
import com.gitblit.utils.JGitUtils;
@@ -59,8 +61,6 @@

      RevCommit commit = getCommit();

      final DiffOutput diff = DiffUtils.getCommitDiff(r, commit, DiffOutputType.HTML);

      List<String> parents = new ArrayList<String>();
      if (commit.getParentCount() > 0) {
         for (RevCommit parent : commit.getParents()) {
@@ -81,6 +81,16 @@
            WicketUtils.newObjectParameter(repositoryName, objectId)));

      add(new CommitHeaderPanel("commitHeader", repositoryName, commit));

      final List<String> imageExtensions = app().settings().getStrings(Keys.web.imageExtensions);
      final ImageDiffHandler handler = new ImageDiffHandler(this, repositoryName,
            parents.isEmpty() ? null : parents.get(0), commit.getName(), imageExtensions);

      final DiffComparator diffComparator = DiffComparator.WS_IGNORE_ALL;
      final DiffOutput diff = DiffUtils.getCommitDiff(r, commit, diffComparator, DiffOutputType.HTML, handler);
      if (handler.getImgDiffCount() > 0) {
         addBottomScript("scripts/imgdiff.js"); // Tiny support script for image diffs
      }

      // add commit diffstat
      int insertions = 0;
@@ -145,10 +155,10 @@
               hasSubmodule = submodule.hasSubmodule;

               // add relative link
               item.add(new LinkPanel("pathName", "list", entry.path + " @ " + getShortObjectId(submoduleId), "#" + entry.path));
               item.add(new LinkPanel("pathName", "list", entry.path + " @ " + getShortObjectId(submoduleId), "#n" + entry.objectId));
            } else {
               // add relative link
               item.add(new LinkPanel("pathName", "list", entry.path, "#" + entry.path));
               item.add(new LinkPanel("pathName", "list", entry.path, "#n" + entry.objectId));
            }

            // quick links
@@ -197,6 +207,11 @@
   }

   @Override
   protected boolean isCommitPage() {
      return true;
   }

   @Override
   protected Class<? extends BasePage> getRepoNavPageClass() {
      return LogPage.class;
   }