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