[java-identity-provider COMMIT] /trunk/idp-saml-api/src/main/java/net/shibboleth/idp/saml/attribute/encoding/Abstract...

[noreply at shibboleth.net]

Author: rdw
Date: Sat Dec 17 14:23:17 2011
New Revision: 4089

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=4089&view=rev
Log:
Do not encode an attribute if the parent class cannot handle the (java) type

Modified:
    trunk/idp-saml-api/src/main/java/net/shibboleth/idp/saml/attribute/encoding/AbstractSamlAttributeEncoder.java

Modified: trunk/idp-saml-api/src/main/java/net/shibboleth/idp/saml/attribute/encoding/AbstractSamlAttributeEncoder.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-saml-api/src/main/java/net/shibboleth/idp/saml/attribute/encoding/AbstractSamlAttributeEncoder.java?rev=4089&r1=4088&r2=4089&view=diff
==============================================================================
--- trunk/idp-saml-api/src/main/java/net/shibboleth/idp/saml/attribute/encoding/AbstractSamlAttributeEncoder.java (original)
+++ trunk/idp-saml-api/src/main/java/net/shibboleth/idp/saml/attribute/encoding/AbstractSamlAttributeEncoder.java Sat Dec 17 14:23:17 2011
@@ -135,9 +135,10 @@
                 continue;
             }
 
-            if (canEncodeValue(attribute, o)) {
+            if (!canEncodeValue(attribute, o)) {
                 log.debug("Skipping value of attribute {}; it can not be encoded by this encoder.", attributeId, o
                         .getClass().getName());
+                return null;
             }
 
             attributeValue = (EncodedType) o;