Moved owner field onto access permissions tab

James Moger

2012-10-30 abcb146f0c4e5c99e5e44349a65e6fd49e9296d9

 src/com/gitblit/models/TeamModel.java

@@ -18,15 +18,19 @@
import java.io.Serializable;

import java.util.ArrayList;

import java.util.Collection;

import java.util.HashMap;

import java.util.Collections;

import java.util.HashSet;

import java.util.LinkedHashMap;

import java.util.List;

import java.util.Map;

import java.util.Set;



import com.gitblit.Constants.AccessPermission;

import com.gitblit.Constants.AccessRestrictionType;

import com.gitblit.Constants.PermissionType;

import com.gitblit.Constants.RegistrantType;

import com.gitblit.Constants.Unused;

import com.gitblit.utils.StringUtils;



/**

 * TeamModel is a serializable model class that represents a group of users and

@@ -48,7 +52,7 @@
   // retained for backwards-compatibility with RPC clients

   @Deprecated

   public final Set<String> repositories = new HashSet<String>();

   public final Map<String, AccessPermission> permissions = new HashMap<String, AccessPermission>();

   public final Map<String, AccessPermission> permissions = new LinkedHashMap<String, AccessPermission>();

   public final Set<String> mailingLists = new HashSet<String>();

   public final List<String> preReceiveScripts = new ArrayList<String>();

   public final List<String> postReceiveScripts = new ArrayList<String>();

@@ -85,6 +89,28 @@
   public void removeRepository(String name) {

      removeRepositoryPermission(name);

   }



	
   /**

    * Returns a list of repository permissions for this team.

    * 
    * @return the team's list of permissions

    */

   public List<RegistrantAccessPermission> getRepositoryPermissions() {

      List<RegistrantAccessPermission> list = new ArrayList<RegistrantAccessPermission>();

      for (Map.Entry<String, AccessPermission> entry : permissions.entrySet()) {

         String registrant = entry.getKey();

         boolean editable = true;

         PermissionType pType = PermissionType.EXPLICIT;

         if (StringUtils.findInvalidCharacter(registrant) != null) {

            // a regex will have at least 1 invalid character

            pType = PermissionType.REGEX;

         }

         list.add(new RegistrantAccessPermission(registrant, entry.getValue(), pType, RegistrantType.REPOSITORY, editable));

      }

      Collections.sort(list);

      return list;

   }

   

   /**

    * Returns true if the team has any type of specified access permission for

@@ -110,6 +136,18 @@
         }

      }

      return false;

   }

	
   /**

    * Returns true if the team has an explicitly specified access permission for

    * this repository.

    * 
    * @param name

    * @return if the team has an explicitly specified access permission

    */

   public boolean hasExplicitRepositoryPermission(String name) {

      String repository = AccessPermission.repositoryFromRole(name).toLowerCase();

      return permissions.containsKey(repository);

   }

   

   /**

@@ -155,12 +193,14 @@
            permission = p;

         }

      } else {

         // search for regex permission match

         // search for case-insensitive regex permission match

         for (String key : permissions.keySet()) {

            if (repository.name.matches(key)) {

            if (StringUtils.matchesIgnoreCase(repository.name, key)) {

               AccessPermission p = permissions.get(key);

               if (p != null) {

                  permission = p;

                  // take first match

                  break;

               }

            }

         }

@@ -168,7 +208,7 @@
      return permission;

   }

   

   private boolean canAccess(RepositoryModel repository, AccessRestrictionType ifRestriction, AccessPermission requirePermission) {

   protected boolean canAccess(RepositoryModel repository, AccessRestrictionType ifRestriction, AccessPermission requirePermission) {

      if (repository.accessRestriction.atLeast(ifRestriction)) {

         AccessPermission permission = getRepositoryPermission(repository);

         return permission.atLeast(requirePermission);