From 80d636ab48d42a7298624fe19cb0638ea9fcd564 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 23 May 2013 23:46:09 -0400
Subject: [PATCH] Fixed submodule npe in compare page

---
 src/main/java/com/gitblit/wicket/pages/RepositoryPage.java |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java b/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java
index 8a233ea..072475a 100644
--- a/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java
+++ b/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);

--
Gitblit v1.9.1