From 9effe1630d97039b3e01cd9b58ed07e75be1d63c Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Mon, 25 Feb 2013 08:40:30 -0500 Subject: [PATCH] Merge pull request #75 from thefake/master --- src/com/gitblit/wicket/pages/SummaryPage.java | 50 +++++++++++++++++++++++++++++++++++++++----------- 1 files changed, 39 insertions(+), 11 deletions(-) diff --git a/src/com/gitblit/wicket/pages/SummaryPage.java b/src/com/gitblit/wicket/pages/SummaryPage.java index acb4180..bd40a1b 100644 --- a/src/com/gitblit/wicket/pages/SummaryPage.java +++ b/src/com/gitblit/wicket/pages/SummaryPage.java @@ -27,6 +27,9 @@ import org.apache.wicket.markup.html.basic.Label; import org.apache.wicket.markup.html.link.BookmarkablePageLink; import org.apache.wicket.markup.html.panel.Fragment; +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.Repository; import org.eclipse.jgit.revwalk.RevCommit; import org.wicketstuff.googlecharts.Chart; @@ -45,13 +48,14 @@ import com.gitblit.models.Metric; import com.gitblit.models.PathModel; import com.gitblit.models.RepositoryModel; +import com.gitblit.models.UserModel; import com.gitblit.utils.ArrayUtils; import com.gitblit.utils.JGitUtils; import com.gitblit.utils.MarkdownUtils; import com.gitblit.utils.StringUtils; -import com.gitblit.utils.TimeUtils; import com.gitblit.wicket.WicketUtils; import com.gitblit.wicket.panels.BranchesPanel; +import com.gitblit.wicket.panels.LinkPanel; import com.gitblit.wicket.panels.LogPanel; import com.gitblit.wicket.panels.RepositoryUrlPanel; import com.gitblit.wicket.panels.TagsPanel; @@ -81,16 +85,37 @@ // repository description add(new Label("repositoryDescription", getRepositoryModel().description)); - add(new Label("repositoryOwner", getRepositoryModel().owner)); - + + // owner links + final List<String> owners = new ArrayList<String>(getRepositoryModel().owners); + ListDataProvider<String> ownersDp = new ListDataProvider<String>(owners); + DataView<String> ownersView = new DataView<String>("repositoryOwners", ownersDp) { + private static final long serialVersionUID = 1L; + int counter = 0; + public void populateItem(final Item<String> item) { + UserModel ownerModel = GitBlit.self().getUserModel(item.getModelObject()); + if (ownerModel != null) { + item.add(new LinkPanel("owner", null, ownerModel.getDisplayName(), UserPage.class, + WicketUtils.newUsernameParameter(ownerModel.username)).setRenderBodyOnly(true)); + } else { + item.add(new Label("owner").setVisible(false)); + } + counter++; + item.add(new Label("comma", ",").setVisible(counter < owners.size())); + item.setRenderBodyOnly(true); + } + }; + ownersView.setRenderBodyOnly(true); + add(ownersView); + add(WicketUtils.createTimestampLabel("repositoryLastChange", - JGitUtils.getLastChange(r, null), getTimeZone())); + JGitUtils.getLastChange(r), getTimeZone(), getTimeUtils())); if (metricsTotal == null) { add(new Label("branchStats", "")); } else { add(new Label("branchStats", - MessageFormat.format("{0} commits and {1} tags in {2}", metricsTotal.count, - metricsTotal.tag, TimeUtils.duration(metricsTotal.duration)))); + MessageFormat.format(getString("gb.branchStats"), metricsTotal.count, + metricsTotal.tag, getTimeUtils().duration(metricsTotal.duration)))); } add(new BookmarkablePageLink<Void>("metrics", MetricsPage.class, WicketUtils.newRepositoryParameter(repositoryName))); @@ -131,12 +156,13 @@ add(new Label("otherUrls", StringUtils.flattenStrings(repositoryUrls, "<br/>")) .setEscapeModelStrings(false)); - add(new LogPanel("commitsPanel", repositoryName, getRepositoryModel().HEAD, r, numberCommits, 0)); + add(new LogPanel("commitsPanel", repositoryName, getRepositoryModel().HEAD, r, numberCommits, 0, getRepositoryModel().showRemoteBranches)); add(new TagsPanel("tagsPanel", repositoryName, r, numberRefs).hideIfEmpty()); - add(new BranchesPanel("branchesPanel", getRepositoryModel(), r, numberRefs).hideIfEmpty()); + add(new BranchesPanel("branchesPanel", getRepositoryModel(), r, numberRefs, false).hideIfEmpty()); if (getRepositoryModel().showReadme) { String htmlText = null; + String markdownText = null; String readme = null; try { RevCommit head = JGitUtils.getCommit(r, null); @@ -158,11 +184,13 @@ } } if (!StringUtils.isEmpty(readme)) { - String markdownText = JGitUtils.getStringContent(r, head.getTree(), readme); + String [] encodings = GitBlit.getEncodings(); + markdownText = JGitUtils.getStringContent(r, head.getTree(), readme, encodings); htmlText = MarkdownUtils.transformMarkdown(markdownText); } } catch (ParseException p) { - error(p.getMessage()); + markdownText = MessageFormat.format("<div class=\"alert alert-error\"><strong>{0}:</strong> {1}</div>{2}", getString("gb.error"), getString("gb.markdownFailure"), markdownText); + htmlText = StringUtils.breakLinesForHtml(markdownText); } Fragment fragment = new Fragment("readme", "markdownPanel"); fragment.add(new Label("readmeFile", readme)); @@ -188,7 +216,7 @@ && GitBlit.getBoolean(Keys.web.generateActivityGraph, true)) { IChartData data = WicketUtils.getChartData(metrics); - ChartProvider provider = new ChartProvider(new Dimension(400, 100), ChartType.LINE, + ChartProvider provider = new ChartProvider(new Dimension(290, 100), ChartType.LINE, data); ChartAxis dateAxis = new ChartAxis(ChartAxisType.BOTTOM); dateAxis.setLabels(new String[] { metrics.get(0).name, -- Gitblit v1.9.1