[jboss-cvs] JBossAS SVN: r73391 - in trunk/security/src/main/org/jboss/security/microcontainer/beans: metadata and 1 other directory.

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

Author: anil.saldhana at jboss.com
Date: 2008-05-14 05:47:41 -0400 (Wed, 14 May 2008)
New Revision: 73391

Modified:
   trunk/security/src/main/org/jboss/security/microcontainer/beans/ApplicationPolicyBean.java
   trunk/security/src/main/org/jboss/security/microcontainer/beans/metadata/ApplicationPolicyMetaDataFactory.java
Log:
minor fixes

Modified: trunk/security/src/main/org/jboss/security/microcontainer/beans/ApplicationPolicyBean.java
===================================================================
--- trunk/security/src/main/org/jboss/security/microcontainer/beans/ApplicationPolicyBean.java	2008-05-14 08:37:09 UTC (rev 73390)
+++ trunk/security/src/main/org/jboss/security/microcontainer/beans/ApplicationPolicyBean.java	2008-05-14 09:47:41 UTC (rev 73391)
@@ -21,6 +21,7 @@
  */
 package org.jboss.security.microcontainer.beans;
 
+import org.jboss.logging.Logger;
 import org.jboss.security.auth.login.XMLLoginConfigImpl;
 import org.jboss.security.config.ApplicationPolicy;
 import org.jboss.security.config.ApplicationPolicyRegistration;
@@ -69,9 +70,10 @@
 
    /** the identity-trust policy configured as part of this application policy. */
    private IdentityTrustPolicyBean identityTrustPolicy;
-
-   // TODO: add logging to the metadata factories and also to the beans.
-
+   
+   protected static Logger log = Logger.getLogger(ApplicationPolicyBean.class);
+   protected boolean trace = log.isTraceEnabled();
+   
    /**
     * <p>
     * Obtains the name of the application policy.
@@ -253,8 +255,7 @@
 
       // TODO: get the ApplicationPolicyRegistration implementation through injection.
       ApplicationPolicyRegistration policyRegistration = XMLLoginConfigImpl.getInstance();
-      policyRegistration.addApplicationPolicy(this.name, policy);
-
+      policyRegistration.addApplicationPolicy(this.name, policy);  
    }
 
    /**
@@ -267,7 +268,7 @@
    public void stop() throws Exception
    {
       // unregister the application policy.
-      XMLLoginConfigImpl.getInstance().removeApplicationPolicy(this.name);
+      XMLLoginConfigImpl.getInstance().removeApplicationPolicy(this.name); 
    }
 
    /**
@@ -310,4 +311,4 @@
 
       return buffer.toString();
    }
-}
+}
\ No newline at end of file

Modified: trunk/security/src/main/org/jboss/security/microcontainer/beans/metadata/ApplicationPolicyMetaDataFactory.java
===================================================================
--- trunk/security/src/main/org/jboss/security/microcontainer/beans/metadata/ApplicationPolicyMetaDataFactory.java	2008-05-14 08:37:09 UTC (rev 73390)
+++ trunk/security/src/main/org/jboss/security/microcontainer/beans/metadata/ApplicationPolicyMetaDataFactory.java	2008-05-14 09:47:41 UTC (rev 73391)
@@ -282,7 +282,7 @@
          policyBuilder.addPropertyMetaData("authorizationPolicy", injectAuthorizationPolicy);
       }
 
-      // role-mappoing metadata.
+      // role-mapping metadata.
       if (this.mappingMetaData != null)
       {
          // get the metadata for the role-mapping policy bean.