Improve effective permission determination

James Moger

2013-05-02 9a6a428bad30be341e4df2c6b0f77d9b9e9881ca

 src/main/java/com/gitblit/Constants.java

@@ -147,6 +147,28 @@
      public String toString() {

         return name();

      }

		
      public boolean isValidPermission(AccessPermission permission) {

         switch (this) {

         case VIEW:

            // VIEW restriction

            // all access permissions are valid

            return true;

         case CLONE:

            // CLONE restriction

            // only CLONE or greater access permissions are valid

            return permission.atLeast(AccessPermission.CLONE);

         case PUSH:

            // PUSH restriction

            // only PUSH or greater access permissions are valid 
            return permission.atLeast(AccessPermission.PUSH);

         case NONE:

            // NO access restriction

            // all access permissions are invalid

            return false;

         }

         return false;

      }

   }

   

   /**

@@ -416,7 +438,7 @@
   }

   

   public static enum PermissionType {

      MISSING, EXPLICIT, TEAM, REGEX, OWNER, ADMINISTRATOR;

      MISSING, ANONYMOUS, EXPLICIT, TEAM, REGEX, OWNER, ADMINISTRATOR;

   }

   

   public static enum GCStatus {