James Moger
2012-01-09 d25c599b9967549fe0ebc7a56ca785c317ba3d4b
Documentation. Find bugs. Organized imports.
7 files modified
17 ■■■■■ changed files
docs/05_roadmap.mkd 3 ●●●●● patch | view | raw | blame | history
src/com/gitblit/GitBlit.java 4 ●●● patch | view | raw | blame | history
src/com/gitblit/models/FederationModel.java 2 ●●● patch | view | raw | blame | history
src/com/gitblit/utils/TimeUtils.java 2 ●●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/pages/SummaryPage.java 2 ●●● patch | view | raw | blame | history
src/com/gitblit/wicket/panels/ObjectContainer.java 2 ●●●●● patch | view | raw | blame | history
tests/com/gitblit/tests/GitBlitTest.java 2 ●●● patch | view | raw | blame | history
docs/05_roadmap.mkd
@@ -9,6 +9,7 @@
* 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 @@
### 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.*
src/com/gitblit/GitBlit.java
@@ -1759,9 +1759,7 @@
        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();
        }
src/com/gitblit/models/FederationModel.java
@@ -114,7 +114,7 @@
     * @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) {
src/com/gitblit/utils/TimeUtils.java
@@ -268,8 +268,6 @@
        } else if (frequency.indexOf("hour") > -1) {
            // convert to minutes
            mins *= 60;
        } else if (frequency.indexOf("min") > -1) {
            // default mins
        }
        return mins;
    }
src/com/gitblit/wicket/pages/SummaryPage.java
@@ -37,9 +37,9 @@
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;
src/com/gitblit/wicket/panels/ObjectContainer.java
@@ -32,8 +32,6 @@
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
 */
tests/com/gitblit/tests/GitBlitTest.java
@@ -17,8 +17,8 @@
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;