From 44c005e7c1efb0564d07394ca2e98308b9b48581 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Fri, 27 Jun 2014 23:06:13 -0400
Subject: [PATCH] Use consistent keys for repository cache lookups

---
 src/main/java/com/gitblit/wicket/pages/RepositoryPage.java |   16 ++++++++++------
 1 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java b/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java
index fcf659a..253c4fe 100644
--- a/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java
+++ b/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java
@@ -166,10 +166,10 @@
 		add(navigationPanel);
 
 		add(new ExternalLink("syndication", SyndicationServlet.asLink(getRequest()
-				.getRelativePathPrefixToContextRoot(), repositoryName, null, 0)));
+				.getRelativePathPrefixToContextRoot(), getRepositoryName(), null, 0)));
 
 		// add floating search form
-		SearchForm searchForm = new SearchForm("searchForm", repositoryName);
+		SearchForm searchForm = new SearchForm("searchForm", getRepositoryName());
 		add(searchForm);
 		searchForm.setTranslatedAttributes();
 
@@ -193,7 +193,7 @@
 	private List<NavLink> registerNavLinks() {
 		PageParameters params = null;
 		if (!StringUtils.isEmpty(repositoryName)) {
-			params = WicketUtils.newRepositoryParameter(repositoryName);
+			params = WicketUtils.newRepositoryParameter(getRepositoryName());
 		}
 		List<NavLink> navLinks = new ArrayList<NavLink>();
 
@@ -216,7 +216,7 @@
 		navLinks.add(new PageNavLink("gb.commits", LogPage.class, params));
 		navLinks.add(new PageNavLink("gb.tree", TreePage.class, params));
 		if (app().tickets().isReady() && (app().tickets().isAcceptingNewTickets(model) || app().tickets().hasTickets(model))) {
-			PageParameters tParams = WicketUtils.newOpenTicketsParameter(repositoryName);
+			PageParameters tParams = WicketUtils.newOpenTicketsParameter(getRepositoryName());
 			navLinks.add(new PageNavLink("gb.tickets", TicketsPage.class, tParams));
 		}
 		navLinks.add(new PageNavLink("gb.docs", DocsPage.class, params, true));
@@ -229,7 +229,7 @@
 		// per-repository extra navlinks
 		if (JGitUtils.getPagesBranch(r) != null) {
 			ExternalNavLink pagesLink = new ExternalNavLink("gb.pages", PagesServlet.asLink(
-					getRequest().getRelativePathPrefixToContextRoot(), repositoryName, null), true);
+					getRequest().getRelativePathPrefixToContextRoot(), getRepositoryName(), null), true);
 			navLinks.add(pagesLink);
 		}
 
@@ -420,6 +420,10 @@
 			m = model;
 		}
 		return m;
+	}
+
+	protected String getRepositoryName() {
+		return getRepositoryModel().name;
 	}
 
 	protected RevCommit getCommit() {
@@ -630,7 +634,7 @@
 			r = null;
 		}
 		// setup page header and footer
-		setupPage(repositoryName, "/ " + getPageName());
+		setupPage(getRepositoryName(), "/ " + getPageName());
 		super.onBeforeRender();
 	}
 

--
Gitblit v1.9.1