[java-identity-provider COMMIT] /trunk/idp-attribute-filter-impl/src/main/java/net/shibboleth/idp/attribute/filter/po...

noreply at shibboleth.net noreply at shibboleth.net
Sat Mar 21 10:23:07 EDT 2015


Author: rdw
Date: Sat Mar 21 10:23:07 2015
New Revision: 7440

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=7440&view=rev
Log:
CheckStyle

Modified:
    trunk/idp-attribute-filter-impl/src/main/java/net/shibboleth/idp/attribute/filter/policyrule/saml/impl/AbstractEntityAttributePolicyRule.java

Modified: trunk/idp-attribute-filter-impl/src/main/java/net/shibboleth/idp/attribute/filter/policyrule/saml/impl/AbstractEntityAttributePolicyRule.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-attribute-filter-impl/src/main/java/net/shibboleth/idp/attribute/filter/policyrule/saml/impl/AbstractEntityAttributePolicyRule.java?rev=7440&r1=7439&r2=7440&view=diff
==============================================================================
--- trunk/idp-attribute-filter-impl/src/main/java/net/shibboleth/idp/attribute/filter/policyrule/saml/impl/AbstractEntityAttributePolicyRule.java (original)
+++ trunk/idp-attribute-filter-impl/src/main/java/net/shibboleth/idp/attribute/filter/policyrule/saml/impl/AbstractEntityAttributePolicyRule.java Sat Mar 21 10:23:07 2015
@@ -97,15 +97,14 @@
     }
 
     /**
-     * Checks to see if the entity returned by {@link #getEntityMetadata(AttributeFilterContext)} contains the
-     * entity attribute specified by this matcher's configuration.
+     * Checks to see if the entity returned by {@link #getEntityMetadata(AttributeFilterContext)} contains the entity
+     * attribute specified by this matcher's configuration.
      * 
      * @param filterContext current request context
      * 
      * @return whether the entity has the configured attribute {@inheritDoc}
      */
-    @Override
-    public Tristate matches(@Nonnull AttributeFilterContext filterContext) {
+    @Override public Tristate matches(@Nonnull AttributeFilterContext filterContext) {
 
         Constraint.isNotNull(filterContext, "Context must be supplied");
         ComponentSupport.ifNotInitializedThrowUninitializedComponentException(this);
@@ -129,7 +128,7 @@
             return Tristate.FALSE;
         }
 
-        log.debug("{} Checking if entity attribute {} contains the required value.", getLogPrefix(), 
+        log.debug("{} Checking if entity attribute {} contains the required value.", getLogPrefix(),
                 getAttributeName());
         String valueString;
         for (XMLObject attributeValue : attributeValues) {
@@ -179,8 +178,8 @@
         }
 
         if (entityAttributesCollection.size() > 1) {
-            log.debug("{} Descriptor for {} contains more than one EntityAttributes extension, only using the first one",
-                    getLogPrefix(), entityDescriptor.getEntityID());
+            log.debug("{} Descriptor for {} contains more than one EntityAttributes extension,"
+                    + " only using the first one", getLogPrefix(), entityDescriptor.getEntityID());
         }
 
         List<Attribute> entityAttributes = ((EntityAttributes) entityAttributesCollection.get(0)).getAttributes();
@@ -207,11 +206,11 @@
                 new Object[] {getLogPrefix(), entityDescriptor.getEntityID(), getAttributeName(), getNameFormat()});
         return null;
     }
+
     // Checkstyle: CyclomaticComplexity ON
-    
+
     /** {@inheritDoc} */
-    @Override
-    protected void doInitialize() throws ComponentInitializationException {
+    @Override protected void doInitialize() throws ComponentInitializationException {
         super.doInitialize();
         if (attrName == null) {
             throw new ComponentInitializationException(getLogPrefix() + " Attribute name is null");



More information about the commits mailing list