From 61b094152515532b42aa5e918f5405d82cde0440 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Wed, 13 Apr 2011 16:27:17 -0400
Subject: [PATCH] Fixed ticgit pages.

---
 src/com/gitblit/wicket/pages/SummaryPage.java |  190 ++++++++---------------------------------------
 1 files changed, 32 insertions(+), 158 deletions(-)

diff --git a/src/com/gitblit/wicket/pages/SummaryPage.java b/src/com/gitblit/wicket/pages/SummaryPage.java
index b7be71a..1dafb52 100644
--- a/src/com/gitblit/wicket/pages/SummaryPage.java
+++ b/src/com/gitblit/wicket/pages/SummaryPage.java
@@ -1,21 +1,12 @@
 package com.gitblit.wicket.pages;
 
 import java.awt.Dimension;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Date;
 import java.util.List;
-import java.util.Map;
 
 import org.apache.wicket.PageParameters;
 import org.apache.wicket.markup.html.basic.Label;
 import org.apache.wicket.markup.html.image.ContextImage;
-import org.apache.wicket.markup.repeater.Item;
-import org.apache.wicket.markup.repeater.data.DataView;
-import org.apache.wicket.markup.repeater.data.ListDataProvider;
-import org.eclipse.jgit.lib.ObjectId;
 import org.eclipse.jgit.lib.Repository;
-import org.eclipse.jgit.revwalk.RevCommit;
 
 import com.codecommit.wicket.AbstractChartData;
 import com.codecommit.wicket.Chart;
@@ -27,22 +18,17 @@
 import com.gitblit.StoredSettings;
 import com.gitblit.utils.JGitUtils;
 import com.gitblit.wicket.GitBlitWebApp;
-import com.gitblit.wicket.GitBlitWebSession;
-import com.gitblit.wicket.LinkPanel;
 import com.gitblit.wicket.RepositoryPage;
 import com.gitblit.wicket.WicketUtils;
 import com.gitblit.wicket.models.Metric;
-import com.gitblit.wicket.models.RefModel;
-import com.gitblit.wicket.panels.AnnotatedTagLinksPanel;
-import com.gitblit.wicket.panels.BranchLinksPanel;
-import com.gitblit.wicket.panels.RefsPanel;
-import com.gitblit.wicket.panels.ShortLogLinksPanel;
-import com.gitblit.wicket.panels.TagLinksPanel;
+import com.gitblit.wicket.panels.BranchesPanel;
+import com.gitblit.wicket.panels.LogPanel;
+import com.gitblit.wicket.panels.TagsPanel;
 
 public class SummaryPage extends RepositoryPage {
 
 	public SummaryPage(PageParameters params) {
-		super(params, "summary");
+		super(params);
 		
 		int numCommitsDef = 20;
 		int numRefsDef = 5;
@@ -57,157 +43,45 @@
 			numberRefs = numRefsDef;
 		}
 		
-		Repository r = getRepository();
-		final Map<ObjectId, List<String>> allRefs = JGitUtils.getAllRefs(r);
-
-		String owner = JGitUtils.getRepositoryOwner(r);
-		GitBlitWebSession session = GitBlitWebSession.get();
-		String lastchange = session.formatDateTimeLong(JGitUtils.getLastChange(r));
-		String cloneurl = GitBlitWebApp.get().getCloneUrl(repositoryName);
+		Repository r = getRepository();		
+		List<Metric> metrics = JGitUtils.getDateMetrics(r);
+		
+		long numberOfCommits = 0;
+		for (Metric m : metrics) {
+			numberOfCommits += m.count;
+		}
 
 		// repository description
 		add(new Label("repositoryDescription", description));
-		add(new Label("repositoryOwner", owner));
-		add(new Label("repositoryLastChange", lastchange));
-		add(new Label("repositoryCloneUrl", cloneurl));
+		add(new Label("repositoryOwner", JGitUtils.getRepositoryOwner(r)));
+		
+		add(WicketUtils.createTimestampLabel("repositoryLastChange", JGitUtils.getLastChange(r), getTimeZone()));
+		add(new Label("repositoryCloneUrl", GitBlitWebApp.get().getCloneUrl(repositoryName)));
 
-		// shortlog
-		add(new LinkPanel("shortlog", "title", "shortlog", ShortLogPage.class, newRepositoryParameter()));
-
-		List<RevCommit> commits = JGitUtils.getRevLog(r, numberCommits);
-		ListDataProvider<RevCommit> dp = new ListDataProvider<RevCommit>(commits);
-		DataView<RevCommit> shortlogView = new DataView<RevCommit>("commit", dp) {
-			private static final long serialVersionUID = 1L;
-			int counter = 0;
-
-			public void populateItem(final Item<RevCommit> item) {
-				RevCommit entry = item.getModelObject();
-				Date date = JGitUtils.getCommitDate(entry);
-
-				item.add(createShortlogDateLabel("commitDate", date));
-
-				String author = entry.getAuthorIdent().getName();
-				item.add(createAuthorLabel("commitAuthor", author));
-
-				String shortMessage = entry.getShortMessage();
-				String trimmedMessage = trimShortLog(shortMessage);
-				LinkPanel shortlog = new LinkPanel("commitShortMessage", "list subject", trimmedMessage, CommitPage.class, newCommitParameter(entry.getName()));
-				if (!shortMessage.equals(trimmedMessage)) {
-					WicketUtils.setHtmlTitle(shortlog, shortMessage);
-				}
-				item.add(shortlog);
-
-				item.add(new RefsPanel("commitRefs", entry, allRefs));
-
-				item.add(new ShortLogLinksPanel("commitLinks", repositoryName, entry.getName()));
-
-				setAlternatingBackground(item, counter);
-				counter++;
-			}
-		};
-		add(shortlogView);
-		if (commits.size() < numberCommits) {
-			add(new Label("shortlogMore", "").setVisible(false));
-		} else {
-			add(new LinkPanel("shortlogMore", "link", "more...", ShortLogPage.class, newRepositoryParameter()));
-		}
-
-		// tags
-		List<RefModel> tags = JGitUtils.getTags(r, numberRefs);
-		add(new LinkPanel("tags", "title", "tags", TagsPage.class, newRepositoryParameter()));
-
-		ListDataProvider<RefModel> tagsDp = new ListDataProvider<RefModel>(tags);
-		DataView<RefModel> tagView = new DataView<RefModel>("tag", tagsDp) {
-			private static final long serialVersionUID = 1L;
-			int counter = 0;
-
-			public void populateItem(final Item<RefModel> item) {
-				final RefModel entry = item.getModelObject();
-
-				item.add(createDateLabel("tagDate", entry.getDate()));
-
-				item.add(new LinkPanel("tagName", "list name", entry.getDisplayName(), CommitPage.class, newCommitParameter(entry.getCommitId().getName())));
-
-				if (entry.isAnnotatedTag()) {
-					// annotated tag
-					item.add(new LinkPanel("tagDescription", "list subject", entry.getShortLog(), TagPage.class, newCommitParameter(entry.getObjectId().getName())));
-					item.add(new AnnotatedTagLinksPanel("tagLinks", repositoryName, entry));
-				} else {
-					// simple tag on commit object
-					item.add(new Label("tagDescription", ""));
-					item.add(new TagLinksPanel("tagLinks", repositoryName, entry));
-				}
-
-				setAlternatingBackground(item, counter);
-				counter++;
-			}
-		};
-		add(tagView);
-		if (tags.size() < numberRefs) {
-			add(new Label("allTags", "").setVisible(false));
-		} else {
-			add(new LinkPanel("allTags", "link", "all tags...", TagsPage.class, newRepositoryParameter()));
-		}
-
-		// branches
-		List<RefModel> branches = new ArrayList<RefModel>();
-		branches.addAll(JGitUtils.getLocalBranches(r, numberRefs));
-		branches.addAll(JGitUtils.getRemoteBranches(r, numberRefs));
-		Collections.sort(branches);
-		Collections.reverse(branches);
-		if (numberRefs > 0 && branches.size() > numberRefs) {
-			branches = new ArrayList<RefModel>(branches.subList(0, numberRefs));
-		}
-
-		add(new LinkPanel("branches", "title", "branches", BranchesPage.class, newRepositoryParameter()));
-
-		ListDataProvider<RefModel> branchesDp = new ListDataProvider<RefModel>(branches);
-		DataView<RefModel> branchesView = new DataView<RefModel>("branch", branchesDp) {
-			private static final long serialVersionUID = 1L;
-			int counter = 0;
-
-			public void populateItem(final Item<RefModel> item) {
-				final RefModel entry = item.getModelObject();
-
-				item.add(createDateLabel("branchDate", entry.getDate()));
-
-				item.add(new LinkPanel("branchName", "list name", trimString(entry.getDisplayName(), 28), ShortLogPage.class, newCommitParameter(entry.getName())));
-
-				item.add(new BranchLinksPanel("branchLinks", repositoryName, entry));
-
-				setAlternatingBackground(item, counter);
-				counter++;
-			}
-		};
-		add(branchesView);
-		if (branches.size() < numberRefs) {
-			add(new Label("allBranches", "").setVisible(false));
-		} else {
-			add(new LinkPanel("allBranches", "link", "all branches...", BranchesPage.class, newRepositoryParameter()));
-		}
+		add(new LogPanel("commitsPanel", repositoryName, null, r, numberCommits, 0));
+		add(new TagsPanel("tagsPanel", repositoryName, r, numberRefs));
+		add(new BranchesPanel("branchesPanel", repositoryName, r, numberRefs));
 		
 		// Display an activity line graph
-		insertActivityGraph(r);
-
-		// close the repository
-		r.close();
-
-		// footer
-		addFooter();
+		insertActivityGraph(metrics);
+	}
+	
+	@Override
+	protected String getPageName() {
+		return getString("gb.summary");
 	}
 
-	private void insertActivityGraph(Repository r) {
-		if (StoredSettings.getBoolean("generateActivityGraph", true)) {
-			List<Metric> dates = JGitUtils.getDateMetrics(r);
-			IChartData data = getChartData(dates);
+	private void insertActivityGraph(List<Metric> metrics) {
+		if (StoredSettings.getBoolean("generateActivityGraph", true)) {			
+			IChartData data = getChartData(metrics);
 
 			ChartProvider provider = new ChartProvider(new Dimension(400, 80), ChartType.LINE, data);
 			ChartAxis dateAxis = new ChartAxis(ChartAxisType.BOTTOM);
-			dateAxis.setLabels(new String[] { dates.get(0).name, dates.get(dates.size() / 2).name, dates.get(dates.size() - 1).name });
+			dateAxis.setLabels(new String[] { metrics.get(0).name, metrics.get(metrics.size() / 2).name, metrics.get(metrics.size() - 1).name });
 			provider.addAxis(dateAxis);
 
 			ChartAxis commitAxis = new ChartAxis(ChartAxisType.LEFT);
-			commitAxis.setLabels(new String[] { "", String.valueOf((int) maxValue(dates)) });
+			commitAxis.setLabels(new String[] { "", String.valueOf((int) maxValue(metrics)) });
 			provider.addAxis(commitAxis);
 
 			add(new Chart("commitsChart", provider));
@@ -216,11 +90,11 @@
 		}
 	}
 
-	protected IChartData getChartData(List<Metric> results) {
-		final double[] counts = new double[results.size()];
+	protected IChartData getChartData(List<Metric> metrics) {
+		final double[] counts = new double[metrics.size()];
 		int i = 0;
 		double max = 0;
-		for (Metric m : results) {
+		for (Metric m : metrics) {
 			counts[i++] = m.count;
 			max = Math.max(max, m.count);
 		}

--
Gitblit v1.9.1