Explicitly declare RepositoryPage subclasses that reference commits

James Moger

2014-09-17 5d5e557f029aa9efe8e81cbc43e9e969fa750b80

Explicitly declare RepositoryPage subclasses that reference commits

 src/main/java/com/gitblit/wicket/GitBlitWebApp.java

@@ -175,9 +175,9 @@

        // setup the standard gitweb-ish urls
        mount("/repositories", RepositoriesPage.class);
        mount("/overview", OverviewPage.class, "r", "h");
        mount("/overview", OverviewPage.class, "r");
        mount("/summary", SummaryPage.class, "r");
        mount("/reflog", ReflogPage.class, "r", "h");
        mount("/reflog", ReflogPage.class, "r");
        mount("/commits", LogPage.class, "r", "h");
        mount("/log", LogPage.class, "r", "h");
        mount("/tags", TagsPage.class, "r");

 src/main/java/com/gitblit/wicket/pages/BlamePage.java

@@ -235,6 +235,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return TreePage.class;

    }


 src/main/java/com/gitblit/wicket/pages/BlobDiffPage.java

@@ -82,6 +82,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return TreePage.class;

    }


 src/main/java/com/gitblit/wicket/pages/BlobPage.java

@@ -224,6 +224,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return TreePage.class;

    }


 src/main/java/com/gitblit/wicket/pages/CommitDiffPage.java

@@ -197,6 +197,11 @@
    }

    @Override
    protected boolean isCommitPage() {
        return true;
    }

    @Override
    protected Class<? extends BasePage> getRepoNavPageClass() {
        return LogPage.class;
    }

 src/main/java/com/gitblit/wicket/pages/CommitPage.java

@@ -248,6 +248,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return LogPage.class;

    }


 src/main/java/com/gitblit/wicket/pages/DocPage.java

@@ -102,6 +102,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return DocsPage.class;

    }


 src/main/java/com/gitblit/wicket/pages/DocsPage.java

@@ -167,4 +167,10 @@
    protected String getPageName() {

        return getString("gb.docs");

    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



}


 src/main/java/com/gitblit/wicket/pages/HistoryPage.java

@@ -67,6 +67,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return TreePage.class;

    }


 src/main/java/com/gitblit/wicket/pages/LogPage.java

@@ -69,4 +69,10 @@
    protected String getPageName() {

        return getString("gb.log");

    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



}


 src/main/java/com/gitblit/wicket/pages/MetricsPage.java

@@ -176,6 +176,11 @@
    @Override

    protected String getPageName() {

        return getString("gb.metrics");

    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override


 src/main/java/com/gitblit/wicket/pages/NoDocsPage.java

@@ -33,6 +33,11 @@
    }

    @Override
    protected boolean isCommitPage() {
        return true;
    }

    @Override
    protected Class<? extends BasePage> getRepoNavPageClass() {
        return DocsPage.class;
    }

 src/main/java/com/gitblit/wicket/pages/RepositoryPage.java

@@ -201,11 +201,13 @@
            objectParams = params;



            // preserve the objectid iff the objectid directly (or indirectly) refers to a ref

            if (!StringUtils.isEmpty(objectId)) {

            if (isCommitPage() && !StringUtils.isEmpty(objectId)) {

                RevCommit commit = JGitUtils.getCommit(r, objectId);

                if (commit != null) {

                String bestId = getBestCommitId(commit);

                if (!commit.getName().equals(bestId)) {

                    objectParams = WicketUtils.newObjectParameter(getRepositoryName(), bestId);

                    }

                }

            }

        }

@@ -574,6 +576,10 @@


    protected abstract String getPageName();



    protected boolean isCommitPage() {

        return false;

    }



    protected Component createPersonPanel(String wicketId, PersonIdent identity,

            Constants.SearchType searchType) {

        String name = identity == null ? "" : identity.getName();


 src/main/java/com/gitblit/wicket/pages/TagPage.java

@@ -101,6 +101,11 @@
    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



    @Override

    protected Class<? extends BasePage> getRepoNavPageClass() {

        return LogPage.class;

    }


 src/main/java/com/gitblit/wicket/pages/TreePage.java

@@ -186,4 +186,10 @@
    protected String getPageName() {

        return getString("gb.tree");

    }



    @Override

    protected boolean isCommitPage() {

        return true;

    }



}