James Moger
2012-01-09 cc1cd7800940f98efe206fe50a4375a3fb9dc621
Refactored named branch retrieval function
2 files modified
55 ■■■■■ changed files
src/com/gitblit/utils/JGitUtils.java 31 ●●●●● patch | view | raw | blame | history
src/com/gitblit/utils/TicgitUtils.java 24 ●●●●● patch | view | raw | blame | history
src/com/gitblit/utils/JGitUtils.java
@@ -1291,29 +1291,40 @@
     * @return a refmodel for the gh-pages branch or null
     */
    public static RefModel getPagesBranch(Repository repository) {
        RefModel ghPages = null;
        return getBranch(repository, "gh-pages");
    }
    /**
     * Returns a RefModel for a specific branch name in the repository. If the
     * branch can not be found, null is returned.
     *
     * @param repository
     * @return a refmodel for the branch or null
     */
    public static RefModel getBranch(Repository repository, String name) {
        RefModel branch = null;
        try {
            // search for gh-pages branch in local heads
            // search for the branch in local heads
            for (RefModel ref : JGitUtils.getLocalBranches(repository, false, -1)) {
                if (ref.displayName.endsWith("gh-pages")) {
                    ghPages = ref;
                if (ref.displayName.endsWith(name)) {
                    branch = ref;
                    break;
                }
            }
            // search for gh-pages branch in remote heads
            if (ghPages == null) {
            // search for the branch in remote heads
            if (branch == null) {
                for (RefModel ref : JGitUtils.getRemoteBranches(repository, false, -1)) {
                    if (ref.displayName.endsWith("gh-pages")) {
                        ghPages = ref;
                    if (ref.displayName.endsWith(name)) {
                        branch = ref;
                        break;
                    }
                }
            }
        } catch (Throwable t) {
            LOGGER.error("Failed to find gh-pages branch!", t);
            LOGGER.error(MessageFormat.format("Failed to find {0} branch!", name), t);
        }
        return ghPages;
        return branch;
    }
    /**
src/com/gitblit/utils/TicgitUtils.java
@@ -48,29 +48,7 @@
     * @return a refmodel for the ticgit branch or null
     */
    public static RefModel getTicketsBranch(Repository repository) {
        RefModel ticgitBranch = null;
        try {
            // search for ticgit branch in local heads
            for (RefModel ref : JGitUtils.getLocalBranches(repository, false, -1)) {
                if (ref.displayName.endsWith("ticgit")) {
                    ticgitBranch = ref;
                    break;
                }
            }
            // search for ticgit branch in remote heads
            if (ticgitBranch == null) {
                for (RefModel ref : JGitUtils.getRemoteBranches(repository, false, -1)) {
                    if (ref.displayName.endsWith("ticgit")) {
                        ticgitBranch = ref;
                        break;
                    }
                }
            }
        } catch (Throwable t) {
            LOGGER.error("Failed to find ticgit branch!", t);
        }
        return ticgitBranch;
        return JGitUtils.getBranch(repository, "ticgit");
    }
    /**