summaryrefslogtreecommitdiffstats
path: root/src/com/gitblit/client/UsersPanel.java
diff options
context:
space:
mode:
authorJames Moger <james.moger@gitblit.com>2012-11-01 09:12:55 -0400
committerJames Moger <james.moger@gitblit.com>2012-11-01 09:12:55 -0400
commit7ba85bfa11c7fcab21ada61650fe30763aafd7b0 (patch)
treeb08e6aa1a0f4dbd0a90eaa7257d348983721c40b /src/com/gitblit/client/UsersPanel.java
parent40b07bca7d02438cd0d660f3b1713ffa86f6df76 (diff)
downloadgitblit-7ba85bfa11c7fcab21ada61650fe30763aafd7b0.tar.gz
gitblit-7ba85bfa11c7fcab21ada61650fe30763aafd7b0.zip
Gracefully deal with missing repository in permissions ui (issue 155)
Diffstat (limited to 'src/com/gitblit/client/UsersPanel.java')
-rw-r--r--src/com/gitblit/client/UsersPanel.java23
1 files changed, 1 insertions, 22 deletions
diff --git a/src/com/gitblit/client/UsersPanel.java b/src/com/gitblit/client/UsersPanel.java
index 2c236958..469d9536 100644
--- a/src/com/gitblit/client/UsersPanel.java
+++ b/src/com/gitblit/client/UsersPanel.java
@@ -25,7 +25,6 @@ import java.awt.event.KeyEvent;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.io.IOException;
-import java.text.MessageFormat;
import java.util.ArrayList;
import java.util.List;
@@ -41,6 +40,7 @@ import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;
import javax.swing.table.TableRowSorter;
+import com.gitblit.Constants.AccessPermission;
import com.gitblit.Constants.PermissionType;
import com.gitblit.Constants.RpcRequest;
import com.gitblit.models.RegistrantAccessPermission;
@@ -313,27 +313,6 @@ public abstract class UsersPanel extends JPanel {
gitblit.getSettings());
dialog.setLocationRelativeTo(UsersPanel.this);
dialog.setUsers(gitblit.getUsers());
-
- List<RegistrantAccessPermission> permissions = user.getRepositoryPermissions();
- for (RegistrantAccessPermission permission : permissions) {
- if (permission.mutable && PermissionType.EXPLICIT.equals(permission.permissionType)) {
- // Ensure this is NOT an owner permission - which is non-editable
- // We don't know this from within the usermodel, ownership is a
- // property of a repository.
- RepositoryModel rm = gitblit.getRepository(permission.registrant);
- if (rm == null) {
- System.out.println(MessageFormat.format("{0}: failed to find registrant repository {1}",
- getClass().getSimpleName(), permission.registrant));
- continue;
- }
- boolean isOwner = rm.isOwner(user.username);
- if (isOwner) {
- permission.permissionType = PermissionType.OWNER;
- permission.mutable = false;
- }
- }
- }
-
dialog.setRepositories(gitblit.getRepositories(), user.getRepositoryPermissions());
dialog.setTeams(gitblit.getTeams(), user.teams == null ? null : new ArrayList<TeamModel>(
user.teams));