Findbugs. CodePro Audit. Checkstyle. Unit test refactoring.

James Moger

2011-05-26 2a7306a1d92522569a8bb6e5a7c0bcdd5cf4cfaa

 src/com/gitblit/wicket/pages/LogPage.java

@@ -30,17 +30,30 @@
      int pageNumber = WicketUtils.getPage(params);

      int prevPage = Math.max(0, pageNumber - 1);

      int nextPage = pageNumber + 1;

      LogPanel logPanel = new LogPanel("logPanel", repositoryName, objectId, getRepository(), -1, pageNumber - 1);

      LogPanel logPanel = new LogPanel("logPanel", repositoryName, objectId, getRepository(), -1,

            pageNumber - 1);

      boolean hasMore = logPanel.hasMore();

      add(logPanel);



      add(new BookmarkablePageLink<Void>("firstPageTop", LogPage.class, WicketUtils.newObjectParameter(repositoryName, objectId)).setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("prevPageTop", LogPage.class, WicketUtils.newLogPageParameter(repositoryName, objectId, prevPage)).setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("nextPageTop", LogPage.class, WicketUtils.newLogPageParameter(repositoryName, objectId, nextPage)).setEnabled(hasMore));

      add(new BookmarkablePageLink<Void>("firstPageTop", LogPage.class,

            WicketUtils.newObjectParameter(repositoryName, objectId))

            .setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("prevPageTop", LogPage.class,

            WicketUtils.newLogPageParameter(repositoryName, objectId, prevPage))

            .setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("nextPageTop", LogPage.class,

            WicketUtils.newLogPageParameter(repositoryName, objectId, nextPage))

            .setEnabled(hasMore));



      add(new BookmarkablePageLink<Void>("firstPageBottom", LogPage.class, WicketUtils.newObjectParameter(repositoryName, objectId)).setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("prevPageBottom", LogPage.class, WicketUtils.newLogPageParameter(repositoryName, objectId, prevPage)).setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("nextPageBottom", LogPage.class, WicketUtils.newLogPageParameter(repositoryName, objectId, nextPage)).setEnabled(hasMore));

      add(new BookmarkablePageLink<Void>("firstPageBottom", LogPage.class,

            WicketUtils.newObjectParameter(repositoryName, objectId))

            .setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("prevPageBottom", LogPage.class,

            WicketUtils.newLogPageParameter(repositoryName, objectId, prevPage))

            .setEnabled(pageNumber > 1));

      add(new BookmarkablePageLink<Void>("nextPageBottom", LogPage.class,

            WicketUtils.newLogPageParameter(repositoryName, objectId, nextPage))

            .setEnabled(hasMore));

   }



   @Override