[jboss-cvs] JBossAS SVN: r68745 - in projects/security/security-spi/trunk/spi/src/main/org/jboss/security: mapping and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2008-01-09 15:05:02 -0500 (Wed, 09 Jan 2008)
New Revision: 68745

Modified:
   projects/security/security-spi/trunk/spi/src/main/org/jboss/security/ISecurityManagement.java
   projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingContext.java
   projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingManager.java
Log:
correct generics and optimize mappingcontext

Modified: projects/security/security-spi/trunk/spi/src/main/org/jboss/security/ISecurityManagement.java
===================================================================
--- projects/security/security-spi/trunk/spi/src/main/org/jboss/security/ISecurityManagement.java	2008-01-09 20:03:58 UTC (rev 68744)
+++ projects/security/security-spi/trunk/spi/src/main/org/jboss/security/ISecurityManagement.java	2008-01-09 20:05:02 UTC (rev 68745)
@@ -52,7 +52,7 @@
    
    /** 
     * Mapping manager configured with providers
-    * @param securityDomain the SecurityDomain
+    * @param securityDomain the SecurityDomain 
     * @return 
     */ 
    public MappingManager getMappingManager(String securityDomain);

Modified: projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingContext.java
===================================================================
--- projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingContext.java	2008-01-09 20:03:58 UTC (rev 68744)
+++ projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingContext.java	2008-01-09 20:05:02 UTC (rev 68745)
@@ -55,8 +55,22 @@
       } 
    } 
    
+   /**
+    * 
+    * @return Result of previous mapping operation
+    */
    public MappingResult<T> getMappingResult()
    {
       return result;
    }
+   
+   /**
+    * Optimization Step to determine if we have configured mapping modules
+    * to avoid unnecessary mapping step
+    * @return true - at least one mapping provider is configured
+    */
+   public boolean hasModules()
+   {
+      return this.modules.size() > 0;
+   }
 }
\ No newline at end of file

Modified: projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingManager.java
===================================================================
--- projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingManager.java	2008-01-09 20:03:58 UTC (rev 68744)
+++ projects/security/security-spi/trunk/spi/src/main/org/jboss/security/mapping/MappingManager.java	2008-01-09 20:05:02 UTC (rev 68745)
@@ -29,7 +29,7 @@
  *  @since  Mar 9, 2007 
  *  @version $Revision$
  */
-public interface MappingManager<T> extends BaseSecurityManager
+public interface MappingManager extends BaseSecurityManager
 {
-   MappingContext<T> getMappingContext(Class<T> mappingType); 
+   <T> MappingContext<T> getMappingContext(Class<T> mappingType); 
 }
\ No newline at end of file