Ignore permission definitions for admins, it just confuses things

James Moger

2012-11-01 b701ed7c4e138c4aaa3acb029f6e35fdf01388e4

 src/com/gitblit/wicket/pages/EditUserPage.java

@@ -34,12 +34,13 @@
import org.apache.wicket.model.util.CollectionModel;

import org.apache.wicket.model.util.ListModel;



import com.gitblit.Constants.PermissionType;

import com.gitblit.Constants.RegistrantType;

import com.gitblit.GitBlit;

import com.gitblit.GitBlitException;

import com.gitblit.Keys;

import com.gitblit.Constants.PermissionType;

import com.gitblit.Constants.RegistrantType;

import com.gitblit.models.RegistrantAccessPermission;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.StringUtils;

@@ -76,6 +77,11 @@
      setStatelessHint(false);

      setOutputMarkupId(true);

   }

	
   @Override

   protected boolean requiresPageMap() {

      return true;

   }



   protected void setupPage(final UserModel userModel) {

      if (isCreate) {

@@ -100,14 +106,20 @@
      final String oldName = userModel.username;

      final List<RegistrantAccessPermission> permissions = userModel.getRepositoryPermissions();

      for (RegistrantAccessPermission permission : permissions) {

         if (permission.isEditable && PermissionType.EXPLICIT.equals(permission.permissionType)) {

         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.

            boolean isOwner = GitBlit.self().getRepositoryModel(permission.registrant).isOwner(oldName);

            RepositoryModel rm = GitBlit.self().getRepositoryModel(permission.registrant);

            if (rm == null) {

               permission.permissionType = PermissionType.MISSING;

               permission.mutable = false;

               continue;

            }

            boolean isOwner = rm.isOwner(oldName);

            if (isOwner) {

               permission.permissionType = PermissionType.OWNER;

               permission.isEditable = false;

               permission.mutable = false;

            }

         }

      }