[java-identity-provider COMMIT] in /trunk: idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/impl/Extrac...

noreply at shibboleth.net noreply at shibboleth.net
Fri Sep 2 06:27:18 EDT 2016


Author: rdw
Date: Fri Sep  2 06:27:18 2016
New Revision: 8367

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=8367&view=rev
Log:
Checkstyle : Mostly ReturnCount

Modified:
    trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/impl/ExtractConsent.java
    trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/storage/impl/AbstractConsentIndexedStorageAction.java
    trunk/idp-core/src/main/java/net/shibboleth/idp/spring/IdPPropertiesApplicationContextInitializer.java

Modified: trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/impl/ExtractConsent.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/impl/ExtractConsent.java?rev=8367&r1=8366&r2=8367&view=diff
==============================================================================
--- trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/impl/ExtractConsent.java	(original)
+++ trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/impl/ExtractConsent.java	Fri Sep  2 06:27:18 2016
@@ -56,6 +56,7 @@
     /** Class logger. */
     @Nonnull private final Logger log = LoggerFactory.getLogger(ExtractConsent.class);
 
+//CheckStyle: ReturnCount OFF
     /** {@inheritDoc} */
     @Override protected void doExecute(@Nonnull final ProfileRequestContext profileRequestContext,
             @Nonnull final ProfileInterceptorContext interceptorContext) {
@@ -91,5 +92,5 @@
 
         log.debug("{} Consent context '{}'", getLogPrefix(), consentContext);
     }
-
+  //CheckStyle: ReturnCount ON
 }

Modified: trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/storage/impl/AbstractConsentIndexedStorageAction.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/storage/impl/AbstractConsentIndexedStorageAction.java?rev=8367&r1=8366&r2=8367&view=diff
==============================================================================
--- trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/storage/impl/AbstractConsentIndexedStorageAction.java	(original)
+++ trunk/idp-consent-impl/src/main/java/net/shibboleth/idp/consent/flow/storage/impl/AbstractConsentIndexedStorageAction.java	Fri Sep  2 06:27:18 2016
@@ -245,6 +245,7 @@
         }
     }
 
+//CheckStyle: ReturnCount OFF
     /**
      * Storage records will be pruned based on the record maximums set on the flow descriptor,
      * and the storage service value size. Below a defined threshold, the basic maximum is applied, while at
@@ -299,7 +300,7 @@
 
             log.debug("{} Deleting storage record with context '{}' and key '{}'", getLogPrefix(), getStorageContext(),
                     keyToDelete);
-            boolean success = getStorageService().delete(getStorageContext(), keyToDelete);
+            final boolean success = getStorageService().delete(getStorageContext(), keyToDelete);
 
             if (success) {
                 numberOfKeys--;
@@ -309,6 +310,7 @@
             removeKeyFromStorageIndex(keyToDelete);
         }
     }
+  //CheckStyle: ReturnCount ON
 
     /**
      * Store a profile interceptor result.

Modified: trunk/idp-core/src/main/java/net/shibboleth/idp/spring/IdPPropertiesApplicationContextInitializer.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-core/src/main/java/net/shibboleth/idp/spring/IdPPropertiesApplicationContextInitializer.java?rev=8367&r1=8366&r2=8367&view=diff
==============================================================================
--- trunk/idp-core/src/main/java/net/shibboleth/idp/spring/IdPPropertiesApplicationContextInitializer.java	(original)
+++ trunk/idp-core/src/main/java/net/shibboleth/idp/spring/IdPPropertiesApplicationContextInitializer.java	Fri Sep  2 06:27:18 2016
@@ -78,6 +78,7 @@
     /** Class logger. */
     @Nonnull private final Logger log = LoggerFactory.getLogger(IdPPropertiesApplicationContextInitializer.class);
 
+//CheckStyle: ReturnCount OFF
     /** {@inheritDoc} */
     @Override public void initialize(@Nonnull final ConfigurableApplicationContext applicationContext) {
         log.debug("Initializing application context '{}'", applicationContext);
@@ -137,6 +138,7 @@
             log.warn("Unable to find '{}' at well known locations '{}'", getSearchTarget(), getSearchLocations());
         }
     }
+//CheckStyle: ReturnCount ON
 
     /**
      * Get the target resource to be searched for. Defaults to {@link #IDP_PROPERTIES}.



More information about the commits mailing list