Improve history display of a submodule link

James Moger

2013-01-13 657a6596eb95635abd29c0a21befffc43da49d09

Improve history display of a submodule link

 docs/04_releases.mkd

@@ -30,6 +30,7 @@


#### changes



- Improve history display of a submodule link

- Updated Korean translation (github/ds5apn)

- Updated checkstyle definition (github/mystygage)




 src/com/gitblit/utils/JGitUtils.java

@@ -1616,6 +1616,32 @@
        }

        return null;

    }

	
    public static String getSubmoduleCommitId(Repository repository, String path, RevCommit commit) {

        String commitId = null;

        RevWalk rw = new RevWalk(repository);

        TreeWalk tw = new TreeWalk(repository);

        tw.setFilter(PathFilterGroup.createFromStrings(Collections.singleton(path)));

        try {

            tw.reset(commit.getTree());

            while (tw.next()) {

                if (tw.isSubtree() && !path.equals(tw.getPathString())) {

                    tw.enterSubtree();

                    continue;

                }

                if (FileMode.GITLINK == tw.getFileMode(0)) {

                    commitId = tw.getObjectId(0).getName();

                    break;

                }

            }

        } catch (Throwable t) {

            error(t, repository, "{0} can't find {1} in commit {2}", path, commit.name());

        } finally {

            rw.dispose();

            tw.release();

        }

        return commitId;

    }



    /**

     * Returns the list of notes entered about the commit from the refs/notes


 src/com/gitblit/wicket/panels/HistoryPanel.java

@@ -15,10 +15,14 @@
 */

package com.gitblit.wicket.panels;



import java.util.ArrayList;

import java.util.Collections;

import java.util.Date;

import java.util.HashMap;

import java.util.LinkedHashSet;

import java.util.List;

import java.util.Map;

import java.util.Set;



import org.apache.wicket.markup.html.basic.Label;

import org.apache.wicket.markup.html.link.BookmarkablePageLink;

@@ -38,6 +42,7 @@
import com.gitblit.GitBlit;

import com.gitblit.Keys;

import com.gitblit.models.PathModel;

import com.gitblit.models.SubmoduleModel;

import com.gitblit.models.PathModel.PathChangeModel;

import com.gitblit.models.RefModel;

import com.gitblit.utils.JGitUtils;

@@ -69,6 +74,11 @@
        RevCommit commit = JGitUtils.getCommit(r, objectId);

        List<PathChangeModel> paths = JGitUtils.getFilesInCommit(r, commit);



        Map<String, SubmoduleModel> submodules = new HashMap<String, SubmoduleModel>();

        for (SubmoduleModel model : JGitUtils.getSubmodules(r, commit.getTree())) {

            submodules.put(model.path, model);

        }



        PathModel matchingPath = null;

        for (PathModel p : paths) {

            if (p.path.equals(path)) {

@@ -99,7 +109,20 @@
        }

        

        final boolean isTree = matchingPath == null ? true : matchingPath.isTree();

        final boolean isSubmodule = matchingPath == null ? true : matchingPath.isSubmodule();



        // submodule

        SubmoduleModel submodule = getSubmodule(submodules, repositoryName, matchingPath.path);

        final String submodulePath;

        final boolean hasSubmodule; 
        if (submodule != null) {

            submodulePath = submodule.gitblitPath;

            hasSubmodule = submodule.hasSubmodule;

        } else {

            submodulePath = "";

            hasSubmodule = false;

        }

		
        final Map<ObjectId, List<RefModel>> allRefs = JGitUtils.getAllRefs(r, showRemoteRefs);

        List<RevCommit> commits;

        if (pageResults) {

@@ -179,6 +202,23 @@
                    links.add(new BookmarkablePageLink<Void>("commitdiff", CommitDiffPage.class,

                            WicketUtils.newObjectParameter(repositoryName, entry.getName())));

                    item.add(links);

                } else if (isSubmodule) {

                    // submodule

                    item.add(new Label("hashLabel", submodulePath + "@"));

                    Repository repository = GitBlit.self().getRepository(repositoryName);

                    String submoduleId = JGitUtils.getSubmoduleCommitId(repository, path, entry);

                    repository.close();

                    LinkPanel commitHash = new LinkPanel("hashLink", null, submoduleId.substring(0, hashLen),

                            TreePage.class, WicketUtils.newObjectParameter(

                                    submodulePath, submoduleId));

                    WicketUtils.setCssClass(commitHash, "shortsha1");

                    WicketUtils.setHtmlTooltip(commitHash, submoduleId);					
                    item.add(commitHash.setEnabled(hasSubmodule));

					
                    Fragment links = new Fragment("historyLinks", "treeLinks", this);

                    links.add(new BookmarkablePageLink<Void>("commitdiff", CommitDiffPage.class,

                            WicketUtils.newObjectParameter(repositoryName, entry.getName())));

                    item.add(links);

                } else {                    

                    // commit

                    item.add(new Label("hashLabel", getString("gb.blob") + "@"));

@@ -230,4 +270,66 @@
    public boolean hasMore() {

        return hasMore;

    }

	
    protected SubmoduleModel getSubmodule(Map<String, SubmoduleModel> submodules, String repositoryName, String path) {

        SubmoduleModel model = submodules.get(path);

        if (model == null) {

            // undefined submodule?!

            model = new SubmoduleModel(path.substring(path.lastIndexOf('/') + 1), path, path);

            model.hasSubmodule = false;

            model.gitblitPath = model.name;

            return model;

        } else {

            // extract the repository name from the clone url

            List<String> patterns = GitBlit.getStrings(Keys.git.submoduleUrlPatterns);

            String submoduleName = StringUtils.extractRepositoryPath(model.url, patterns.toArray(new String[0]));

			
            // determine the current path for constructing paths relative

            // to the current repository

            String currentPath = "";

            if (repositoryName.indexOf('/') > -1) {

                currentPath = repositoryName.substring(0, repositoryName.lastIndexOf('/') + 1);

            }



            // try to locate the submodule repository

            // prefer bare to non-bare names

            List<String> candidates = new ArrayList<String>();



            // relative

            candidates.add(currentPath + StringUtils.stripDotGit(submoduleName));

            candidates.add(candidates.get(candidates.size() - 1) + ".git");



            // relative, no subfolder

            if (submoduleName.lastIndexOf('/') > -1) {

                String name = submoduleName.substring(submoduleName.lastIndexOf('/') + 1);

                candidates.add(currentPath + StringUtils.stripDotGit(name));

                candidates.add(currentPath + candidates.get(candidates.size() - 1) + ".git");

            }



            // absolute

            candidates.add(StringUtils.stripDotGit(submoduleName));

            candidates.add(candidates.get(candidates.size() - 1) + ".git");



            // absolute, no subfolder

            if (submoduleName.lastIndexOf('/') > -1) {

                String name = submoduleName.substring(submoduleName.lastIndexOf('/') + 1);

                candidates.add(StringUtils.stripDotGit(name));

                candidates.add(candidates.get(candidates.size() - 1) + ".git");

            }



            // create a unique, ordered set of candidate paths

            Set<String> paths = new LinkedHashSet<String>(candidates);

            for (String candidate : paths) {

                if (GitBlit.self().hasRepository(candidate)) {

                    model.hasSubmodule = true;

                    model.gitblitPath = candidate;

                    return model;

                }

            }

			
            // we do not have a copy of the submodule, but we need a path

            model.gitblitPath = candidates.get(0);

            return model;

        }		
    }

}