[java-identity-provider COMMIT] /trunk/idp-authn-api/src/main/java/net/shibboleth/idp/session/IdPSessionSubcontext.java

[noreply at shibboleth.net]

Author: rdw
Date: Wed Aug 24 16:28:48 2011
New Revision: 4031

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=4031&view=rev
Log:
Set the context into the owning container *after* all other initialization is done (other wise the act of adding it in will fail).

Modified:
    trunk/idp-authn-api/src/main/java/net/shibboleth/idp/session/IdPSessionSubcontext.java

Modified: trunk/idp-authn-api/src/main/java/net/shibboleth/idp/session/IdPSessionSubcontext.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-authn-api/src/main/java/net/shibboleth/idp/session/IdPSessionSubcontext.java?rev=4031&r1=4030&r2=4031&view=diff
==============================================================================
--- trunk/idp-authn-api/src/main/java/net/shibboleth/idp/session/IdPSessionSubcontext.java (original)
+++ trunk/idp-authn-api/src/main/java/net/shibboleth/idp/session/IdPSessionSubcontext.java Wed Aug 24 16:28:48 2011
@@ -42,12 +42,13 @@
     public IdPSessionSubcontext(final SubcontextContainer owningContext, final IdPSession idpSession){
         Assert.isNotNull(owningContext, "Owning context can not be null");
         owner = owningContext;
+        
+        Assert.isNotNull(idpSession, "IdP session can not be null");
+        session = idpSession;
+
         if (owner != null) {
             owner.addSubcontext(this);
         }
-        
-        Assert.isNotNull(idpSession, "IdP session can not be null");
-        session = idpSession;
     }
     
     /** {@inheritDoc} */