summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJames Moger <james.moger@gitblit.com>2012-01-09 21:07:01 -0500
committerJames Moger <james.moger@gitblit.com>2012-01-09 21:07:01 -0500
commitd25c599b9967549fe0ebc7a56ca785c317ba3d4b (patch)
tree4bb9a4462f538ce44babc5d427da626cf6cbaded
parent11924dc5db4bc44cb32e905700a8557124b1fd56 (diff)
downloadgitblit-d25c599b9967549fe0ebc7a56ca785c317ba3d4b.tar.gz
gitblit-d25c599b9967549fe0ebc7a56ca785c317ba3d4b.zip
Documentation. Find bugs. Organized imports.
-rw-r--r--docs/05_roadmap.mkd3
-rw-r--r--src/com/gitblit/GitBlit.java4
-rw-r--r--src/com/gitblit/models/FederationModel.java2
-rw-r--r--src/com/gitblit/utils/TimeUtils.java2
-rw-r--r--src/com/gitblit/wicket/pages/SummaryPage.java2
-rw-r--r--src/com/gitblit/wicket/panels/ObjectContainer.java2
-rw-r--r--tests/com/gitblit/tests/GitBlitTest.java2
7 files changed, 7 insertions, 10 deletions
diff --git a/docs/05_roadmap.mkd b/docs/05_roadmap.mkd
index c1581551..86888314 100644
--- a/docs/05_roadmap.mkd
+++ b/docs/05_roadmap.mkd
@@ -9,6 +9,7 @@ This list is volatile.
* Manager: support federation RPCs
* Manager: redesign ref indicators in log, search, and activity views to support multiple local branches, remote branches, and tags
* Gitblit: Lucene integration with multi-repository search (issue 16)
+* Gitblit: Serve repositories on root URL rather than /git (investigate JGit 1.2 GitFilter)
### TODO (medium priority)
@@ -31,6 +32,8 @@ This list is volatile.
### IDEAS
+* Gitblit: Re-use the EGit branch visualization table cell renderer as some sort of servlet
+* Gitblit: Support personal repositories (~username/repo)
* Gitblit: diff should highlight inserted/removed fragment compared to original line
* Gitblit: implement branch permission controls as Groovy pre-receive script.
*Maintain permissions text file similar to a gitolite configuration file or svn authz file.*
diff --git a/src/com/gitblit/GitBlit.java b/src/com/gitblit/GitBlit.java
index 2448f7ae..bf3660d0 100644
--- a/src/com/gitblit/GitBlit.java
+++ b/src/com/gitblit/GitBlit.java
@@ -1759,9 +1759,7 @@ public class GitBlit implements ServletContextListener {
try {
// check to see if this "file" is a login service class
Class<?> realmClass = Class.forName(realm);
- if (IUserService.class.isAssignableFrom(realmClass)) {
- loginService = (IUserService) realmClass.newInstance();
- }
+ loginService = (IUserService) realmClass.newInstance();
} catch (Throwable t) {
loginService = new GitblitUserService();
}
diff --git a/src/com/gitblit/models/FederationModel.java b/src/com/gitblit/models/FederationModel.java
index 61c8576b..1d211ce9 100644
--- a/src/com/gitblit/models/FederationModel.java
+++ b/src/com/gitblit/models/FederationModel.java
@@ -114,7 +114,7 @@ public class FederationModel implements Serializable, Comparable<FederationModel
* @param status
*/
public void updateStatus(RepositoryModel repository, FederationPullStatus status) {
- if (!results.containsKey(repository)) {
+ if (!results.containsKey(repository.name)) {
results.put(repository.name, FederationPullStatus.PENDING);
}
if (status != null) {
diff --git a/src/com/gitblit/utils/TimeUtils.java b/src/com/gitblit/utils/TimeUtils.java
index 6cc4dcb5..cd8e8956 100644
--- a/src/com/gitblit/utils/TimeUtils.java
+++ b/src/com/gitblit/utils/TimeUtils.java
@@ -268,8 +268,6 @@ public class TimeUtils {
} else if (frequency.indexOf("hour") > -1) {
// convert to minutes
mins *= 60;
- } else if (frequency.indexOf("min") > -1) {
- // default mins
}
return mins;
}
diff --git a/src/com/gitblit/wicket/pages/SummaryPage.java b/src/com/gitblit/wicket/pages/SummaryPage.java
index d928b06d..904ec44d 100644
--- a/src/com/gitblit/wicket/pages/SummaryPage.java
+++ b/src/com/gitblit/wicket/pages/SummaryPage.java
@@ -37,9 +37,9 @@ import org.wicketstuff.googlecharts.LineStyle;
import org.wicketstuff.googlecharts.MarkerType;
import org.wicketstuff.googlecharts.ShapeMarker;
+import com.gitblit.Constants.AccessRestrictionType;
import com.gitblit.GitBlit;
import com.gitblit.Keys;
-import com.gitblit.Constants.AccessRestrictionType;
import com.gitblit.models.Metric;
import com.gitblit.models.PathModel;
import com.gitblit.models.RepositoryModel;
diff --git a/src/com/gitblit/wicket/panels/ObjectContainer.java b/src/com/gitblit/wicket/panels/ObjectContainer.java
index 79bd3a7c..d7f1f789 100644
--- a/src/com/gitblit/wicket/panels/ObjectContainer.java
+++ b/src/com/gitblit/wicket/panels/ObjectContainer.java
@@ -32,8 +32,6 @@ import org.apache.wicket.protocol.http.request.WebClientInfo;
import org.apache.wicket.request.ClientInfo;
import org.apache.wicket.util.value.IValueMap;
-import com.gitblit.wicket.WicketUtils;
-
/**
* https://cwiki.apache.org/WICKET/object-container-adding-flash-to-a-wicket-application.html
*/
diff --git a/tests/com/gitblit/tests/GitBlitTest.java b/tests/com/gitblit/tests/GitBlitTest.java
index 28603f87..418f9384 100644
--- a/tests/com/gitblit/tests/GitBlitTest.java
+++ b/tests/com/gitblit/tests/GitBlitTest.java
@@ -17,8 +17,8 @@ package com.gitblit.tests;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
import java.util.List;