[java-identity-provider COMMIT] in /trunk: idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/Authenticat...

[noreply at shibboleth.net]

Author: rdw
Date: Tue Oct 11 06:00:59 2016
New Revision: 8486

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=8486&view=rev
Log:
IDP-1047 Annotations for constructor arguments.

https://issues.shibboleth.net/jira/browse/IDP-1047



Modified:
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/AuthenticationResultPrincipal.java
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/DefaultPrincipalDeterminationStrategy.java
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/IdPAttributePrincipal.java
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/PasswordPrincipal.java
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/PrincipalEvalPredicateFactoryRegistry.java
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/UsernamePrincipal.java
    trunk/idp-cas-api/src/main/java/net/shibboleth/idp/cas/service/ServiceDefinition.java
    trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/ticket/impl/EncodingTicketService.java
    trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/ticket/impl/SimpleTicketService.java
    trunk/idp-cas-impl/src/main/java/net/shibboleth/idp/cas/ticket/impl/TicketIdentifierGenerationStrategy.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/context/navigate/RelyingPartyContextLookupById.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/context/navigate/RelyingPartyContextLookupByLabel.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/context/navigate/SpringExpressionContextLookupFunction.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/logic/DateAttributePredicate.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/logic/RelyingPartyIdPredicate.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/logic/ScriptedPredicate.java
    trunk/idp-profile-api/src/main/java/net/shibboleth/idp/profile/logic/SpringExpressionPredicate.java
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/security/impl/MetadataPKIXValidationInformationResolver.java

Modified: trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/AuthenticationResultPrincipal.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/AuthenticationResultPrincipal.java?rev=8486&r1=8485&r2=8486&view=diff
==============================================================================
--- trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/AuthenticationResultPrincipal.java	(original)
+++ trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/AuthenticationResultPrincipal.java	Tue Oct 11 06:00:59 2016
@@ -22,6 +22,7 @@
 import javax.annotation.Nonnull;
 
 import net.shibboleth.idp.authn.AuthenticationResult;
+import net.shibboleth.utilities.java.support.annotation.ParameterName;
 import net.shibboleth.utilities.java.support.annotation.constraint.NotEmpty;
 import net.shibboleth.utilities.java.support.logic.Constraint;
 
@@ -38,11 +39,12 @@
      * 
      * @param result the result to wrap
      */
-    public AuthenticationResultPrincipal(@Nonnull final AuthenticationResult result) {
+    public AuthenticationResultPrincipal(@Nonnull @ParameterName(name="result") final AuthenticationResult result) {
         authnResult = Constraint.isNotNull(result, "AuthenticationResult cannot be null");
     }
 
     /** {@inheritDoc} */
+    @Override
     @Nonnull @NotEmpty public String getName() {
         return authnResult.getAuthenticationFlowId();
     }
@@ -57,11 +59,13 @@
     }
 
     /** {@inheritDoc} */
+    @Override
     public int hashCode() {
         return authnResult.hashCode();
     }
 
     /** {@inheritDoc} */
+    @Override
     public boolean equals(final Object other) {
         if (other == null) {
             return false;
@@ -79,6 +83,7 @@
     }
 
     /** {@inheritDoc} */
+    @Override
     public String toString() {
         return MoreObjects.toStringHelper(this).add("authnResult", authnResult).toString();
     }

Modified: trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/DefaultPrincipalDeterminationStrategy.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/DefaultPrincipalDeterminationStrategy.java?rev=8486&r1=8485&r2=8486&view=diff
==============================================================================
--- trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/DefaultPrincipalDeterminationStrategy.java	(original)
+++ trunk/idp-authn-api/src/main/java/net/shibboleth/idp/authn/principal/DefaultPrincipalDeterminationStrategy.java	Tue Oct 11 06:00:59 2016
@@ -29,6 +29,7 @@
 import javax.annotation.Nullable;
 
 import net.shibboleth.idp.authn.context.AuthenticationContext;
+import net.shibboleth.utilities.java.support.annotation.ParameterName;

[... 476 lines stripped ...]