From cb285cbfddfc0b633d6b8cdb4dc0d2bd2b8b51ef Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Thu, 05 Jan 2012 17:34:05 -0500 Subject: [PATCH] Fixed bug in receive hook for repositories in subfolders --- src/com/gitblit/wicket/panels/UsersPanel.java | 74 +++++++++++++++++++++++++++++++------ 1 files changed, 62 insertions(+), 12 deletions(-) diff --git a/src/com/gitblit/wicket/panels/UsersPanel.java b/src/com/gitblit/wicket/panels/UsersPanel.java index 1721272..ad2ed92 100644 --- a/src/com/gitblit/wicket/panels/UsersPanel.java +++ b/src/com/gitblit/wicket/panels/UsersPanel.java @@ -1,16 +1,35 @@ +/* + * Copyright 2011 gitblit.com. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ package com.gitblit.wicket.panels; +import java.text.MessageFormat; +import java.util.List; + +import org.apache.wicket.markup.html.basic.Label; import org.apache.wicket.markup.html.link.BookmarkablePageLink; +import org.apache.wicket.markup.html.link.Link; import org.apache.wicket.markup.html.panel.Fragment; import org.apache.wicket.markup.repeater.Item; import org.apache.wicket.markup.repeater.data.DataView; import org.apache.wicket.markup.repeater.data.ListDataProvider; import com.gitblit.GitBlit; -import com.gitblit.wicket.LinkPanel; +import com.gitblit.models.UserModel; import com.gitblit.wicket.WicketUtils; import com.gitblit.wicket.pages.EditUserPage; -import com.gitblit.wicket.pages.RepositoriesPage; public class UsersPanel extends BasePanel { @@ -22,19 +41,50 @@ Fragment adminLinks = new Fragment("adminPanel", "adminLinks", this); adminLinks.add(new BookmarkablePageLink<Void>("newUser", EditUserPage.class)); add(adminLinks.setVisible(showAdmin)); - - DataView<String> usersView = new DataView<String>("userRow", new ListDataProvider<String>(GitBlit.self().getAllUsernames())) { - private static final long serialVersionUID = 1L; - private int counter = 0; - public void populateItem(final Item<String> item) { - final String entry = item.getModelObject(); - LinkPanel editLink = new LinkPanel("username", "list", entry, EditUserPage.class, WicketUtils.newUsernameParameter(entry)); - WicketUtils.setHtmlTooltip(editLink, getString("gb.edit") + " " + entry); + final List<UserModel> users = GitBlit.self().getAllUsers(); + DataView<UserModel> usersView = new DataView<UserModel>("userRow", + new ListDataProvider<UserModel>(users)) { + private static final long serialVersionUID = 1L; + private int counter; + + @Override + protected void onBeforeRender() { + super.onBeforeRender(); + counter = 0; + } + + public void populateItem(final Item<UserModel> item) { + final UserModel entry = item.getModelObject(); + LinkPanel editLink = new LinkPanel("username", "list", entry.username, + EditUserPage.class, WicketUtils.newUsernameParameter(entry.username)); + WicketUtils.setHtmlTooltip(editLink, getString("gb.edit") + " " + entry.username); item.add(editLink); + item.add(new Label("accesslevel", entry.canAdmin ? "administrator" : "")); + item.add(new Label("teams", entry.teams.size() > 0 ? ("" + entry.teams.size()) : "")); + item.add(new Label("repositories", + entry.repositories.size() > 0 ? ("" + entry.repositories.size()) : "")); Fragment userLinks = new Fragment("userLinks", "userAdminLinks", this); - userLinks.add(new BookmarkablePageLink<Void>("editUser", EditUserPage.class, WicketUtils.newUsernameParameter(entry))); - userLinks.add(new BookmarkablePageLink<Void>("deleteUser", RepositoriesPage.class, WicketUtils.newUsernameParameter(entry)).setEnabled(false)); + userLinks.add(new BookmarkablePageLink<Void>("editUser", EditUserPage.class, + WicketUtils.newUsernameParameter(entry.username))); + Link<Void> deleteLink = new Link<Void>("deleteUser") { + + private static final long serialVersionUID = 1L; + + @Override + public void onClick() { + if (GitBlit.self().deleteUser(entry.username)) { + users.remove(entry); + info(MessageFormat.format("User ''{0}'' deleted.", entry.username)); + } else { + error(MessageFormat.format("Failed to delete user ''{0}''!", + entry.username)); + } + } + }; + deleteLink.add(new JavascriptEventConfirmation("onclick", MessageFormat.format( + "Delete user \"{0}\"?", entry.username))); + userLinks.add(deleteLink); item.add(userLinks); WicketUtils.setAlternatingBackground(item, counter); -- Gitblit v1.9.1