Fix typo in default config
[java-idp.git] / src / edu / internet2 / middleware / shibboleth / idp / config / profile / ProfileHandlerGroupBeanDefinitionParser.java
index 09d7334..605b540 100644 (file)
@@ -21,58 +21,42 @@ import java.util.Map;
 
 import javax.xml.namespace.QName;
 
-import org.apache.log4j.Logger;
 import org.opensaml.xml.util.XMLHelper;
-import org.springframework.beans.factory.support.AbstractBeanDefinition;
-import org.springframework.beans.factory.support.BeanDefinitionBuilder;
-import org.springframework.beans.factory.xml.AbstractBeanDefinitionParser;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.springframework.beans.factory.config.BeanDefinition;
+import org.springframework.beans.factory.xml.BeanDefinitionParser;
 import org.springframework.beans.factory.xml.ParserContext;
 import org.w3c.dom.Element;
 
 import edu.internet2.middleware.shibboleth.common.config.SpringConfigurationUtils;
 
-/**
- * Spring bean definition parser for profile handler root element.
- */
-public class ProfileHandlerGroupBeanDefinitionParser extends AbstractBeanDefinitionParser {
-    
-    /** Class logger. */
-    private static Logger log = Logger.getLogger(ProfileHandlerGroupBeanDefinitionParser.class);
+/** Spring bean definition parser for profile handler root element. */
+public class ProfileHandlerGroupBeanDefinitionParser implements BeanDefinitionParser {
 
     /** Schema type name. */
     public static final QName SCHEMA_TYPE = new QName(ProfileHandlerNamespaceHandler.NAMESPACE, "ProfileHandlerGroup");
 
-    /** {@inheritDoc} */
-    protected AbstractBeanDefinition parseInternal(Element config, ParserContext context) {
-        BeanDefinitionBuilder builder = BeanDefinitionBuilder.rootBeanDefinition(ProfileHandlerGroup.class);
+    /** Class logger. */
+    private static Logger log = LoggerFactory.getLogger(ProfileHandlerGroupBeanDefinitionParser.class);
 
+    /** {@inheritDoc} */
+    public BeanDefinition parse(Element config, ParserContext context) {
         Map<QName, List<Element>> configChildren = XMLHelper.getChildElements(config);
         List<Element> children;
 
         children = configChildren.get(new QName(ProfileHandlerNamespaceHandler.NAMESPACE, "ErrorHandler"));
-        if(log.isDebugEnabled()){
-            log.debug(children.size() + " error handler definitions found");
-        }
-        builder.addPropertyValue("errorHandler", SpringConfigurationUtils.parseCustomElement(children.get(0), context));
+        log.debug("{} error handler definitions found", children.size());
+        SpringConfigurationUtils.parseCustomElement(children.get(0), context);
 
         children = configChildren.get(new QName(ProfileHandlerNamespaceHandler.NAMESPACE, "ProfileHandler"));
-        if(log.isDebugEnabled()){
-            log.debug(children.size() + " profile handler definitions found");
-        }
-        builder.addPropertyValue("profileHandlers", SpringConfigurationUtils.parseCustomElements(children, context));
-
-        children = configChildren.get(new QName(ProfileHandlerNamespaceHandler.NAMESPACE, "AuthenticationHandler"));
-        if(log.isDebugEnabled()){
-            log.debug(children.size() + " authentication handler definitions found");
-        }
-        builder.addPropertyValue("authenticationHandlers", SpringConfigurationUtils.parseCustomElements(children,
-                context));
+        log.debug("{} profile handler definitions found", children.size());
+        SpringConfigurationUtils.parseCustomElements(children, context);
 
-        return builder.getBeanDefinition();
-    }
+        children = configChildren.get(new QName(ProfileHandlerNamespaceHandler.NAMESPACE, "LoginHandler"));
+        log.debug("{} login handler definitions found", children.size());
+        SpringConfigurationUtils.parseCustomElements(children, context);
 
-    /** {@inheritDoc} */
-    protected boolean shouldGenerateId() {
-        return true;
+        return null;
     }
 }
\ No newline at end of file