From a70b43cde76b4baab82b4ce0d9ff82883f80b8df Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Wed, 26 Oct 2011 17:54:07 -0400 Subject: [PATCH] Added status icon --- src/com/gitblit/wicket/pages/CommitPage.java | 57 +++++++++++++++++++++++++++++++-------------------------- 1 files changed, 31 insertions(+), 26 deletions(-) diff --git a/src/com/gitblit/wicket/pages/CommitPage.java b/src/com/gitblit/wicket/pages/CommitPage.java index 3af9cf1..ad33446 100644 --- a/src/com/gitblit/wicket/pages/CommitPage.java +++ b/src/com/gitblit/wicket/pages/CommitPage.java @@ -16,9 +16,9 @@ package com.gitblit.wicket.pages; import java.util.ArrayList; +import java.util.Arrays; import java.util.List; -import org.apache.wicket.Component; import org.apache.wicket.PageParameters; import org.apache.wicket.markup.html.basic.Label; import org.apache.wicket.markup.html.link.BookmarkablePageLink; @@ -37,12 +37,12 @@ import com.gitblit.models.GitNote; import com.gitblit.models.PathModel.PathChangeModel; import com.gitblit.utils.JGitUtils; -import com.gitblit.utils.StringUtils; import com.gitblit.utils.JGitUtils.SearchType; import com.gitblit.wicket.WicketUtils; import com.gitblit.wicket.panels.CommitHeaderPanel; import com.gitblit.wicket.panels.CommitLegendPanel; import com.gitblit.wicket.panels.LinkPanel; +import com.gitblit.wicket.panels.RefsPanel; public class CommitPage extends RepositoryPage { @@ -64,8 +64,8 @@ add(new Label("parentLink", "none")); add(new Label("commitdiffLink", getString("gb.commitdiff"))); } else { - add(new LinkPanel("parentLink", null, parents.get(0).substring(0, 8), CommitPage.class, - newCommitParameter(parents.get(0)))); + add(new LinkPanel("parentLink", null, getShortObjectId(parents.get(0)), + CommitPage.class, newCommitParameter(parents.get(0)))); add(new LinkPanel("commitdiffLink", null, new StringResourceModel("gb.commitdiff", this, null), CommitDiffPage.class, WicketUtils.newObjectParameter( repositoryName, objectId))); @@ -92,9 +92,9 @@ add(new LinkPanel("commitTree", "list", c.getTree().getName(), TreePage.class, newCommitParameter())); add(new BookmarkablePageLink<Void>("treeLink", TreePage.class, newCommitParameter())); - add(new ExternalLink("zipLink", DownloadZipServlet.asLink(getRequest() - .getRelativePathPrefixToContextRoot(), repositoryName, objectId, null)) - .setVisible(GitBlit.getBoolean(Keys.web.allowZipDownloads, true))); + final String baseUrl = WicketUtils.getGitblitURL(getRequest()); + add(new ExternalLink("zipLink", DownloadZipServlet.asLink(baseUrl, repositoryName, + objectId, null)).setVisible(GitBlit.getBoolean(Keys.web.allowZipDownloads, true))); // Parent Commits ListDataProvider<String> parentsDp = new ListDataProvider<String>(parents); @@ -123,19 +123,17 @@ public void populateItem(final Item<GitNote> item) { GitNote entry = item.getModelObject(); - Component c = new LinkPanel("refName", null, entry.notesRef.displayName, - CommitPage.class, newCommitParameter(entry.notesRef.commit.getName())); - WicketUtils.setCssClass(c, "headRef"); - item.add(c); - item.add(createPersonPanel("authorName", entry.notesRef.commit.getAuthorIdent(), SearchType.AUTHOR)); - item.add(WicketUtils.createTimestampLabel("authorDate", - entry.notesRef.commit.getAuthorIdent().getWhen(), getTimeZone())); - item.add(new Label("noteContent", StringUtils.breakLinesForHtml(entry.content)).setEscapeModelStrings(false)); + item.add(new RefsPanel("refName", repositoryName, Arrays.asList(entry.notesRef))); + item.add(createPersonPanel("authorName", entry.notesRef.getAuthorIdent(), + SearchType.AUTHOR)); + item.add(WicketUtils.createTimestampLabel("authorDate", entry.notesRef + .getAuthorIdent().getWhen(), getTimeZone())); + item.add(new Label("noteContent", GitBlit.self().processCommitMessage( + repositoryName, entry.content)).setEscapeModelStrings(false)); } }; add(notesView.setVisible(notes.size() > 0)); - - + // changed paths list List<PathChangeModel> paths = JGitUtils.getFilesInCommit(r, c); add(new CommitLegendPanel("commitLegend", paths)); @@ -152,19 +150,26 @@ item.add(changeType); if (entry.isTree()) { item.add(new LinkPanel("pathName", null, entry.path, TreePage.class, - newPathParameter(entry.path))); + WicketUtils + .newPathParameter(repositoryName, entry.commitId, entry.path))); } else { item.add(new LinkPanel("pathName", "list", entry.path, BlobPage.class, - newPathParameter(entry.path))); + WicketUtils + .newPathParameter(repositoryName, entry.commitId, entry.path))); } - item.add(new BookmarkablePageLink<Void>("diff", BlobDiffPage.class, - newPathParameter(entry.path)).setEnabled(!entry.changeType.equals(ChangeType.ADD) && !entry.changeType.equals(ChangeType.DELETE))); - item.add(new BookmarkablePageLink<Void>("view", BlobPage.class, - newPathParameter(entry.path))); - item.add(new BookmarkablePageLink<Void>("blame", BlobPage.class).setEnabled(false)); - item.add(new BookmarkablePageLink<Void>("history", HistoryPage.class, - newPathParameter(entry.path)).setEnabled(!entry.changeType.equals(ChangeType.ADD))); + item.add(new BookmarkablePageLink<Void>("diff", BlobDiffPage.class, WicketUtils + .newPathParameter(repositoryName, entry.commitId, entry.path)) + .setEnabled(!entry.changeType.equals(ChangeType.ADD) + && !entry.changeType.equals(ChangeType.DELETE))); + item.add(new BookmarkablePageLink<Void>("view", BlobPage.class, WicketUtils + .newPathParameter(repositoryName, entry.commitId, entry.path))); + item.add(new BookmarkablePageLink<Void>("blame", BlamePage.class, WicketUtils + .newPathParameter(repositoryName, entry.commitId, entry.path)) + .setEnabled(!entry.changeType.equals(ChangeType.ADD))); + item.add(new BookmarkablePageLink<Void>("history", HistoryPage.class, WicketUtils + .newPathParameter(repositoryName, entry.commitId, entry.path)) + .setEnabled(!entry.changeType.equals(ChangeType.ADD))); WicketUtils.setAlternatingBackground(item, counter); counter++; -- Gitblit v1.9.1