From: cantor <cantor@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Date: Thu, 20 Oct 2005 16:30:44 +0000 (+0000)
Subject: Fixed attribute value consumption in AA/IdP roles.
X-Git-Tag: v2.1.3~866
X-Git-Url: https://repo.niif.hu/gitweb/gitweb.cgi?p=java-idp.git;a=commitdiff_plain;h=38bd2e6124aca3524d0c2c90ee42f103f3185e0e;hp=a50467d563236171d6583920f394f4cdb76be0d3

Fixed attribute value consumption in AA/IdP roles.


git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/trunk@1887 ab3bd59b-922f-494d-bb5f-6f0a3c29deca
---

diff --git a/src/edu/internet2/middleware/shibboleth/metadata/provider/XMLMetadataProvider.java b/src/edu/internet2/middleware/shibboleth/metadata/provider/XMLMetadataProvider.java
index 1f35333..1099300 100644
--- a/src/edu/internet2/middleware/shibboleth/metadata/provider/XMLMetadataProvider.java
+++ b/src/edu/internet2/middleware/shibboleth/metadata/provider/XMLMetadataProvider.java
@@ -749,7 +749,6 @@ public class XMLMetadataProvider implements Metadata, PluggableConfigurationComp
                     copy.setAttributeNS(null,"AttributeNamespace",src.getAttributeNS(null,"NameFormat"));
                     src=XML.getFirstChildElement(src,edu.internet2.middleware.shibboleth.common.XML.SAML2ASSERT_NS,"AttributeValue");
                     while (src != null) {
-                        src=XML.getNextSiblingElement(src,edu.internet2.middleware.shibboleth.common.XML.SAML2ASSERT_NS,"AttributeValue");
                         Element val=e.getOwnerDocument().createElementNS(XML.SAML_NS,"AttributeValue");
                         NamedNodeMap attrs = src.getAttributes();
                         for (int j=0; j<attrs.getLength(); j++)
@@ -757,6 +756,7 @@ public class XMLMetadataProvider implements Metadata, PluggableConfigurationComp
                         while (src.hasChildNodes())
                             val.appendChild(src.getFirstChild());
                         copy.appendChild(val);
+                        src=XML.getNextSiblingElement(src,edu.internet2.middleware.shibboleth.common.XML.SAML2ASSERT_NS,"AttributeValue");
                     }
                     attrs.add(SAMLAttribute.getInstance(copy));
                 }
@@ -875,7 +875,6 @@ public class XMLMetadataProvider implements Metadata, PluggableConfigurationComp
                     copy.setAttributeNS(null,"AttributeNamespace",src.getAttributeNS(null,"NameFormat"));
                     src=XML.getFirstChildElement(src,edu.internet2.middleware.shibboleth.common.XML.SAML2ASSERT_NS,"AttributeValue");
                     while (src != null) {
-                        src=XML.getNextSiblingElement(src,edu.internet2.middleware.shibboleth.common.XML.SAML2ASSERT_NS,"AttributeValue");
                         Element val=e.getOwnerDocument().createElementNS(XML.SAML_NS,"AttributeValue");
                         NamedNodeMap attrs = src.getAttributes();
                         for (int j=0; j<attrs.getLength(); j++)
@@ -883,6 +882,7 @@ public class XMLMetadataProvider implements Metadata, PluggableConfigurationComp
                         while (src.hasChildNodes())
                             val.appendChild(src.getFirstChild());
                         copy.appendChild(val);
+                        src=XML.getNextSiblingElement(src,edu.internet2.middleware.shibboleth.common.XML.SAML2ASSERT_NS,"AttributeValue");
                     }
                     attrs.add(SAMLAttribute.getInstance(copy));
                 }