From 595bbff87be7ff763bfad6fe297703141547710d Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Mon, 17 Oct 2011 17:42:27 -0400
Subject: [PATCH] Updated to MarkdownPapers 1.2.4.

---
 src/com/gitblit/client/GitblitPanel.java |  575 ++++++++++++++++++++-------------------------------------
 1 files changed, 201 insertions(+), 374 deletions(-)

diff --git a/src/com/gitblit/client/GitblitPanel.java b/src/com/gitblit/client/GitblitPanel.java
index a6be156..2c1c5a6 100644
--- a/src/com/gitblit/client/GitblitPanel.java
+++ b/src/com/gitblit/client/GitblitPanel.java
@@ -19,8 +19,6 @@
 import java.awt.Color;
 import java.awt.Component;
 import java.awt.Desktop;
-import java.awt.Dimension;
-import java.awt.GridLayout;
 import java.awt.Insets;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
@@ -28,14 +26,10 @@
 import java.net.URI;
 import java.text.MessageFormat;
 import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Date;
 import java.util.List;
-import java.util.Map;
 
 import javax.swing.JButton;
 import javax.swing.JLabel;
-import javax.swing.JList;
 import javax.swing.JOptionPane;
 import javax.swing.JPanel;
 import javax.swing.JScrollPane;
@@ -44,25 +38,16 @@
 import javax.swing.JTextField;
 import javax.swing.RowFilter;
 import javax.swing.SwingConstants;
-import javax.swing.SwingWorker;
 import javax.swing.event.ListSelectionEvent;
 import javax.swing.event.ListSelectionListener;
 import javax.swing.table.DefaultTableCellRenderer;
-import javax.swing.table.DefaultTableColumnModel;
 import javax.swing.table.TableCellRenderer;
-import javax.swing.table.TableColumn;
 import javax.swing.table.TableRowSorter;
 
 import com.gitblit.Constants.RpcRequest;
-import com.gitblit.GitBlitException.ForbiddenException;
-import com.gitblit.GitBlitException.UnauthorizedException;
-import com.gitblit.IStoredSettings;
-import com.gitblit.Keys;
 import com.gitblit.client.ClosableTabComponent.CloseTabListener;
-import com.gitblit.models.FederationModel;
 import com.gitblit.models.RepositoryModel;
 import com.gitblit.models.UserModel;
-import com.gitblit.utils.RpcUtils;
 import com.gitblit.utils.StringUtils;
 
 /**
@@ -80,17 +65,7 @@
 
 	private final Insets insets = new Insets(margin, margin, margin, margin);
 
-	private final String url;
-
-	private final String account;
-
-	private final char[] password;
-
-	private volatile boolean isAdmin;
-
-	private volatile List<UserModel> allUsers;
-
-	private volatile IStoredSettings settings;
+	private GitblitModel gitblit;
 
 	private JTabbedPane tabs;
 
@@ -98,7 +73,9 @@
 
 	private RepositoriesModel repositoriesModel;
 
-	private JList usersList;
+	private JTable usersTable;
+
+	private UsersModel usersModel;
 
 	private JPanel usersPanel;
 
@@ -114,25 +91,25 @@
 
 	private DefaultTableCellRenderer sizeRenderer;
 
-	private TableRowSorter<RepositoriesModel> defaultSorter;
+	private TableRowSorter<RepositoriesModel> defaultRepositoriesSorter;
 
-	private List<RepositoryModel> allRepositories;
+	private TableRowSorter<UsersModel> defaultUsersSorter;
+
+	private JButton editRepository;
 
 	public GitblitPanel(GitblitRegistration reg) {
 		this(reg.url, reg.account, reg.password);
 	}
 
 	public GitblitPanel(String url, String account, char[] password) {
-		this.url = url;
-		this.account = account;
-		this.password = password;
+		this.gitblit = new GitblitModel(url, account, password);
 
 		final JButton browseRepository = new JButton(Translation.get("gb.browse"));
 		browseRepository.setEnabled(false);
 		browseRepository.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e) {
 				RepositoryModel model = getSelectedRepositories().get(0);
-				String u = MessageFormat.format("{0}/summary/{1}", GitblitPanel.this.url,
+				String u = MessageFormat.format("{0}/summary/{1}", gitblit.url,
 						StringUtils.encodeURL(model.name));
 				try {
 					Desktop.getDesktop().browse(new URI(u));
@@ -145,15 +122,7 @@
 		JButton refreshRepositories = new JButton(Translation.get("gb.refresh"));
 		refreshRepositories.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e) {
-				try {
-					refreshRepositoriesTable();
-				} catch (ForbiddenException x) {
-					explainForbidden(RpcRequest.LIST_REPOSITORIES);
-				} catch (UnauthorizedException x) {
-					explainUnauthorized(RpcRequest.LIST_REPOSITORIES);
-				} catch (Throwable t) {
-					showException(t);
-				}
+				refreshRepositories();
 			}
 		});
 
@@ -164,7 +133,7 @@
 			}
 		});
 
-		final JButton editRepository = new JButton(Translation.get("gb.edit"));
+		editRepository = new JButton(Translation.get("gb.edit"));
 		editRepository.setEnabled(false);
 		editRepository.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e) {
@@ -180,16 +149,6 @@
 			}
 		});
 
-		final JButton cloneRepository = new JButton(Translation.get("gb.clone"));
-		cloneRepository.setEnabled(false);
-		cloneRepository.addActionListener(new ActionListener() {
-			public void actionPerformed(ActionEvent e) {
-				for (RepositoryModel model : getSelectedRepositories()) {
-					System.out.println("TODO Clone " + model);
-				}
-			}
-		});
-
 		nameRenderer = new NameRenderer();
 		typeRenderer = new IndicatorsRenderer();
 
@@ -202,23 +161,16 @@
 		ownerRenderer.setHorizontalAlignment(SwingConstants.CENTER);
 
 		repositoriesModel = new RepositoriesModel();
-		defaultSorter = new TableRowSorter<RepositoriesModel>(repositoriesModel);
-		repositoriesTable = new JTable(repositoriesModel);
-		repositoriesTable.setRowSorter(defaultSorter);
+		defaultRepositoriesSorter = new TableRowSorter<RepositoriesModel>(repositoriesModel);
+		repositoriesTable = Utils.newTable(repositoriesModel);
+		repositoriesTable.setRowHeight(nameRenderer.getFont().getSize() + 8);
+		repositoriesTable.setRowSorter(defaultRepositoriesSorter);
 		repositoriesTable.getRowSorter().toggleSortOrder(RepositoriesModel.Columns.Name.ordinal());
 
-		repositoriesTable.setCellSelectionEnabled(false);
-		repositoriesTable.setRowSelectionAllowed(true);
-		repositoriesTable.setRowHeight(nameRenderer.getFont().getSize() + 8);
-		repositoriesTable.getTableHeader().setReorderingAllowed(false);
-		repositoriesTable.setGridColor(new Color(0xd9d9d9));
-		repositoriesTable.setBackground(Color.white);
-		repositoriesTable.setDefaultRenderer(Date.class,
-				new DateCellRenderer(null, Color.orange.darker()));
-		setRenderer(RepositoriesModel.Columns.Name, nameRenderer);
-		setRenderer(RepositoriesModel.Columns.Indicators, typeRenderer);
-		setRenderer(RepositoriesModel.Columns.Owner, ownerRenderer);
-		setRenderer(RepositoriesModel.Columns.Size, sizeRenderer);
+		setRepositoryRenderer(RepositoriesModel.Columns.Name, nameRenderer);
+		setRepositoryRenderer(RepositoriesModel.Columns.Indicators, typeRenderer);
+		setRepositoryRenderer(RepositoriesModel.Columns.Owner, ownerRenderer);
+		setRepositoryRenderer(RepositoriesModel.Columns.Size, sizeRenderer);
 
 		repositoriesTable.getSelectionModel().addListSelectionListener(new ListSelectionListener() {
 			@Override
@@ -230,14 +182,13 @@
 				boolean selected = repositoriesTable.getSelectedRow() > -1;
 				browseRepository.setEnabled(singleSelection);
 				delRepository.setEnabled(selected);
-				cloneRepository.setEnabled(selected);
 				if (selected) {
 					int viewRow = repositoriesTable.getSelectedRow();
 					int modelRow = repositoriesTable.convertRowIndexToModel(viewRow);
 					RepositoryModel model = ((RepositoriesModel) repositoriesTable.getModel()).list
 							.get(modelRow);
 					editRepository.setEnabled(singleSelection
-							&& (isAdmin || model.owner.equalsIgnoreCase(GitblitPanel.this.account)));
+							&& (gitblit.allowAdmin() || gitblit.isOwner(model)));
 				} else {
 					editRepository.setEnabled(false);
 				}
@@ -260,8 +211,8 @@
 		tablePanel.add(new JScrollPane(repositoriesTable), BorderLayout.CENTER);
 
 		JPanel repositoryControls = new JPanel();
+		repositoryControls.add(refreshRepositories);
 		repositoryControls.add(browseRepository);
-		repositoryControls.add(cloneRepository);
 		repositoryControls.add(createRepository);
 		repositoryControls.add(editRepository);
 		repositoryControls.add(delRepository);
@@ -275,15 +226,7 @@
 		JButton refreshUsers = new JButton(Translation.get("gb.refresh"));
 		refreshUsers.addActionListener(new ActionListener() {
 			public void actionPerformed(ActionEvent e) {
-				try {
-					refreshUsersTable();
-				} catch (ForbiddenException x) {
-					explainForbidden(RpcRequest.LIST_USERS);
-				} catch (UnauthorizedException x) {
-					explainUnauthorized(RpcRequest.LIST_USERS);
-				} catch (Throwable t) {
-					showException(t);
-				}
+				refreshUsers();
 			}
 		});
 
@@ -310,22 +253,29 @@
 			}
 		});
 
-		usersList = new JList();
-		usersList.addListSelectionListener(new ListSelectionListener() {
+		usersModel = new UsersModel();
+		defaultUsersSorter = new TableRowSorter<UsersModel>(usersModel);
+		usersTable = Utils.newTable(usersModel);
+		String name = usersTable.getColumnName(UsersModel.Columns.Name.ordinal());
+		usersTable.setRowHeight(nameRenderer.getFont().getSize() + 8);
+		usersTable.getColumn(name).setCellRenderer(nameRenderer);
+		usersTable.setRowSorter(defaultUsersSorter);
+		usersTable.getRowSorter().toggleSortOrder(UsersModel.Columns.Name.ordinal());
+		usersTable.getSelectionModel().addListSelectionListener(new ListSelectionListener() {
 
 			@Override
 			public void valueChanged(ListSelectionEvent e) {
 				if (e.getValueIsAdjusting()) {
 					return;
 				}
-				boolean selected = usersList.getSelectedIndex() > -1;
-				boolean singleSelection = usersList.getSelectedIndices().length == 1;
+				boolean selected = usersTable.getSelectedRow() > -1;
+				boolean singleSelection = usersTable.getSelectedRows().length == 1;
 				editUser.setEnabled(singleSelection && selected);
 				delUser.setEnabled(selected);
 			}
 		});
 
-		JPanel userControls = new JPanel(new GridLayout(0, 2));
+		JPanel userControls = new JPanel();
 		userControls.add(refreshUsers);
 		userControls.add(createUser);
 		userControls.add(editUser);
@@ -333,18 +283,12 @@
 
 		usersPanel = new JPanel(new BorderLayout(margin, margin));
 		usersPanel.add(newHeaderLabel(Translation.get("gb.users")), BorderLayout.NORTH);
-		usersPanel.add(new JScrollPane(usersList), BorderLayout.CENTER);
+		usersPanel.add(new JScrollPane(usersTable), BorderLayout.CENTER);
 		usersPanel.add(userControls, BorderLayout.SOUTH);
 
-		/*
-		 * Assemble the main panel
-		 */
-		JPanel mainPanel = new JPanel(new BorderLayout(margin, margin));
-		mainPanel.add(repositoriesPanel, BorderLayout.CENTER);
-		mainPanel.add(usersPanel, BorderLayout.EAST);
-
 		tabs = new JTabbedPane(JTabbedPane.BOTTOM);
-		tabs.addTab(Translation.get("gb.repositories"), mainPanel);
+		tabs.addTab(Translation.get("gb.repositories"), repositoriesPanel);
+		tabs.addTab(Translation.get("gb.users"), usersPanel);
 		tabs.addTab(Translation.get("gb.federation"), new JPanel());
 
 		setLayout(new BorderLayout());
@@ -360,63 +304,51 @@
 		return label;
 	}
 
-	public void login() throws IOException {
-		refreshRepositoriesTable();
+	private void setRepositoryRenderer(RepositoriesModel.Columns col, TableCellRenderer renderer) {
+		String name = repositoriesTable.getColumnName(col.ordinal());
+		repositoriesTable.getColumn(name).setCellRenderer(renderer);
+	}
 
-		try {
-			refreshUsersTable();
-			refreshSettings();
-			isAdmin = true;
-			refreshFederationPanel();
-		} catch (ForbiddenException e) {
+	public void login() throws IOException {
+		gitblit.login();
+
+		updateRepositoriesTable();
+		Utils.packColumns(repositoriesTable, 2);
+
+		if (gitblit.allowAdmin()) {
+			updateUsersTable();
+		} else {
 			// user does not have administrator privileges
 			// hide admin repository buttons
 			createRepository.setVisible(false);
+			editRepository.setVisible(false);
 			delRepository.setVisible(false);
 
 			// hide users panel
 			usersPanel.setVisible(false);
 
-			// remove federation tab
-			tabs.removeTabAt(1);
-		} catch (IOException e) {
-			System.err.println(e.getMessage());
+			while (tabs.getTabCount() > 1) {
+				// remove admin tabs
+				tabs.removeTabAt(1);
+			}
 		}
 	}
 
-	private void refreshRepositoriesTable() throws IOException {
-		Map<String, RepositoryModel> repositories = RpcUtils
-				.getRepositories(url, account, password);
-		allRepositories = new ArrayList<RepositoryModel>(repositories.values());
-		Collections.sort(allRepositories);
+	private void updateRepositoriesTable() {
 		repositoriesModel.list.clear();
-		repositoriesModel.list.addAll(allRepositories);
+		repositoriesModel.list.addAll(gitblit.getRepositories());
 		repositoriesModel.fireTableDataChanged();
-		packColumns(repositoriesTable, 2);
 	}
 
-	private void setRenderer(RepositoriesModel.Columns col, TableCellRenderer renderer) {
-		String name = repositoriesTable.getColumnName(col.ordinal());
-		repositoriesTable.getColumn(name).setCellRenderer(renderer);
-	}
-
-	private void refreshUsersTable() throws IOException {
-		allUsers = RpcUtils.getUsers(url, account, password);
-		usersList.setListData(allUsers.toArray());
-	}
-
-	private void refreshSettings() throws IOException {
-		settings = RpcUtils.getSettings(url, account, password);
-	}
-
-	private void refreshFederationPanel() throws IOException {
-		List<FederationModel> registrations = RpcUtils.getFederationRegistrations(url, account,
-				password);
+	private void updateUsersTable() {
+		usersModel.list.clear();
+		usersModel.list.addAll(gitblit.getUsers());
+		usersModel.fireTableDataChanged();
 	}
 
 	private void filterRepositories(final String fragment) {
 		if (StringUtils.isEmpty(fragment)) {
-			repositoriesTable.setRowSorter(defaultSorter);
+			repositoriesTable.setRowSorter(defaultRepositoriesSorter);
 			return;
 		}
 		RowFilter<RepositoriesModel, Object> containsFilter = new RowFilter<RepositoriesModel, Object>() {
@@ -439,8 +371,7 @@
 		List<RepositoryModel> repositories = new ArrayList<RepositoryModel>();
 		for (int viewRow : repositoriesTable.getSelectedRows()) {
 			int modelRow = repositoriesTable.convertRowIndexToModel(viewRow);
-			RepositoryModel model = ((RepositoriesModel) repositoriesTable.getModel()).list
-					.get(modelRow);
+			RepositoryModel model = repositoriesModel.list.get(modelRow);
 			repositories.add(model);
 		}
 		return repositories;
@@ -448,50 +379,12 @@
 
 	private List<UserModel> getSelectedUsers() {
 		List<UserModel> users = new ArrayList<UserModel>();
-		for (int viewRow : usersList.getSelectedIndices()) {
-			UserModel model = (UserModel) usersList.getModel().getElementAt(viewRow);
+		for (int viewRow : usersTable.getSelectedRows()) {
+			int modelRow = usersTable.convertRowIndexToModel(viewRow);
+			UserModel model = usersModel.list.get(modelRow);
 			users.add(model);
 		}
 		return users;
-	}
-
-	private void packColumns(JTable table, int margin) {
-		for (int c = 0; c < table.getColumnCount(); c++) {
-			packColumn(table, c, 4);
-		}
-	}
-
-	// Sets the preferred width of the visible column specified by vColIndex.
-	// The column will be just wide enough to show the column head and the
-	// widest cell in the column. margin pixels are added to the left and right
-	// (resulting in an additional width of 2*margin pixels).
-	private void packColumn(JTable table, int vColIndex, int margin) {
-		DefaultTableColumnModel colModel = (DefaultTableColumnModel) table.getColumnModel();
-		TableColumn col = colModel.getColumn(vColIndex);
-		int width = 0;
-
-		// Get width of column header
-		TableCellRenderer renderer = col.getHeaderRenderer();
-		if (renderer == null) {
-			renderer = table.getTableHeader().getDefaultRenderer();
-		}
-		Component comp = renderer.getTableCellRendererComponent(table, col.getHeaderValue(), false,
-				false, 0, 0);
-		width = comp.getPreferredSize().width;
-
-		// Get maximum width of column data
-		for (int r = 0; r < table.getRowCount(); r++) {
-			renderer = table.getCellRenderer(r, vColIndex);
-			comp = renderer.getTableCellRendererComponent(table, table.getValueAt(r, vColIndex),
-					false, false, r, vColIndex);
-			width = Math.max(width, comp.getPreferredSize().width);
-		}
-
-		// Add margin
-		width += 2 * margin;
-
-		// Set the width
-		col.setPreferredWidth(width);
 	}
 
 	@Override
@@ -500,15 +393,24 @@
 	}
 
 	@Override
-	public Dimension getPreferredSize() {
-		if (isAdmin) {
-			return new Dimension(950, 550);
-		}
-		return new Dimension(775, 450);
+	public void closeTab(Component c) {
+		gitblit = null;
 	}
 
-	@Override
-	public void closeTab(Component c) {
+	protected void refreshRepositories() {
+		GitblitWorker worker = new GitblitWorker(GitblitPanel.this, RpcRequest.LIST_REPOSITORIES) {
+			@Override
+			protected Boolean doRequest() throws IOException {
+				gitblit.refreshRepositories();
+				return true;
+			}
+
+			@Override
+			protected void onSuccess() {
+				updateRepositoriesTable();
+			}
+		};
+		worker.execute();
 	}
 
 	/**
@@ -517,14 +419,9 @@
 	 * 
 	 */
 	protected void createRepository() {
-		List<String> usernames = new ArrayList<String>();
-		for (UserModel user : this.allUsers) {
-			usernames.add(user.username);
-		}
-		Collections.sort(usernames);
 		EditRepositoryDialog dialog = new EditRepositoryDialog();
-		dialog.setUsers(null, usernames, null);
-		dialog.setRepositories(allRepositories);
+		dialog.setUsers(null, gitblit.getUsernames(), null);
+		dialog.setRepositories(gitblit.getRepositories());
 		dialog.setVisible(true);
 		final RepositoryModel newRepository = dialog.getRepository();
 		final List<String> permittedUsers = dialog.getPermittedUsers();
@@ -532,44 +429,30 @@
 			return;
 		}
 
-		final RpcRequest request = RpcRequest.CREATE_REPOSITORY;
-		SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
+		GitblitWorker worker = new GitblitWorker(this, RpcRequest.CREATE_REPOSITORY) {
 
 			@Override
-			protected Boolean doInBackground() throws IOException {
-				boolean success = true;
-				success &= RpcUtils.createRepository(newRepository, url, account, password);
-				if (permittedUsers.size() > 0) {
-					// if new repository has named members, set them
-					success &= RpcUtils.setRepositoryMembers(newRepository, permittedUsers, url,
-							account, password);
+			protected Boolean doRequest() throws IOException {
+				boolean success = gitblit.createRepository(newRepository, permittedUsers);
+				if (success) {
+					gitblit.refreshRepositories();
+					if (permittedUsers.size() > 0) {
+						gitblit.refreshUsers();
+					}
 				}
 				return success;
 			}
 
 			@Override
-			protected void done() {
-				try {
-					boolean success = get();
-					if (success) {
-						refreshRepositoriesTable();
-						if (permittedUsers.size() > 0) {
-							refreshUsersTable();
-						}
-					} else {
-						String msg = MessageFormat.format(
-								"Failed to execute request \"{0}\" for repository \"{1}\".",
-								request.name(), newRepository.name);
-						JOptionPane.showMessageDialog(GitblitPanel.this, msg,
-								Translation.get("gb.error"), JOptionPane.ERROR_MESSAGE);
-					}
-				} catch (ForbiddenException e) {
-					explainForbidden(request);
-				} catch (UnauthorizedException e) {
-					explainUnauthorized(request);
-				} catch (Throwable t) {
-					showException(t);
-				}
+			protected void onSuccess() {
+				updateRepositoriesTable();
+				updateUsersTable();
+			}
+
+			@Override
+			protected void onFailure() {
+				showFailure("Failed to execute request \"{0}\" for repository \"{1}\".",
+						getRequestType(), newRepository.name);
 			}
 		};
 		worker.execute();
@@ -583,18 +466,10 @@
 	 */
 	protected void editRepository(final RepositoryModel repository) {
 		EditRepositoryDialog dialog = new EditRepositoryDialog(repository);
-		List<String> members = new ArrayList<String>();
-		List<String> usernames = new ArrayList<String>();
-		for (UserModel user : this.allUsers) {
-			usernames.add(user.username);
-			if (user.repositories.contains(repository.name)) {
-				members.add(user.username);
-			}
-		}
-		Collections.sort(usernames);
+		List<String> usernames = gitblit.getUsernames();
+		List<String> members = gitblit.getPermittedUsernames(repository);
 		dialog.setUsers(repository.owner, usernames, members);
-		dialog.setFederationSets(settings.getStrings(Keys.federation.sets),
-				repository.federationSets);
+		dialog.setFederationSets(gitblit.getFederationSets(), repository.federationSets);
 		dialog.setVisible(true);
 		final RepositoryModel revisedRepository = dialog.getRepository();
 		final List<String> permittedUsers = dialog.getPermittedUsers();
@@ -602,41 +477,29 @@
 			return;
 		}
 
-		final RpcRequest request = RpcRequest.EDIT_REPOSITORY;
-		SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
+		GitblitWorker worker = new GitblitWorker(this, RpcRequest.EDIT_REPOSITORY) {
 
 			@Override
-			protected Boolean doInBackground() throws IOException {
-				boolean success = true;
-				success &= RpcUtils.updateRepository(repository.name, revisedRepository, url,
-						account, password);
-				// always set the repository members
-				success &= RpcUtils.setRepositoryMembers(repository, permittedUsers, url, account,
-						password);
+			protected Boolean doRequest() throws IOException {
+				boolean success = gitblit.updateRepository(repository.name, revisedRepository,
+						permittedUsers);
+				if (success) {
+					gitblit.refreshRepositories();
+					gitblit.refreshUsers();
+				}
 				return success;
 			}
 
 			@Override
-			protected void done() {
-				try {
-					boolean success = get();
-					if (success) {
-						refreshRepositoriesTable();
-						refreshUsersTable();
-					} else {
-						String msg = MessageFormat.format(
-								"Failed to execute request \"{0}\" for repository \"{1}\".",
-								request.name(), repository.name);
-						JOptionPane.showMessageDialog(GitblitPanel.this, msg,
-								Translation.get("gb.error"), JOptionPane.ERROR_MESSAGE);
-					}
-				} catch (ForbiddenException e) {
-					explainForbidden(request);
-				} catch (UnauthorizedException e) {
-					explainUnauthorized(request);
-				} catch (Throwable t) {
-					showException(t);
-				}
+			protected void onSuccess() {
+				updateRepositoriesTable();
+				updateUsersTable();
+			}
+
+			@Override
+			protected void onFailure() {
+				showFailure("Failed to execute request \"{0}\" for repository \"{1}\".",
+						getRequestType(), repository.name);
 			}
 		};
 		worker.execute();
@@ -653,39 +516,48 @@
 		int result = JOptionPane.showConfirmDialog(GitblitPanel.this, message.toString(),
 				"Delete Repositories?", JOptionPane.YES_NO_OPTION);
 		if (result == JOptionPane.YES_OPTION) {
-			final RpcRequest request = RpcRequest.DELETE_REPOSITORY;
-			SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
+			GitblitWorker worker = new GitblitWorker(this, RpcRequest.DELETE_REPOSITORY) {
 				@Override
-				protected Boolean doInBackground() throws Exception {
+				protected Boolean doRequest() throws IOException {
 					boolean success = true;
 					for (RepositoryModel repository : repositories) {
-						success &= RpcUtils.deleteRepository(repository, url, account, password);
+						success &= gitblit.deleteRepository(repository);
+					}
+					if (success) {
+						gitblit.refreshUsers();
 					}
 					return success;
 				}
 
 				@Override
-				protected void done() {
-					try {
-						boolean success = get();
-						if (success) {
-							refreshRepositoriesTable();
-						} else {
-							String msg = "Failed to delete specified repositories!";
-							JOptionPane.showMessageDialog(GitblitPanel.this, msg,
-									Translation.get("gb.error"), JOptionPane.ERROR_MESSAGE);
-						}
-					} catch (ForbiddenException e) {
-						explainForbidden(request);
-					} catch (UnauthorizedException e) {
-						explainUnauthorized(request);
-					} catch (Throwable t) {
-						showException(t);
-					}
+				protected void onSuccess() {
+					updateRepositoriesTable();
+					updateUsersTable();
+				}
+
+				@Override
+				protected void onFailure() {
+					showFailure("Failed to delete specified repositories!");
 				}
 			};
 			worker.execute();
 		}
+	}
+
+	protected void refreshUsers() {
+		GitblitWorker worker = new GitblitWorker(GitblitPanel.this, RpcRequest.LIST_USERS) {
+			@Override
+			protected Boolean doRequest() throws IOException {
+				gitblit.refreshUsers();
+				return true;
+			}
+
+			@Override
+			protected void onSuccess() {
+				updateUsersTable();
+			}
+		};
+		worker.execute();
 	}
 
 	/**
@@ -694,43 +566,35 @@
 	 * 
 	 */
 	protected void createUser() {
-		EditUserDialog dialog = new EditUserDialog(settings);
-		dialog.setUsers(allUsers);
-		dialog.setRepositories(allRepositories, null);
+		EditUserDialog dialog = new EditUserDialog(gitblit.getSettings());
+		dialog.setUsers(gitblit.getUsers());
+		dialog.setRepositories(gitblit.getRepositories(), null);
 		dialog.setVisible(true);
 		final UserModel newUser = dialog.getUser();
 		if (newUser == null) {
 			return;
 		}
 
-		final RpcRequest request = RpcRequest.CREATE_USER;
-		SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
+		GitblitWorker worker = new GitblitWorker(this, RpcRequest.CREATE_USER) {
 
 			@Override
-			protected Boolean doInBackground() throws IOException {
-				return RpcUtils.createUser(newUser, url, account, password);
+			protected Boolean doRequest() throws IOException {
+				boolean success = gitblit.createUser(newUser);
+				if (success) {
+					gitblit.refreshUsers();
+				}
+				return success;
 			}
 
 			@Override
-			protected void done() {
-				try {
-					boolean success = get();
-					if (success) {
-						refreshUsersTable();
-					} else {
-						String msg = MessageFormat.format(
-								"Failed to execute request \"{0}\" for user \"{1}\".",
-								request.name(), newUser.username);
-						JOptionPane.showMessageDialog(GitblitPanel.this, msg,
-								Translation.get("gb.error"), JOptionPane.ERROR_MESSAGE);
-					}
-				} catch (ForbiddenException e) {
-					explainForbidden(request);
-				} catch (UnauthorizedException e) {
-					explainUnauthorized(request);
-				} catch (Throwable t) {
-					showException(t);
-				}
+			protected void onSuccess() {
+				updateUsersTable();
+			}
+
+			@Override
+			protected void onFailure() {
+				showFailure("Failed to execute request \"{0}\" for user \"{1}\".",
+						getRequestType(), newUser.username);
 			}
 		};
 		worker.execute();
@@ -743,42 +607,33 @@
 	 * @param user
 	 */
 	protected void editUser(final UserModel user) {
-		EditUserDialog dialog = new EditUserDialog(user, settings);
-		dialog.setRepositories(allRepositories, new ArrayList<String>(user.repositories));
+		EditUserDialog dialog = new EditUserDialog(user, gitblit.getSettings());
+		dialog.setRepositories(gitblit.getRepositories(), new ArrayList<String>(user.repositories));
 		dialog.setVisible(true);
 		final UserModel revisedUser = dialog.getUser();
 		if (revisedUser == null) {
 			return;
 		}
 
-		final RpcRequest request = RpcRequest.EDIT_USER;
-		SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
-
+		GitblitWorker worker = new GitblitWorker(this, RpcRequest.EDIT_USER) {
 			@Override
-			protected Boolean doInBackground() throws IOException {
-				return RpcUtils.updateUser(user.username, revisedUser, url, account, password);
+			protected Boolean doRequest() throws IOException {
+				boolean success = gitblit.updateUser(user.username, revisedUser);
+				if (success) {
+					gitblit.refreshUsers();
+				}
+				return success;
 			}
 
 			@Override
-			protected void done() {
-				try {
-					boolean success = get();
-					if (success) {
-						refreshUsersTable();
-					} else {
-						String msg = MessageFormat.format(
-								"Failed to execute request \"{0}\" for user \"{1}\".",
-								request.name(), user.username);
-						JOptionPane.showMessageDialog(GitblitPanel.this, msg,
-								Translation.get("gb.error"), JOptionPane.ERROR_MESSAGE);
-					}
-				} catch (ForbiddenException e) {
-					explainForbidden(request);
-				} catch (UnauthorizedException e) {
-					explainUnauthorized(request);
-				} catch (Throwable t) {
-					showException(t);
-				}
+			protected void onSuccess() {
+				updateUsersTable();
+			}
+
+			@Override
+			protected void onFailure() {
+				showFailure("Failed to execute request \"{0}\" for user \"{1}\".",
+						getRequestType(), user.username);
 			}
 		};
 		worker.execute();
@@ -795,58 +650,30 @@
 		int result = JOptionPane.showConfirmDialog(GitblitPanel.this, message.toString(),
 				"Delete Users?", JOptionPane.YES_NO_OPTION);
 		if (result == JOptionPane.YES_OPTION) {
-			final RpcRequest request = RpcRequest.DELETE_USER;
-			SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
+			GitblitWorker worker = new GitblitWorker(this, RpcRequest.DELETE_USER) {
 				@Override
-				protected Boolean doInBackground() throws Exception {
+				protected Boolean doRequest() throws IOException {
 					boolean success = true;
 					for (UserModel user : users) {
-						success &= RpcUtils.deleteUser(user, url, account, password);
+						success &= gitblit.deleteUser(user);
+					}
+					if (success) {
+						gitblit.refreshUsers();
 					}
 					return success;
 				}
 
 				@Override
-				protected void done() {
-					try {
-						boolean success = get();
-						if (success) {
-							refreshUsersTable();
-						} else {
-							String msg = "Failed to delete specified users!";
-							JOptionPane.showMessageDialog(GitblitPanel.this, msg,
-									Translation.get("gb.error"), JOptionPane.ERROR_MESSAGE);
-						}
-					} catch (ForbiddenException e) {
-						explainForbidden(request);
-					} catch (UnauthorizedException e) {
-						explainUnauthorized(request);
-					} catch (Throwable t) {
-						showException(t);
-					}
+				protected void onSuccess() {
+					updateUsersTable();
+				}
+
+				@Override
+				protected void onFailure() {
+					showFailure("Failed to delete specified users!");
 				}
 			};
 			worker.execute();
 		}
 	}
-
-	private void explainForbidden(RpcRequest request) {
-		String msg = MessageFormat.format(
-				"The request \"{0}\" has been forbidden by the Gitblit server @ {1}.",
-				request.name(), url);
-		JOptionPane.showMessageDialog(GitblitPanel.this, msg, "Forbidden",
-				JOptionPane.ERROR_MESSAGE);
-	}
-
-	private void explainUnauthorized(RpcRequest request) {
-		String msg = MessageFormat.format(
-				"The account \"{0}\" is not authorized to execute the request \"{1}\".", account,
-				request.name());
-		JOptionPane.showMessageDialog(GitblitPanel.this, msg, "Unauthorized",
-				JOptionPane.ERROR_MESSAGE);
-	}
-
-	private void showException(Throwable t) {
-		// TODO show the unexpected exception
-	}
-}
+}
\ No newline at end of file

--
Gitblit v1.9.1