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/models/RefModel.java |   28 +++++++++++++++++-----------
 1 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/src/com/gitblit/wicket/models/RefModel.java b/src/com/gitblit/wicket/models/RefModel.java
index ab44593..e0831f5 100644
--- a/src/com/gitblit/wicket/models/RefModel.java
+++ b/src/com/gitblit/wicket/models/RefModel.java
@@ -27,9 +27,9 @@
 public class RefModel implements Serializable, Comparable<RefModel> {
 
 	private static final long serialVersionUID = 1L;
-	final String displayName;
-	transient Ref ref;
-	final RevCommit commit;
+	public final String displayName;
+	public final RevCommit commit;
+	public transient Ref ref;
 
 	public RefModel(String displayName, Ref ref, RevCommit commit) {
 		this.displayName = displayName;
@@ -41,16 +41,8 @@
 		return JGitUtils.getCommitDate(commit);
 	}
 
-	public String getDisplayName() {
-		return displayName;
-	}
-
 	public String getName() {
 		return ref.getName();
-	}
-
-	public RevCommit getCommit() {
-		return commit;
 	}
 
 	public ObjectId getCommitId() {
@@ -71,6 +63,20 @@
 	}
 
 	@Override
+	public int hashCode() {
+		return getCommitId().hashCode() + getName().hashCode();
+	}
+
+	@Override
+	public boolean equals(Object o) {
+		if (o instanceof RefModel) {
+			RefModel other = (RefModel) o;
+			return getName().equals(other.getName());
+		}
+		return super.equals(o);
+	}
+
+	@Override
 	public int compareTo(RefModel o) {
 		return getDate().compareTo(o.getDate());
 	}

--
Gitblit v1.9.1