Blame support finished, requires JGit 1.0.0. Checkstyle. Findbugs.

James Moger

2011-06-08 008322bec70a3a20bd00ed2219215a9f42fe0ca5

 src/com/gitblit/wicket/panels/SearchPanel.java

@@ -1,3 +1,18 @@
/*

 * Copyright 2011 gitblit.com.

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *     http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.gitblit.wicket.panels;



import java.util.Date;

@@ -14,10 +29,10 @@


import com.gitblit.GitBlit;

import com.gitblit.Keys;

import com.gitblit.models.RefModel;

import com.gitblit.utils.JGitUtils;

import com.gitblit.utils.JGitUtils.SearchType;

import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.LinkPanel;

import com.gitblit.wicket.WicketUtils;

import com.gitblit.wicket.pages.CommitDiffPage;

import com.gitblit.wicket.pages.CommitPage;

@@ -28,23 +43,25 @@


   private static final long serialVersionUID = 1L;



   private boolean hasMore = false;

   private boolean hasMore;



   public SearchPanel(String wicketId, final String repositoryName, final String objectId, final String value, SearchType searchType, Repository r, int limit, int pageOffset) {

   public SearchPanel(String wicketId, final String repositoryName, final String objectId,

         final String value, SearchType searchType, Repository r, int limit, int pageOffset) {

      super(wicketId);

      boolean pageResults = limit <= 0;

      int itemsPerPage = GitBlit.self().settings().getInteger(Keys.web.itemsPerPage, 50);

      int itemsPerPage = GitBlit.getInteger(Keys.web.itemsPerPage, 50);

      if (itemsPerPage <= 1) {

         itemsPerPage = 50;

      }



      RevCommit commit = JGitUtils.getCommit(r, objectId);



      final Map<ObjectId, List<String>> allRefs = JGitUtils.getAllRefs(r);

      final Map<ObjectId, List<RefModel>> allRefs = JGitUtils.getAllRefs(r);

      List<RevCommit> commits;

      if (pageResults) {

         // Paging result set

         commits = JGitUtils.searchRevlogs(r, objectId, value, searchType, pageOffset * itemsPerPage, itemsPerPage);

         commits = JGitUtils.searchRevlogs(r, objectId, value, searchType, pageOffset

               * itemsPerPage, itemsPerPage);

      } else {

         // Fixed size result set

         commits = JGitUtils.searchRevlogs(r, objectId, value, searchType, 0, limit);

@@ -55,12 +72,14 @@
      hasMore = commits.size() >= itemsPerPage;



      // header

      add(new LinkPanel("header", "title", commit.getShortMessage(), CommitPage.class, WicketUtils.newObjectParameter(repositoryName, commit.getName())));

      add(new LinkPanel("header", "title", commit == null ? "" : commit.getShortMessage(),

            CommitPage.class, WicketUtils.newObjectParameter(repositoryName,

                  commit == null ? "" : commit.getName())));



      ListDataProvider<RevCommit> dp = new ListDataProvider<RevCommit>(commits);

      DataView<RevCommit> searchView = new DataView<RevCommit>("commit", dp) {

         private static final long serialVersionUID = 1L;

         int counter = 0;

         int counter;



         public void populateItem(final Item<RevCommit> item) {

            final RevCommit entry = item.getModelObject();

@@ -70,7 +89,9 @@


            // author search link

            String author = entry.getAuthorIdent().getName();

            LinkPanel authorLink = new LinkPanel("commitAuthor", "list", author, SearchPage.class, WicketUtils.newSearchParameter(repositoryName, objectId, author, SearchType.AUTHOR));

            LinkPanel authorLink = new LinkPanel("commitAuthor", "list", author,

                  SearchPage.class, WicketUtils.newSearchParameter(repositoryName, objectId,

                        author, SearchType.AUTHOR));

            setPersonSearchTooltip(authorLink, author, SearchType.AUTHOR);

            item.add(authorLink);



@@ -83,8 +104,9 @@


            String shortMessage = entry.getShortMessage();

            String trimmedMessage = StringUtils.trimShortLog(shortMessage);

            // TODO highlight matches

            LinkPanel shortlog = new LinkPanel("commitShortMessage", "list subject", trimmedMessage, CommitPage.class, WicketUtils.newObjectParameter(repositoryName, entry.getName()));

            LinkPanel shortlog = new LinkPanel("commitShortMessage", "list subject",

                  trimmedMessage, CommitPage.class, WicketUtils.newObjectParameter(

                        repositoryName, entry.getName()));

            if (!shortMessage.equals(trimmedMessage)) {

               WicketUtils.setHtmlTooltip(shortlog, shortMessage);

            }

@@ -92,9 +114,12 @@


            item.add(new RefsPanel("commitRefs", repositoryName, entry, allRefs));



            item.add(new BookmarkablePageLink<Void>("commit", CommitPage.class, WicketUtils.newObjectParameter(repositoryName, entry.getName())));

            item.add(new BookmarkablePageLink<Void>("commitdiff", CommitDiffPage.class, WicketUtils.newObjectParameter(repositoryName, entry.getName())));

            item.add(new BookmarkablePageLink<Void>("tree", TreePage.class, WicketUtils.newObjectParameter(repositoryName, entry.getName())));

            item.add(new BookmarkablePageLink<Void>("commit", CommitPage.class, WicketUtils

                  .newObjectParameter(repositoryName, entry.getName())));

            item.add(new BookmarkablePageLink<Void>("commitdiff", CommitDiffPage.class,

                  WicketUtils.newObjectParameter(repositoryName, entry.getName())));

            item.add(new BookmarkablePageLink<Void>("tree", TreePage.class, WicketUtils

                  .newObjectParameter(repositoryName, entry.getName())));



            WicketUtils.setAlternatingBackground(item, counter);

            counter++;