Permission regexes are now case-insensitive

James Moger

2012-10-22 e5aaa5db9b323f58d9eb8338532fd04fce885048

 src/com/gitblit/models/TeamModel.java

@@ -18,6 +18,7 @@
import java.io.Serializable;

import java.util.ArrayList;

import java.util.Collection;

import java.util.Collections;

import java.util.HashMap;

import java.util.HashSet;

import java.util.List;

@@ -26,7 +27,9 @@


import com.gitblit.Constants.AccessPermission;

import com.gitblit.Constants.AccessRestrictionType;

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

@@ -85,6 +88,21 @@
   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()) {

         list.add(new RegistrantAccessPermission(entry.getKey(), entry.getValue(), true, RegistrantType.REPOSITORY));

      }

      Collections.sort(list);

      return list;

   }

   

   /**

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

@@ -110,6 +128,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,9 +185,9 @@
            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;