Fixed submodule npe in compare page

James Moger

2013-05-23 80d636ab48d42a7298624fe19cb0638ea9fcd564

Fixed submodule npe in compare page

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

@@ -103,6 +103,9 @@
                toCommit = getCommit(r, toCommitId.getObject());
            }

            // prepare submodules
            getSubmodules(toCommit);
			
            final String startId = fromCommit.getId().getName();
            final String endId = toCommit.getId().getName();


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

@@ -339,7 +339,7 @@
        return commit;

    }

    

    private Map<String, SubmoduleModel> getSubmodules(RevCommit commit) {	
    protected Map<String, SubmoduleModel> getSubmodules(RevCommit commit) {	
        if (submodules == null) {

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

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

@@ -350,7 +350,10 @@
    }

    

    protected SubmoduleModel getSubmodule(String path) {

        SubmoduleModel model = submodules.get(path);

        SubmoduleModel model = null;

        if (submodules != null) {

            model = submodules.get(path);

        }

        if (model == null) {

            // undefined submodule?!

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