[jboss-cvs] Picketlink SVN: r1391 - federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2.

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

Author: anil.saldhana at jboss.com
Date: 2012-02-14 16:51:15 -0500 (Tue, 14 Feb 2012)
New Revision: 1391

Modified:
   federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AttributeHandler.java
Log:
PLFED-262: fix the cce

Modified: federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AttributeHandler.java
===================================================================
--- federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AttributeHandler.java	2012-02-14 14:11:15 UTC (rev 1390)
+++ federation/trunk/picketlink-web/src/main/java/org/picketlink/identity/federation/web/handlers/saml2/SAML2AttributeHandler.java	2012-02-14 21:51:15 UTC (rev 1391)
@@ -91,7 +91,6 @@
       }
    }
 
-   @SuppressWarnings("unchecked")
    @Override
    public void initHandlerConfig(SAML2HandlerConfig handlerConfig) throws ConfigurationException
    {
@@ -99,9 +98,12 @@
 
       String attribStr = (String) this.handlerConfig.getParameter(GeneralConstants.ATTIBUTE_MANAGER);
       this.insantiateAttributeManager(attribStr);
-      List<String> ak = (List<String>) this.handlerConfig.getParameter(GeneralConstants.ATTRIBUTE_KEYS);
-      if (ak != null)
-         this.attributeKeys.addAll(ak);
+      //Get a list of attributes we are interested in
+      String attribList = (String) this.handlerConfig.getParameter(GeneralConstants.ATTRIBUTE_KEYS);
+      if (StringUtil.isNotNull(attribList))
+      {
+         this.attributeKeys.addAll(StringUtil.tokenize(attribList));
+      }
 
       String chooseFriendlyNameStr = (String) handlerConfig
             .getParameter(GeneralConstants.ATTRIBUTE_CHOOSE_FRIENDLY_NAME);