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/panels/RepositoriesPanel.java | 221 ++++++++++++++++++++++++++++++++++++++++++++++--------- 1 files changed, 185 insertions(+), 36 deletions(-) diff --git a/src/com/gitblit/wicket/panels/RepositoriesPanel.java b/src/com/gitblit/wicket/panels/RepositoriesPanel.java index c744148..726af61 100644 --- a/src/com/gitblit/wicket/panels/RepositoriesPanel.java +++ b/src/com/gitblit/wicket/panels/RepositoriesPanel.java @@ -25,6 +25,7 @@ import java.util.List; import java.util.Map; +import org.apache.wicket.Component; import org.apache.wicket.PageParameters; import org.apache.wicket.extensions.markup.html.repeater.data.sort.OrderByBorder; import org.apache.wicket.extensions.markup.html.repeater.util.SortParam; @@ -45,49 +46,99 @@ import com.gitblit.GitBlit; import com.gitblit.Keys; import com.gitblit.SyndicationServlet; +import com.gitblit.models.ProjectModel; import com.gitblit.models.RepositoryModel; import com.gitblit.models.UserModel; +import com.gitblit.utils.ArrayUtils; import com.gitblit.utils.StringUtils; -import com.gitblit.utils.TimeUtils; import com.gitblit.wicket.GitBlitWebSession; import com.gitblit.wicket.WicketUtils; +import com.gitblit.wicket.pages.BasePage; import com.gitblit.wicket.pages.EditRepositoryPage; +import com.gitblit.wicket.pages.EmptyRepositoryPage; +import com.gitblit.wicket.pages.ProjectPage; +import com.gitblit.wicket.pages.RepositoriesPage; import com.gitblit.wicket.pages.SummaryPage; +import com.gitblit.wicket.pages.UserPage; public class RepositoriesPanel extends BasePanel { private static final long serialVersionUID = 1L; - public RepositoriesPanel(String wicketId, final boolean showAdmin, + public RepositoriesPanel(String wicketId, final boolean showAdmin, final boolean showManagement, + List<RepositoryModel> models, boolean enableLinks, final Map<AccessRestrictionType, String> accessRestrictionTranslations) { super(wicketId); + final boolean linksActive = enableLinks; + final boolean showSize = GitBlit.getBoolean(Keys.web.showRepositorySizes, true); + final UserModel user = GitBlitWebSession.get().getUser(); - List<RepositoryModel> models = GitBlit.self().getRepositoryModels(user); + final IDataProvider<RepositoryModel> dp; - Fragment adminLinks = new Fragment("adminPanel", "adminLinks", this); - adminLinks.add(new BookmarkablePageLink<Void>("newRepository", EditRepositoryPage.class)); - add(adminLinks.setVisible(showAdmin)); + Fragment managementLinks; + if (showAdmin) { + // user is admin + managementLinks = new Fragment("managementPanel", "adminLinks", this); + managementLinks.add(new Link<Void>("clearCache") { + + private static final long serialVersionUID = 1L; + + @Override + public void onClick() { + GitBlit.self().resetRepositoryListCache(); + setResponsePage(RepositoriesPage.class); + } + }.setVisible(GitBlit.getBoolean(Keys.git.cacheRepositoryList, true))); + managementLinks.add(new BookmarkablePageLink<Void>("newRepository", EditRepositoryPage.class)); + add(managementLinks); + } else if (showManagement && user != null && user.canCreate()) { + // user can create personal repositories + managementLinks = new Fragment("managementPanel", "personalLinks", this); + managementLinks.add(new BookmarkablePageLink<Void>("newRepository", EditRepositoryPage.class)); + add(managementLinks); + } else { + // user has no management permissions + add (new Label("managementPanel").setVisible(false)); + } 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 + roots.add(0, ""); + groups.put("", rootRepositories); + } + List<RepositoryModel> groupedModels = new ArrayList<RepositoryModel>(); for (String root : roots) { List<RepositoryModel> subModels = groups.get(root); - groupedModels.add(new GroupRepositoryModel(root, subModels.size())); + ProjectModel project = GitBlit.self().getProjectModel(root); + GroupRepositoryModel group = new GroupRepositoryModel(project.name, subModels.size()); + if (project != null) { + group.title = project.title; + group.description = project.description; + } + groupedModels.add(group); + Collections.sort(subModels); groupedModels.addAll(subModels); } dp = new RepositoriesProvider(groupedModels); @@ -95,9 +146,13 @@ dp = new SortableRepositoriesProvider(models); } + final String baseUrl = WicketUtils.getGitblitURL(getRequest()); + final boolean showSwatch = GitBlit.getBoolean(Keys.web.repositoryListSwatches, true); + DataView<RepositoryModel> dataView = new DataView<RepositoryModel>("row", dp) { private static final long serialVersionUID = 1L; int counter; + String currGroupName; @Override protected void onBeforeRender() { @@ -108,26 +163,89 @@ public void populateItem(final Item<RepositoryModel> item) { final RepositoryModel entry = item.getModelObject(); if (entry instanceof GroupRepositoryModel) { + GroupRepositoryModel groupRow = (GroupRepositoryModel) entry; + currGroupName = entry.name; Fragment row = new Fragment("rowContent", "groupRepositoryRow", this); item.add(row); - row.add(new Label("groupName", entry.toString())); + + String name = groupRow.name; + if (name.charAt(0) == '~') { + // user page + String username = name.substring(1); + UserModel user = GitBlit.self().getUserModel(username); + row.add(new LinkPanel("groupName", null, (user == null ? username : user.getDisplayName()) + " (" + groupRow.count + ")", UserPage.class, WicketUtils.newUsernameParameter(username))); + row.add(new Label("groupDescription", getString("gb.personalRepositories"))); + } else { + // project page + row.add(new LinkPanel("groupName", null, groupRow.toString(), ProjectPage.class, WicketUtils.newProjectParameter(entry.name))); + row.add(new Label("groupDescription", entry.description == null ? "":entry.description)); + } 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); + + // try to strip group name for less cluttered list + String repoName = entry.toString(); + if (!StringUtils.isEmpty(currGroupName) && (repoName.indexOf('/') > -1)) { + repoName = repoName.substring(currGroupName.length() + 1); + } + + // repository swatch + Component swatch; + if (entry.isBare){ + swatch = new Label("repositorySwatch", " ").setEscapeModelStrings(false); + } else { + swatch = new Label("repositorySwatch", "!"); + WicketUtils.setHtmlTooltip(swatch, getString("gb.workingCopyWarning")); + } + WicketUtils.setCssBackground(swatch, entry.toString()); + row.add(swatch); + swatch.setVisible(showSwatch); + + if (linksActive) { + Class<? extends BasePage> linkPage; + if (entry.hasCommits) { + // repository has content + linkPage = SummaryPage.class; + } else { + // new/empty repository OR proposed repository + linkPage = EmptyRepositoryPage.class; + } + + PageParameters pp = WicketUtils.newRepositoryParameter(entry.name); + row.add(new LinkPanel("repositoryName", "list", repoName, linkPage, pp)); + row.add(new LinkPanel("repositoryDescription", "list", entry.description, + linkPage, pp)); + } else { + // no links like on a federation page + row.add(new Label("repositoryName", repoName)); + row.add(new Label("repositoryDescription", entry.description)); + } if (entry.hasCommits) { // Existing repository - 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)); + row.add(new Label("repositorySize", entry.size).setVisible(showSize)); } else { // New repository - row.add(new Label("repositoryName", entry.name - + "<span class='empty'>(empty)</span>").setEscapeModelStrings(false)); - row.add(new Label("repositoryDescription", entry.description)); + row.add(new Label("repositorySize", "<span class='empty'>(" + getString("gb.empty") + ")</span>") + .setEscapeModelStrings(false)); + } + + if (entry.isSparkleshared()) { + row.add(WicketUtils.newImage("sparkleshareIcon", "star_16x16.png", + getString("gb.isSparkleshared"))); + } else { + row.add(WicketUtils.newClearPixel("sparkleshareIcon").setVisible(false)); + } + + if (entry.isFork()) { + row.add(WicketUtils.newImage("forkIcon", "commit_divide_16x16.png", + getString("gb.isFork"))); + } else { + row.add(WicketUtils.newClearPixel("forkIcon").setVisible(false)); } if (entry.useTickets) { @@ -150,6 +268,13 @@ } 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: row.add(WicketUtils.newBlankImage("accessRestrictionIcon")); @@ -170,15 +295,37 @@ row.add(WicketUtils.newBlankImage("accessRestrictionIcon")); } - row.add(new Label("repositoryOwner", entry.owner)); + String owner = ""; + if (!ArrayUtils.isEmpty(entry.owners)) { + // display first owner + for (String username : entry.owners) { + UserModel ownerModel = GitBlit.self().getUserModel(username); + if (ownerModel != null) { + owner = ownerModel.getDisplayName(); + break; + } + } + if (entry.owners.size() > 1) { + owner += ", ..."; + } + } + Label ownerLabel = new Label("repositoryOwner", owner); + WicketUtils.setHtmlTooltip(ownerLabel, ArrayUtils.toString(entry.owners)); + row.add(ownerLabel); - String lastChange = TimeUtils.timeAgo(entry.lastChange); + String lastChange; + if (entry.lastChange.getTime() == 0) { + lastChange = "--"; + } else { + lastChange = getTimeUtils().timeAgo(entry.lastChange); + } Label lastChangeLabel = new Label("repositoryLastChange", lastChange); row.add(lastChangeLabel); - WicketUtils.setCssClass(lastChangeLabel, TimeUtils.timeAgoCss(entry.lastChange)); + WicketUtils.setCssClass(lastChangeLabel, getTimeUtils().timeAgoCss(entry.lastChange)); - boolean showOwner = user != null && user.username.equalsIgnoreCase(entry.owner); - if (showAdmin) { + boolean showOwner = user != null && entry.isOwner(user.username); + boolean myPersonalRepository = showOwner && entry.isUsersPersonalRepository(user.username); + if (showAdmin || myPersonalRepository) { Fragment repositoryLinks = new Fragment("repositoryLinks", "repositoryAdminLinks", this); repositoryLinks.add(new BookmarkablePageLink<Void>("editRepository", @@ -191,20 +338,19 @@ @Override public void onClick() { if (GitBlit.self().deleteRepositoryModel(entry)) { - info(MessageFormat.format("Repository ''{0}'' deleted.", entry)); if (dp instanceof SortableRepositoriesProvider) { + info(MessageFormat.format(getString("gb.repositoryDeleted"), entry)); ((SortableRepositoriesProvider) dp).remove(entry); } else { - ((RepositoriesProvider) dp).remove(entry); + setResponsePage(getPage().getClass(), getPage().getPageParameters()); } } else { - error(MessageFormat.format("Failed to delete repository ''{0}''!", - entry)); + error(MessageFormat.format(getString("gb.repositoryDeleteFailed"), entry)); } } }; deleteLink.add(new JavascriptEventConfirmation("onclick", MessageFormat.format( - "Delete repository \"{0}\"?", entry))); + getString("gb.deleteRepository"), entry))); repositoryLinks.add(deleteLink); row.add(repositoryLinks); } else if (showOwner) { @@ -217,8 +363,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++; } @@ -246,6 +392,7 @@ private static final long serialVersionUID = 1L; int count; + String title; GroupRepositoryModel(String name, int count) { super(name, "", "", new Date(0)); @@ -254,7 +401,7 @@ @Override public String toString() { - return name + " (" + count + ")"; + return (StringUtils.isEmpty(title) ? name : title) + " (" + count + ")"; } } @@ -385,10 +532,12 @@ Collections.sort(list, new Comparator<RepositoryModel>() { @Override public int compare(RepositoryModel o1, RepositoryModel o2) { + String own1 = ArrayUtils.toString(o1.owners); + String own2 = ArrayUtils.toString(o2.owners); if (asc) { - return o1.owner.compareTo(o2.owner); + return own1.compareTo(own2); } - return o2.owner.compareTo(o1.owner); + return own2.compareTo(own1); } }); } else if (prop.equals(SortBy.description.name())) { -- Gitblit v1.9.1