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