From e19d3da3cdb4e3d0524e3a3ffc6a2cfa8876e927 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Wed, 16 Nov 2011 22:33:18 -0500 Subject: [PATCH] Metric object now implements comparable. Adjust gravatar css. --- src/com/gitblit/wicket/panels/RepositoriesPanel.java | 65 +++++++++++++++++++++++--------- 1 files changed, 47 insertions(+), 18 deletions(-) diff --git a/src/com/gitblit/wicket/panels/RepositoriesPanel.java b/src/com/gitblit/wicket/panels/RepositoriesPanel.java index 2527f4f..d18d5bb 100644 --- a/src/com/gitblit/wicket/panels/RepositoriesPanel.java +++ b/src/com/gitblit/wicket/panels/RepositoriesPanel.java @@ -47,7 +47,6 @@ import com.gitblit.SyndicationServlet; import com.gitblit.models.RepositoryModel; import com.gitblit.models.UserModel; -import com.gitblit.utils.ByteFormat; import com.gitblit.utils.StringUtils; import com.gitblit.utils.TimeUtils; import com.gitblit.wicket.GitBlitWebSession; @@ -60,11 +59,23 @@ private static final long serialVersionUID = 1L; public RepositoriesPanel(String wicketId, final boolean showAdmin, + List<RepositoryModel> models, final Map<AccessRestrictionType, String> accessRestrictionTranslations) { super(wicketId); + final boolean linksActive; + final boolean showSize = GitBlit.getBoolean(Keys.web.showRepositorySizes, true); + final UserModel user = GitBlitWebSession.get().getUser(); - List<RepositoryModel> models = GitBlit.self().getRepositoryModels(user); + if (models == null) { + linksActive = true; + models = GitBlit.self().getRepositoryModels(user); + } else { + // disable links if the repositories are already provided + // the repositories are most likely from a proposal + linksActive = false; + } + final IDataProvider<RepositoryModel> dp; Fragment adminLinks = new Fragment("adminPanel", "adminLinks", this); @@ -100,6 +111,7 @@ for (String root : roots) { List<RepositoryModel> subModels = groups.get(root); groupedModels.add(new GroupRepositoryModel(root, subModels.size())); + Collections.sort(subModels); groupedModels.addAll(subModels); } dp = new RepositoriesProvider(groupedModels); @@ -107,11 +119,12 @@ dp = new SortableRepositoriesProvider(models); } - final boolean showSize = GitBlit.getBoolean(Keys.web.showRepositorySizes, true); - final ByteFormat byteFormat = new ByteFormat(); + final String baseUrl = WicketUtils.getGitblitURL(getRequest()); + DataView<RepositoryModel> dataView = new DataView<RepositoryModel>("row", dp) { private static final long serialVersionUID = 1L; int counter; + String currGroupName; @Override protected void onBeforeRender() { @@ -122,31 +135,40 @@ public void populateItem(final Item<RepositoryModel> item) { final RepositoryModel entry = item.getModelObject(); if (entry instanceof GroupRepositoryModel) { + currGroupName = entry.name; Fragment row = new Fragment("rowContent", "groupRepositoryRow", this); item.add(row); row.add(new Label("groupName", entry.toString())); WicketUtils.setCssClass(item, "group"); + // reset counter so that first row is light background + counter = 0; return; } Fragment row = new Fragment("rowContent", "repositoryRow", this); item.add(row); - if (entry.hasCommits) { - // Existing repository + + // try to strip group name for less cluttered list + String repoName = entry.name; + if (!StringUtils.isEmpty(currGroupName) && (repoName.indexOf('/') > -1)) { + repoName = repoName.substring(currGroupName.length() + 1); + } + + if (entry.hasCommits && linksActive) { PageParameters pp = WicketUtils.newRepositoryParameter(entry.name); - row.add(new LinkPanel("repositoryName", "list", entry.name, SummaryPage.class, - pp)); + row.add(new LinkPanel("repositoryName", "list", repoName, SummaryPage.class, pp)); row.add(new LinkPanel("repositoryDescription", "list", entry.description, SummaryPage.class, pp)); - if (showSize) { - row.add(new Label("repositorySize", byteFormat.format(GitBlit.self() - .calculateSize(entry)))); - } else { - row.add(new Label("repositorySize").setVisible(false)); - } + } else { + // new/empty repository OR proposed repository + row.add(new Label("repositoryName", repoName)); + row.add(new Label("repositoryDescription", entry.description)); + } + + if (entry.hasCommits) { + // Existing repository + row.add(new Label("repositorySize", entry.size).setVisible(showSize)); } else { // New repository - row.add(new Label("repositoryName", entry.name)); - row.add(new Label("repositoryDescription", entry.description)); row.add(new Label("repositorySize", "<span class='empty'>(empty)</span>") .setEscapeModelStrings(false)); } @@ -170,6 +192,13 @@ getString("gb.isFrozen"))); } else { row.add(WicketUtils.newClearPixel("frozenIcon").setVisible(false)); + } + + if (entry.isFederated) { + row.add(WicketUtils.newImage("federatedIcon", "federated_16x16.png", + getString("gb.isFederated"))); + } else { + row.add(WicketUtils.newClearPixel("federatedIcon").setVisible(false)); } switch (entry.accessRestriction) { case NONE: @@ -243,8 +272,8 @@ } else { row.add(new Label("repositoryLinks")); } - row.add(new ExternalLink("syndication", SyndicationServlet.asLink(getRequest() - .getRelativePathPrefixToContextRoot(), entry.name, null, 0))); + row.add(new ExternalLink("syndication", SyndicationServlet.asLink(baseUrl, + entry.name, null, 0)).setVisible(linksActive)); WicketUtils.setAlternatingBackground(item, counter); counter++; } -- Gitblit v1.9.1