From fd6ac68c0b1136182b5d713ec88ea94e26c4a76d Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Thu, 29 Sep 2011 23:47:13 -0400 Subject: [PATCH] Adjusted Metric chart widths. Diff hunk header css tweak. --- src/com/gitblit/wicket/panels/RepositoriesPanel.java | 84 +++++++++++++++++++++++++++++++++++------- 1 files changed, 70 insertions(+), 14 deletions(-) diff --git a/src/com/gitblit/wicket/panels/RepositoriesPanel.java b/src/com/gitblit/wicket/panels/RepositoriesPanel.java index c744148..63b512c 100644 --- a/src/com/gitblit/wicket/panels/RepositoriesPanel.java +++ b/src/com/gitblit/wicket/panels/RepositoriesPanel.java @@ -47,6 +47,7 @@ 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; @@ -59,11 +60,29 @@ 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); + final ByteFormat byteFormat = new ByteFormat(); + if (showSize) { + for (RepositoryModel model : models) { + model.size = byteFormat.format(GitBlit.self().calculateSize(model)); + } + } + } 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); @@ -71,29 +90,43 @@ add(adminLinks.setVisible(showAdmin)); if (GitBlit.getString(Keys.web.repositoryListType, "flat").equalsIgnoreCase("grouped")) { + List<RepositoryModel> rootRepositories = new ArrayList<RepositoryModel>(); Map<String, List<RepositoryModel>> groups = new HashMap<String, List<RepositoryModel>>(); for (RepositoryModel model : models) { String rootPath = StringUtils.getRootPath(model.name); if (StringUtils.isEmpty(rootPath)) { - rootPath = GitBlit.getString(Keys.web.repositoryRootGroupName, " "); + // root repository + rootRepositories.add(model); + } else { + // non-root, grouped repository + if (!groups.containsKey(rootPath)) { + groups.put(rootPath, new ArrayList<RepositoryModel>()); + } + groups.get(rootPath).add(model); } - if (!groups.containsKey(rootPath)) { - groups.put(rootPath, new ArrayList<RepositoryModel>()); - } - groups.get(rootPath).add(model); } List<String> roots = new ArrayList<String>(groups.keySet()); Collections.sort(roots); + + if (rootRepositories.size() > 0) { + // inject the root repositories at the top of the page + String rootPath = GitBlit.getString(Keys.web.repositoryRootGroupName, " "); + roots.add(0, rootPath); + groups.put(rootPath, rootRepositories); + } List<RepositoryModel> groupedModels = new ArrayList<RepositoryModel>(); 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); } else { dp = new SortableRepositoriesProvider(models); } + + final String baseUrl = WicketUtils.getGitblitURL(getRequest()); DataView<RepositoryModel> dataView = new DataView<RepositoryModel>("row", dp) { private static final long serialVersionUID = 1L; @@ -116,18 +149,29 @@ } Fragment row = new Fragment("rowContent", "repositoryRow", this); item.add(row); - if (entry.hasCommits) { - // Existing repository + 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("repositoryDescription", "list", entry.description, SummaryPage.class, pp)); } else { - // New repository - row.add(new Label("repositoryName", entry.name - + "<span class='empty'>(empty)</span>").setEscapeModelStrings(false)); + // new/empty repository OR proposed repository + row.add(new Label("repositoryName", entry.name)); row.add(new Label("repositoryDescription", entry.description)); + } + + if (entry.hasCommits) { + // Existing repository + if (showSize) { + row.add(new Label("repositorySize", entry.size)); + } else { + row.add(new Label("repositorySize").setVisible(false)); + } + } else { + // New repository + row.add(new Label("repositorySize", "<span class='empty'>(empty)</span>") + .setEscapeModelStrings(false)); } if (entry.useTickets) { @@ -149,6 +193,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: @@ -172,7 +223,12 @@ row.add(new Label("repositoryOwner", entry.owner)); - String lastChange = TimeUtils.timeAgo(entry.lastChange); + String lastChange; + if (entry.lastChange.getTime() == 0) { + lastChange = "--"; + } else { + lastChange = TimeUtils.timeAgo(entry.lastChange); + } Label lastChangeLabel = new Label("repositoryLastChange", lastChange); row.add(lastChangeLabel); WicketUtils.setCssClass(lastChangeLabel, TimeUtils.timeAgoCss(entry.lastChange)); @@ -217,8 +273,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