Findbugs. CodePro Audit. Checkstyle. Unit test refactoring.

James Moger

2011-05-26 2a7306a1d92522569a8bb6e5a7c0bcdd5cf4cfaa

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

@@ -33,9 +33,10 @@


   private static final long serialVersionUID = 1L;



   private final String ROOT = "--ROOT--";

   private static final String ROOT = "--ROOT--";



   public PathBreadcrumbsPanel(String id, final String repositoryName, String pathName, final String objectId) {

   public PathBreadcrumbsPanel(String id, final String repositoryName, String pathName,

         final String objectId) {

      super(id);

      List<BreadCrumb> crumbs = new ArrayList<BreadCrumb>();

      crumbs.add(new BreadCrumb("[" + repositoryName + "]", ROOT, false));

@@ -47,8 +48,8 @@
         for (int i = 0; i < paths.length; i++) {

            String path = paths[i];

            sb.append(path);

            crumbs.add(new BreadCrumb(path, sb.toString(), (i == (paths.length - 1))));

            sb.append("/");

            crumbs.add(new BreadCrumb(path, sb.toString(), i == (paths.length - 1)));

            sb.append('/');

         }

      }



@@ -63,7 +64,8 @@
               item.add(new Label("pathLink", entry.name));

               item.add(new Label("pathSeparator", "").setVisible(false));

            } else {

               item.add(new LinkPanel("pathLink", null, entry.name, TreePage.class, WicketUtils.newPathParameter(repositoryName, objectId, path)));

               item.add(new LinkPanel("pathLink", null, entry.name, TreePage.class,

                     WicketUtils.newPathParameter(repositoryName, objectId, path)));

               item.add(new Label("pathSeparator", "/"));

            }

         }

@@ -71,7 +73,7 @@
      add(pathsView);

   }



   private class BreadCrumb implements Serializable {

   private static class BreadCrumb implements Serializable {



      private static final long serialVersionUID = 1L;