From 2a7306a1d92522569a8bb6e5a7c0bcdd5cf4cfaa Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 26 May 2011 17:11:38 -0400
Subject: [PATCH] Findbugs. CodePro Audit. Checkstyle. Unit test refactoring.

---
 src/com/gitblit/wicket/panels/PathBreadcrumbsPanel.java |   14 ++++++++------
 1 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/com/gitblit/wicket/panels/PathBreadcrumbsPanel.java b/src/com/gitblit/wicket/panels/PathBreadcrumbsPanel.java
index d4ce442..2693cc2 100644
--- a/src/com/gitblit/wicket/panels/PathBreadcrumbsPanel.java
+++ b/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;
 

--
Gitblit v1.9.1