From ed21d23885c2f112e5c5650e9436870b1ac99392 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Tue, 05 Jul 2011 17:32:45 -0400
Subject: [PATCH] Documentation. Added JavaDoc. Clarified repository access restriction.

---
 src/com/gitblit/wicket/pages/SummaryPage.java |   33 ++++++++-------------------------
 1 files changed, 8 insertions(+), 25 deletions(-)

diff --git a/src/com/gitblit/wicket/pages/SummaryPage.java b/src/com/gitblit/wicket/pages/SummaryPage.java
index e85901a..e753e5d 100644
--- a/src/com/gitblit/wicket/pages/SummaryPage.java
+++ b/src/com/gitblit/wicket/pages/SummaryPage.java
@@ -22,12 +22,9 @@
 import java.util.ArrayList;
 import java.util.List;
 
-import javax.servlet.http.HttpServletRequest;
-
 import org.apache.wicket.PageParameters;
 import org.apache.wicket.markup.html.basic.Label;
 import org.apache.wicket.markup.html.link.BookmarkablePageLink;
-import org.apache.wicket.protocol.http.WebRequest;
 import org.eclipse.jgit.lib.Repository;
 import org.eclipse.jgit.revwalk.RevCommit;
 import org.wicketstuff.googlecharts.Chart;
@@ -61,18 +58,11 @@
 	public SummaryPage(PageParameters params) {
 		super(params);
 
-		int numCommitsDef = 20;
-		int numRefsDef = 5;
-
-		int numberCommits = GitBlit.getInteger(Keys.web.summaryCommitCount, numCommitsDef);
+		int numberCommits = GitBlit.getInteger(Keys.web.summaryCommitCount, 20);
 		if (numberCommits <= 0) {
-			numberCommits = numCommitsDef;
+			numberCommits = 20;
 		}
-
-		int numberRefs = GitBlit.getInteger(Keys.web.summaryRefsCount, numRefsDef);
-		if (numberRefs <= 0) {
-			numberRefs = numRefsDef;
-		}
+		int numberRefs = GitBlit.getInteger(Keys.web.summaryRefsCount, 5);
 
 		Repository r = getRepository();
 		List<Metric> metrics = null;
@@ -82,11 +72,13 @@
 			metricsTotal = metrics.remove(0);
 		}
 
+		addSyndicationDiscoveryLink();
+
 		// repository description
 		add(new Label("repositoryDescription", getRepositoryModel().description));
 		add(new Label("repositoryOwner", getRepositoryModel().owner));
 
-		add(WicketUtils.createTimestampLabel("repositoryLastChange", JGitUtils.getLastChange(r),
+		add(WicketUtils.createTimestampLabel("repositoryLastChange", JGitUtils.getLastChange(r, null),
 				getTimeZone()));
 		if (metricsTotal == null) {
 			add(new Label("branchStats", ""));
@@ -121,18 +113,9 @@
 			default:
 				add(WicketUtils.newClearPixel("accessRestrictionIcon").setVisible(false));
 			}
-
-			HttpServletRequest req = ((WebRequest) getRequestCycle().getRequest())
-					.getHttpServletRequest();
 			StringBuilder sb = new StringBuilder();
-			sb.append(req.getScheme());
-			sb.append("://");
-			sb.append(req.getServerName());
-			if ((req.getScheme().equals("http") && req.getServerPort() != 80)
-					|| (req.getScheme().equals("https") && req.getServerPort() != 443)) {
-				sb.append(":" + req.getServerPort());
-			}
-			sb.append(Constants.GIT_SERVLET_PATH);
+			sb.append(WicketUtils.getHostURL(getRequestCycle().getRequest()));
+			sb.append(Constants.GIT_PATH);
 			sb.append(repositoryName);
 			repositoryUrls.add(sb.toString());
 		} else {

--
Gitblit v1.9.1