[java-identity-provider] branch main updated: Revert "IDP-1795 Complete the removal of Collections from IdPAttributes"

[Rod Widdowson]

This is an automated email from the git hooks/post-receive script.

rdw pushed a commit to branch main
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=bf3f7d07fe15ff9b4655a9cc7d707c4528b30435

The following commit(s) were added to refs/heads/main by this push:
       new  bf3f7d07f Revert "IDP-1795 Complete the removal of Collections from IdPAttributes"
bf3f7d07f is described below

commit bf3f7d07fe15ff9b4655a9cc7d707c4528b30435
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Tue Apr 13 12:38:59 2021 +0100

    Revert "IDP-1795 Complete the removal of Collections from IdPAttributes"
    
    This reverts commit 78da41cf3f9081c2398e8c1c57f41958f0ab4890.
---
 .../attribute/filter/spring/AttributeFilterServiceTest.java  |  4 ++--
 .../resolver/ad/impl/SimpleAttributeDefinition.java          | 12 ------------
 .../dc/storage/impl/SimpleStorageMappingStrategy.java        |  2 +-
 3 files changed, 3 insertions(+), 15 deletions(-)

diff --git a/idp-attribute-filter-spring/src/test/java/net/shibboleth/idp/attribute/filter/spring/AttributeFilterServiceTest.java b/idp-attribute-filter-spring/src/test/java/net/shibboleth/idp/attribute/filter/spring/AttributeFilterServiceTest.java
index 8f323d4b8..11b46362e 100644
--- a/idp-attribute-filter-spring/src/test/java/net/shibboleth/idp/attribute/filter/spring/AttributeFilterServiceTest.java
+++ b/idp-attribute-filter-spring/src/test/java/net/shibboleth/idp/attribute/filter/spring/AttributeFilterServiceTest.java
@@ -99,11 +99,11 @@ public class AttributeFilterServiceTest {
         attributesToBeFiltered = new HashMap<>();
 
         IdPAttribute firstName = new IdPAttribute("firstName");
-        firstName.setValues(Collections.singletonList(new StringAttributeValue("john")));
+        firstName.setValues(Collections.singleton(new StringAttributeValue("john")));
         attributesToBeFiltered.put(firstName.getId(), firstName);
 
         IdPAttribute lastName = new IdPAttribute("lastName");
-        lastName.setValues(Collections.singletonList(new StringAttributeValue("smith")));
+        lastName.setValues(Collections.singleton(new StringAttributeValue("smith")));
         attributesToBeFiltered.put(lastName.getId(), lastName);
 
         IdPAttribute email = new IdPAttribute("email");
diff --git a/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/ad/impl/SimpleAttributeDefinition.java b/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/ad/impl/SimpleAttributeDefinition.java
index aa8a496af..f8287094e 100644
--- a/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/ad/impl/SimpleAttributeDefinition.java
+++ b/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/ad/impl/SimpleAttributeDefinition.java
@@ -27,7 +27,6 @@ import net.shibboleth.idp.attribute.resolver.ResolutionException;
 import net.shibboleth.idp.attribute.resolver.context.AttributeResolutionContext;
 import net.shibboleth.idp.attribute.resolver.context.AttributeResolverWorkContext;
 import net.shibboleth.utilities.java.support.component.ComponentInitializationException;
-import net.shibboleth.utilities.java.support.component.ComponentSupport;
 import net.shibboleth.utilities.java.support.logic.Constraint;
 
 /**
@@ -36,17 +35,6 @@ import net.shibboleth.utilities.java.support.logic.Constraint;
  */
 @ThreadSafe
 public class SimpleAttributeDefinition extends AbstractAttributeDefinition {
-    
-    /** Do we filter null/empty values out? */
-    private boolean filterNullValues;
-    
-    /** Do we filter null/empty values out?
-     * @param what what to set.
-     */
-    public void setFilterNullValues(boolean what) {
-        ComponentSupport.ifInitializedThrowUnmodifiabledComponentException(this);
-        filterNullValues = what;
-    }
 
     /** {@inheritDoc} */
     @Override @Nonnull protected IdPAttribute doAttributeDefinitionResolve(
diff --git a/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/storage/impl/SimpleStorageMappingStrategy.java b/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/storage/impl/SimpleStorageMappingStrategy.java
index aef12b50e..0db5484b2 100644
--- a/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/storage/impl/SimpleStorageMappingStrategy.java
+++ b/idp-attribute-resolver-impl/src/main/java/net/shibboleth/idp/attribute/resolver/dc/storage/impl/SimpleStorageMappingStrategy.java
@@ -58,7 +58,7 @@ public class SimpleStorageMappingStrategy implements StorageMappingStrategy {
             @Nonnull final StorageRecord<?> results) throws ResolutionException {
         
         final IdPAttribute attribute = new IdPAttribute(attributeId);
-        attribute.setValues(Collections.singletonList(StringAttributeValue.valueOf(results.getValue())));
+        attribute.setValues(Collections.singleton(StringAttributeValue.valueOf(results.getValue())));
         
         return Collections.singletonMap(attributeId, attribute);
     }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.