[java-identity-provider] branch main updated: Checkstyle and javadoc.

Scott Cantor cantor.2 at osu.edu
Mon Feb 19 13:32:25 UTC 2024


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

scantor 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=d8d6e9267600ced12ecdd64e26357579f1efffe0

The following commit(s) were added to refs/heads/main by this push:
     new d8d6e9267 Checkstyle and javadoc.
d8d6e9267 is described below

commit d8d6e9267600ced12ecdd64e26357579f1efffe0
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Mon Feb 19 08:32:22 2024 -0500

    Checkstyle and javadoc.
---
 .../shibboleth/idp/ui/helper/AttributeHelper.java  | 24 ++++++++++++++--------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/idp-ui/src/main/java/net/shibboleth/idp/ui/helper/AttributeHelper.java b/idp-ui/src/main/java/net/shibboleth/idp/ui/helper/AttributeHelper.java
index b72c290e5..a6f67c144 100644
--- a/idp-ui/src/main/java/net/shibboleth/idp/ui/helper/AttributeHelper.java
+++ b/idp-ui/src/main/java/net/shibboleth/idp/ui/helper/AttributeHelper.java
@@ -38,7 +38,8 @@ public final class AttributeHelper extends AbstractIdentifiableInitializableComp
     /** How to get the AttributeContext we are looking at. */
     @SuppressWarnings("null")
         @Nonnull private Function<ProfileRequestContext,AttributeContext> attributeContextStrategy =
-            new ChildContextLookup<>(AttributeContext.class).compose(new ChildContextLookup<>(RelyingPartyContext.class));
+            new ChildContextLookup<>(AttributeContext.class).compose(
+                    new ChildContextLookup<>(RelyingPartyContext.class));
 
     /** Logger. */
     @Nonnull private final Logger log = LoggerFactory.getLogger(AttributeHelper.class);
@@ -46,8 +47,7 @@ public final class AttributeHelper extends AbstractIdentifiableInitializableComp
     /** Set the way to get the {@link AttributeContext}.
      * @param strategy the strategy to use
      */
-    public void setAttributeContextStrategy(@Nonnull
-            Function<ProfileRequestContext, AttributeContext> strategy) {
+    public void setAttributeContextStrategy(@Nonnull final Function<ProfileRequestContext, AttributeContext> strategy) {
         attributeContextStrategy = Constraint.isNotNull(strategy, "Injected strategy must not be null");
     }
 
@@ -121,10 +121,13 @@ public final class AttributeHelper extends AbstractIdentifiableInitializableComp
     }
 
 
-    /** Return the first (unfiltered) attribute Value from the attribute of that name as a Display String
+    /**
+     * Return the first (unfiltered) attribute Value from the attribute of that name as a Display String.
+     * 
      * @param prc The ProfileRequestContext
      * @param attributeName The attribute name to look up
-     * @param defaultValue What to return if nothing found.
+     * @param defaultValue What to return if nothing found
+     * 
      * @return The default value or the attribute value
      */
     @Nonnull public String getFirstUnfilteredAttributeDisplayValue(final ProfileRequestContext prc,
@@ -150,16 +153,19 @@ public final class AttributeHelper extends AbstractIdentifiableInitializableComp
         return getFirstUnfilteredAttributeDisplayValue(prc, attributeName, "");
     }
 
-    /** Helper method to get the first attribute name from the attribute.
-     * @param Attribute the Attribute or null if there wasn't one
+    /**
+     * Helper method to get the first attribute name from the attribute.
+     * 
+     * @param attribute the Attribute or null if there wasn't one
+     * 
      * @return the first attribute valkue or null
      */
-    @Nullable private IdPAttributeValue getFirstValue(final @Nullable IdPAttribute attribute) {
+    @Nullable private IdPAttributeValue getFirstValue(@Nullable final IdPAttribute attribute) {
         if (attribute == null) {
             log.debug("No attribute found");
             return null;
         }
-        List<IdPAttributeValue> values = attribute.getValues();
+        final List<IdPAttributeValue> values = attribute.getValues();
         if (values == null|| values.size() < 1) {
             log.info("No attribute values associated with {}", attribute.getId());
             return null;

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


More information about the commits mailing list