James Moger
2012-11-01 b701ed7c4e138c4aaa3acb029f6e35fdf01388e4
src/com/gitblit/wicket/pages/EditUserPage.java
@@ -33,7 +33,6 @@
import org.apache.wicket.model.Model;
import org.apache.wicket.model.util.CollectionModel;
import org.apache.wicket.model.util.ListModel;
import org.slf4j.LoggerFactory;
import com.gitblit.Constants.PermissionType;
import com.gitblit.Constants.RegistrantType;
@@ -107,19 +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.
            RepositoryModel rm = GitBlit.self().getRepositoryModel(permission.registrant);
            if (rm == null) {
               LoggerFactory.getLogger(getClass()).error("Missing repository " + permission.registrant, new NullPointerException());
               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;
            }
         }
      }