[jboss-cvs] Picketlink SVN: r673 - in federation/trunk: picketlink-fed-api/src/main/java/org/picketlink/identity/federation/api/openid and 4 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Thu Jan 27 11:42:24 EST 2011


Author: anil.saldhana at jboss.com
Date: 2011-01-27 11:42:24 -0500 (Thu, 27 Jan 2011)
New Revision: 673

Modified:
   federation/trunk/picketlink-bindings-jboss/src/main/java/org/picketlink/identity/federation/bindings/jboss/subject/PicketLinkGroup.java
   federation/trunk/picketlink-fed-api/src/main/java/org/picketlink/identity/federation/api/openid/OpenIDManager.java
   federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/saml/v2/common/SAMLProtocolContext.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AdviceType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AssertionType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeStatementType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AudienceRestrictionType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthnContextType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthzDecisionStatementType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ConditionsType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/EvidenceType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ProxyRestrictionType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectConfirmationDataType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeAuthorityDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeConsumingServiceType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AuthnAuthorityDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ContactType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EndpointType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntitiesDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntityDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ExtensionsType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/IDPSSODescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/KeyDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/OrganizationType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/PDPDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/RoleDescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SPSSODescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SSODescriptorType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/TypeWithOtherAttributes.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AssertionIDRequestType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AttributeQueryType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AuthzDecisionQueryType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ExtensionsType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/IDPListType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/LogoutRequestType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/RequestedAuthnContextType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ResponseType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ScopingType.java
   federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/StatusDetailType.java
Log:
javadoc, add/remove combo for collection elements

Modified: federation/trunk/picketlink-bindings-jboss/src/main/java/org/picketlink/identity/federation/bindings/jboss/subject/PicketLinkGroup.java
===================================================================
--- federation/trunk/picketlink-bindings-jboss/src/main/java/org/picketlink/identity/federation/bindings/jboss/subject/PicketLinkGroup.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-bindings-jboss/src/main/java/org/picketlink/identity/federation/bindings/jboss/subject/PicketLinkGroup.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -43,23 +43,38 @@
    {
      super(name);   
    }
-   
-   public boolean addMember(Principal user)
+   /**
+    * Add a role principal to group
+    * @see java.security.acl.Group#addMember(java.security.Principal)
+    */
+   public boolean addMember(Principal role )
    { 
-      return roles.add(user);
+      return roles.add( role );
    }
 
-   public boolean isMember(Principal member)
+   /**
+    * Check if the role is a member of the group
+    * @see java.security.acl.Group#isMember(java.security.Principal)
+    */
+   public boolean isMember(Principal role )
    { 
-      return roles.contains(member);
+      return roles.contains( role );
    }
 
+   /**
+    * Get the group members
+    * @see java.security.acl.Group#members()
+    */
    public Enumeration<? extends Principal> members()
    {
       Set<Principal> readOnly = Collections.unmodifiableSet(roles);
       return Collections.enumeration(readOnly);
    }
-
+   
+   /**
+    * Remove role from groups
+    * @see java.security.acl.Group#removeMember(java.security.Principal)
+    */
    public boolean removeMember(Principal user)
    {
       return roles.remove(user);

Modified: federation/trunk/picketlink-fed-api/src/main/java/org/picketlink/identity/federation/api/openid/OpenIDManager.java
===================================================================
--- federation/trunk/picketlink-fed-api/src/main/java/org/picketlink/identity/federation/api/openid/OpenIDManager.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-api/src/main/java/org/picketlink/identity/federation/api/openid/OpenIDManager.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -415,6 +415,11 @@
          this.providers = providers;
       }
       
+      void addProvider( DiscoveryInformation provider )
+      {
+         this.providers.add( provider );
+      }
+      
       List<DiscoveryInformation> get()
       {
          return Collections.unmodifiableList(providers);
@@ -424,6 +429,5 @@
       {
          return this.providers != null ? providers.size() : 0;
       }
-
    }
 }
\ No newline at end of file

Modified: federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/saml/v2/common/SAMLProtocolContext.java
===================================================================
--- federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/saml/v2/common/SAMLProtocolContext.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-core/src/main/java/org/picketlink/identity/federation/core/saml/v2/common/SAMLProtocolContext.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -60,46 +60,82 @@
    
    protected AssertionType issuedAssertion;
    
+   /**
+    * Get the Issuer ID
+    * @return instance of {@link NameIDType}
+    */
    public NameIDType getIssuerID()
    {
       return issuerID;
    }
 
+   /**
+    * Set the Issuer ID
+    * @param issuerID {@link NameIDType}
+    */
    public void setIssuerID( NameIDType issuerID)
    {
       this.issuerID = issuerID;
    }
 
+   /**
+    * Get the subject
+    * @return {@link SubjectType}
+    */
    public SubjectType getSubjectType()
    {
       return subjectType;
    }
 
+   /**
+    * Set the subject
+    * @param subjectType {@link SubjectType}
+    */
    public void setSubjectType(SubjectType subjectType)
    {
       this.subjectType = subjectType;
    }
 
+   /**
+    * Get the conditions
+    * @return {@link ConditionsType}
+    */
    public ConditionsType getConditions()
    {
       return conditions;
    }
 
+   /**
+    * Set the conditions
+    * @param conditions {@link ConditionsType}
+    */
    public void setConditions(ConditionsType conditions)
    {
       this.conditions = conditions;
    }
 
+   /**
+    * Get the statements as a read-only list
+    * @return {@link StatementAbstractType}
+    */
    public List<StatementAbstractType> getStatements()
    {
       return Collections.unmodifiableList( statements );
    }
 
+   /**
+    * Add a list of statements 
+    * @param statements {@link List}
+    */
    public void setStatements(List<StatementAbstractType> statements)
    {
       this.statements = statements;
    } 
 
+   /**
+    * Get the previously issued assertion by the STS
+    * @return {@link AssertionType}
+    */
    public AssertionType getIssuedAssertion()
    {
       SecurityManager sm = System.getSecurityManager();
@@ -109,6 +145,10 @@
       return issuedAssertion;
    }
 
+   /**
+    * Set the assertion issued by the STS
+    * @param issuedAssertion {@link AssertionType}
+    */
    public void setIssuedAssertion(AssertionType issuedAssertion)
    {
       SecurityManager sm = System.getSecurityManager();

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AdviceType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AdviceType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AdviceType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -54,13 +54,18 @@
 {  
    protected List<Object> advices = new ArrayList<Object>();
 
+   /**
+    * Add an advice
+    * @param obj
+    */
    public void addAdvice( Object obj )
    {
       advices.add( obj );
    }
 
    /**
-    * Gets the advices. 
+    * Gets the advices. (Read only list)
+    * @return {@link List} read only 
     */
    public List<Object> getAdvices() 
    { 

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AssertionType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AssertionType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AssertionType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -75,98 +75,212 @@
 
    private Set<StatementAbstractType> statements = new LinkedHashSet<StatementAbstractType>();
 
+   /**
+    * Create an assertion
+    * @param iD ID of the assertion (Required)
+    * @param issueInstant {@link XMLGregorianCalendar} issue instant (required)
+    * @param version
+    */
    public AssertionType(String iD, XMLGregorianCalendar issueInstant, String version)
    { 
+      if( iD == null )
+         throw new IllegalArgumentException( "iD is null" );
+      if( issueInstant == null )
+         throw new IllegalArgumentException( "issueInstant is null" );
+      if( version == null )
+         throw new IllegalArgumentException( "version is null" );
+      
       this.ID = iD;
       this.issueInstant = issueInstant;
       this.version = version;
    }
 
+   /**
+    * Get the assertion id
+    * @return {@link String}
+    */
    public String getID()
    {
       return ID;
    }
 
+   /**
+    * Get the subject
+    * @return {@link SubjectType}
+    */
    public SubjectType getSubject()
    {
+      checkSTSPermission();
       return subject;
    }
 
+   /**
+    * Set the subject
+    * @param subject
+    */
    public void setSubject(SubjectType subject)
    {
+      checkSTSPermission();
       this.subject = subject;
    }
 
+   /**
+    * Get the Issue Instant
+    * @return {@link XMLGregorianCalendar}
+    */
    public XMLGregorianCalendar getIssueInstant()
    {
       return issueInstant;
    }
 
+   /**
+    * Get the version of SAML
+    * @return {@link String}
+    */
    public String getVersion()
    {
       return version;
    }   
 
+   /**
+    * Get the advice
+    * @return {@link AdviceType}
+    */
    public AdviceType getAdvice()
    {
       return advice;
    }
 
+   /**
+    * Set the advice
+    * @param advice {@link advice}
+    */
    public void setAdvice(AdviceType advice)
    {
+      checkSTSPermission();
+      
       this.advice = advice;
    }
 
+   /**
+    * Get the conditions
+    * @return {@link ConditionsType}
+    */
    public ConditionsType getConditions()
    {
+      checkSTSPermission();
+      
       return conditions;
    }
 
+   /**
+    * Set the conditions 
+    * @param conditions {@link ConditionsType}
+    */
    public void setConditions(ConditionsType conditions)
    {
+      checkSTSPermission();
+      
       this.conditions = conditions;
    }
 
+   /**
+    * Get the issuer
+    * @return {@link NameIDType}
+    */
    public NameIDType getIssuer()
    {
       return issuer;
    }
 
+   /**
+    * Set the issuer
+    * @param issuer {@link NameIDType}
+    */
    public void setIssuer(NameIDType issuer)
    {
+      checkSTSPermission();
+      
       this.issuer = issuer;
    } 
 
+   /**
+    * Add a statement
+    * @param statement {@link StatementAbstractType}
+    */
    public void addStatement( StatementAbstractType statement )
    {
+      checkSTSPermission();
+      
       this.statements.add( statement );
    }
+   /**
+    * Add a collection of statements
+    * @param statement {@link Collection}
+    */
    public void addStatements( Collection<StatementAbstractType> statement )
    {
+      checkSTSPermission();
+      
       this.statements.addAll( statement );
    }
 
+   /**
+    * Add a set of statements
+    * @param statement {@link Collection}
+    */
+   public void addStatements( Set<StatementAbstractType> statement )
+   {
+      checkSTSPermission();
+      
+      this.statements.addAll( statement );
+   }
+   
+   /**
+    * Get a read only set of statements
+    * @return {@link Set}
+    */
    public Set<StatementAbstractType> getStatements()
    {
+      checkSTSPermission();
+      
       return Collections.unmodifiableSet( statements );
    }
 
+   /**
+    * Get the signature as a DOM element
+    * @return {@link Element}
+    */
    public Element getSignature()
    {
       return signature;
    }
 
+   /**
+    * Set the signature DOM element
+    * @param signature
+    */
    public void setSignature(Element signature)
    {
       this.signature = signature;
    } 
    
+   /**
+    * Update the issue instant
+    * @param xg
+    */
    public void updateIssueInstant( XMLGregorianCalendar xg )
    {
+      checkSTSPermission();
+      
+      this.issueInstant = xg; 
+   }
+   
+
+   protected void checkSTSPermission()
+   {
       SecurityManager sm = System.getSecurityManager();
       if( sm != null )
          sm.checkPermission( new RuntimePermission( "org.picketlink.sts") );
-      
-      this.issueInstant = xg; 
    }
 }
\ No newline at end of file

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeStatementType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeStatementType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeStatementType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -54,13 +54,27 @@
    private static final long serialVersionUID = 1L;
    protected List<ASTChoiceType> attributes = new ArrayList<ASTChoiceType>();
 
+   /**
+    * Add an attribute
+    * @param attribute
+    */
    public void addAttribute( ASTChoiceType attribute )
    {
       attributes.add( attribute );
    }
+   
+   /**
+    * Remove an attribute
+    * @param attribute
+    */
+   public void removeAttribute( ASTChoiceType attribute )
+   {
+      attributes.remove( attribute );
+   }
 
    /**
-    * Gets the attributes. 
+    * Gets the attributes.
+    * @return a read only {@link List} 
     */
    public List<ASTChoiceType> getAttributes() 
    {

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AttributeType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -63,10 +63,23 @@
       this.name = name;
    }
 
+   /**
+    * Add an attribute value to the attribute
+    * @param value {@link Object}
+    */
    public void addAttributeValue( Object value )
    {
       attributeValue.add(value);
    }
+   
+   /**
+    * Remove an attribute value to the attribute
+    * @param value {@link Object}
+    */
+   public void removeAttributeValue( Object value )
+   {
+      attributeValue.remove(value);
+   }
 
    /**
     * Gets the value of the attributeValue property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AudienceRestrictionType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AudienceRestrictionType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AudienceRestrictionType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -50,10 +50,23 @@
 { 
    protected List<URI> audience = new ArrayList<URI>();
 
+   /**
+    * Add an audience
+    * @param audienceval
+    */
    public void addAudience( URI audienceval )
    {
       audience.add( audienceval ); 
    }
+   
+   /**
+    * Remove an audience
+    * @param audienceval
+    */
+   public void removeAudience( URI audienceval )
+   {
+      audience.remove( audienceval ); 
+   }
 
    /**
     * Gets the value of the audience property.  

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthnContextType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthnContextType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthnContextType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -70,46 +70,95 @@
 
    private Set<URIType> URITypes = new HashSet<URIType>();
 
-
+   /**
+    * Add an authenticating authority
+    * @param aa {@link URI}
+    */
    public void addAuthenticatingAuthority( URI aa )
    {
       authenticatingAuthority.add( aa );
    }
 
+   /**
+    * Add Authenticating Authority
+    * @param aas an array of {@link URI}
+    */
    public void addAuthenticatingAuthority( URI[] aas )
    {
       authenticatingAuthority.addAll( Arrays.asList( aas ) );
    }
 
+   /**
+    * Remove an authenticating authority
+    * @param aa
+    */
+   public void removeAuthenticatingAuthority( URI aa )
+   {
+      authenticatingAuthority.remove( aa );
+   }
+   
+   /**
+    * Get a read only set of authenticating authority
+    * @return
+    */
    public Set<URI> getAuthenticatingAuthority()
    {
       return Collections.unmodifiableSet( authenticatingAuthority );
    }   
 
+   /**
+    * Get the sequence
+    * @return
+    */
    public AuthnContextTypeSequence getSequence()
    {
       return sequence;
    }
 
+   /**
+    * Set the authn context sequence
+    * @param sequence
+    */
    public void setSequence(AuthnContextTypeSequence sequence)
    {
       this.sequence = sequence;
    } 
 
+   /**
+    * Add an URI type
+    * @param aa
+    */
    public void addURIType( URIType aa )
    {
       URITypes.add( aa );
    }
 
+   /**
+    * Add an array of URI Type
+    * @param aas
+    */
    public void addURIType( URIType[] aas )
    {
       URITypes.addAll( Arrays.asList( aas ) );
    }
 
+   /**
+    * Get a read only set of URI type
+    * @return
+    */
    public Set<URIType> getURIType()
    {
       return Collections.unmodifiableSet( URITypes );
-   } 
+   }
+   
+   /**
+    * Add an URI type
+    * @param aa
+    */
+   public void removeURIType( URIType aa )
+   {
+      URITypes.remove( aa );
+   }
 
    /**
     <sequence>
@@ -136,6 +185,11 @@
       {
          URITypes.add( aa );
       }
+      
+      public void removeURIType( URIType aa )
+      {
+         URITypes.remove( aa );
+      }
 
       public void addURIType( URIType[] aas )
       {

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthzDecisionStatementType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthzDecisionStatementType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/AuthzDecisionStatementType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -58,17 +58,31 @@
    protected DecisionType decision;
 
    /**
-    * Gets the value of the action property. 
+    * Get the list of actions (read-only list)
+    * @return {@link List} read only 
     */
    public List<ActionType> getAction() 
    { 
       return Collections.unmodifiableList( this.action );
    }
 
+   /**
+    * Add an action
+    * @param actionType
+    */
    public void addAction( ActionType actionType )
    {
       action.add(actionType); 
    }
+   
+   /**
+    * Remove an action
+    * @param actionType
+    */
+   public void removeAction( ActionType actionType )
+   {
+      action.remove(actionType); 
+   }
 
    /**
     * Gets the value of the evidence property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ConditionsType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ConditionsType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ConditionsType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -60,10 +60,23 @@
    protected XMLGregorianCalendar notBefore; 
    protected XMLGregorianCalendar notOnOrAfter;
 
+   /**
+    * Add a condition
+    * @param condition
+    */
    public void addCondition( ConditionAbstractType condition )
    {
       this.conditions.add( condition );
    }
+   
+   /**
+    * Remove a condition
+    * @param condition
+    */
+   public void removeCondition( ConditionAbstractType condition )
+   {
+      this.conditions.remove( condition );
+   }
 
    /**
     * Gets an read only conditions list. 

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/EvidenceType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/EvidenceType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/EvidenceType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -54,11 +54,28 @@
 {
    protected List<ChoiceType> evidences = new ArrayList<ChoiceType>();
 
+   /**
+    * Add an evidence
+    * @param evidence
+    */
    public void addEvidence( ChoiceType evidence )
    {
       evidences.add( evidence );
    }
+   
+   /**
+    * Remove an evidence
+    * @param evidence
+    */
+   public void removeEvidence( ChoiceType evidence )
+   {
+      evidences.remove( evidence );
+   }
 
+   /**
+    * Get the list of evidences as a read only list
+    * @return
+    */
    public List<ChoiceType> evidences()
    {
       return Collections.unmodifiableList( evidences );
@@ -110,6 +127,5 @@
       {
          return encryptedAssertion;
       } 
-
    }
 }
\ No newline at end of file

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ProxyRestrictionType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ProxyRestrictionType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/ProxyRestrictionType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -55,6 +55,10 @@
 
    protected BigInteger count;
 
+   /**
+    * Add an audience
+    * @param a
+    */
    public void addAudience( URI a )
    {
       this.audience.add( a );
@@ -68,6 +72,15 @@
    { 
       return Collections.unmodifiableList( audience );
    }
+   
+   /**
+    * Remove an audience
+    * @param a
+    */
+   public void removeAudience( URI a )
+   {
+      this.audience.remove( a );
+   }
 
    /**
     * Gets the value of the count property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectConfirmationDataType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectConfirmationDataType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectConfirmationDataType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -209,8 +209,22 @@
       return Collections.unmodifiableMap( otherAttributes );
    }
 
+   /**
+    * Add an other attribute
+    * @param qname
+    * @param str
+    */
    public void addOtherAttribute( QName qname, String str )
    {
       otherAttributes.put(qname, str );
-   } 
-}
+   }
+   
+   /**
+    * Remove an other attribute
+    * @param qname {@link QName} of the attribute to be removed
+    */
+   public void removeOtherAttribute( QName qname )
+   {
+      otherAttributes.remove( qname );
+   }
+}
\ No newline at end of file

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/assertion/SubjectType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -63,30 +63,58 @@
 
    protected STSubType subType; 
 
+   /**
+    * Get the {@link STSubType}
+    * @return
+    */
    public STSubType getSubType()
    {
       return subType;
    }
 
+   /**
+    * Set the {@link STSubType}
+    * @param subType
+    */
    public void setSubType(STSubType subType)
    {
       this.subType = subType;
    }
 
+   /**
+    * Get the size of subject confirmations
+    * @return
+    */
    public int getCount()
    {
       return subjectConfirmation.size();
    }
 
+   /**
+    * Get a list of subject confirmations
+    * @return {@link} read only list of subject confirmation
+    */
    public List<SubjectConfirmationType> getConfirmation()
    {
       return Collections.unmodifiableList( subjectConfirmation );
    }
 
+   /**
+    * Add a subject confirmation
+    * @param con
+    */
    public void addConfirmation( SubjectConfirmationType con )
    {
       subjectConfirmation.add( con );
    } 
+   /**
+    * Remove a subject confirmation
+    * @param con
+    */
+   public void removeConfirmation( SubjectConfirmationType con )
+   {
+      subjectConfirmation.remove( con );
+   } 
 
    public static class STSubType
    {

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeAuthorityDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeAuthorityDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeAuthorityDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -45,32 +45,97 @@
       super(protocolSupport);
    }
 
+   /**
+    * Add an attribute service
+    * @param endpoint
+    */
    public void addAttributeService( EndpointType endpoint )
    {
       this.attributeService.add(endpoint);
    }
 
+   /**
+    * Add an assertion id request service
+    * @param endpoint
+    */
    public void addAssertionIDRequestService( EndpointType endpoint )
    {
       this.assertionIDRequestService.add(endpoint);
    }
 
+   /**
+    * Add a name id
+    * @param str
+    */
    public void addNameIDFormat( String str )
    {
       this.nameIDFormat.add(str);
    }
 
+   /**
+    * Add an attribute profile
+    * @param str
+    */
    public void addAttributeProfile( String str )
    {
       this.attributeProfile.add(str);
    }
 
+   /**
+    * Add an attribute
+    * @param attribute
+    */
    public void addAttribute( AttributeType attribute )
    {
       this.attribute.add(attribute);
    }
+   
+   /**
+    * Remove an attribute service
+    * @param endpoint
+    */
+   public void removeAttributeService( EndpointType endpoint )
+   {
+      this.attributeService.remove( endpoint);
+   }
 
    /**
+    * Remove assertion id request service
+    * @param endpoint
+    */
+   public void removeAssertionIDRequestService( EndpointType endpoint )
+   {
+      this.assertionIDRequestService.remove(endpoint);
+   }
+
+   /**
+    * Remove Name ID
+    * @param str
+    */
+   public void removeNameIDFormat( String str )
+   {
+      this.nameIDFormat.remove(str);
+   }
+
+   /**
+    * Remove attribute profile
+    * @param str
+    */
+   public void removeAttributeProfile( String str )
+   {
+      this.attributeProfile.remove(str);
+   }
+
+   /**
+    * Remove attribute
+    * @param attribute
+    */
+   public void removeAttribute( AttributeType attribute )
+   {
+      this.attribute.remove(attribute);
+   }
+
+   /**
     * Gets the value of the attributeService property. 
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeConsumingServiceType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeConsumingServiceType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AttributeConsumingServiceType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -48,6 +48,83 @@
    }
 
    /**
+    * Add serviceName  
+    * <p>
+    * Objects of the following type(s) are allowed in the list
+    * {@link LocalizedNameType }
+    * 
+    * 
+    */
+   public void addServiceName( LocalizedNameType service ) 
+   {
+      this.serviceName.add(service) ;
+   }
+
+   /**
+    * Add serviceDescription.
+    * <p>
+    * Objects of the following type(s) are allowed in the list
+    * {@link LocalizedNameType }
+    * 
+    * 
+    */
+   public void addServiceDescription( LocalizedNameType desc ) 
+   {
+      this.serviceDescription.add(desc);
+   }
+
+   /**
+    * Add requestedAttribute 
+    * <p>
+    * Objects of the following type(s) are allowed in the list
+    * {@link RequestedAttributeType }
+    * 
+    */
+   public void addRequestedAttribute( RequestedAttributeType req ) 
+   {
+      this.requestedAttribute.add(req);
+   }
+   
+   /**
+    * remove serviceName  
+    * <p>
+    * Objects of the following type(s) are allowed in the list
+    * {@link LocalizedNameType }
+    * 
+    * 
+    */
+   public void removeServiceName( LocalizedNameType service ) 
+   {
+      this.serviceName.remove(service) ;
+   }
+
+   /**
+    * remove serviceDescription.
+    * <p>
+    * Objects of the following type(s) are allowed in the list
+    * {@link LocalizedNameType }
+    * 
+    * 
+    */
+   public void removeServiceDescription( LocalizedNameType desc ) 
+   {
+      this.serviceDescription.remove(desc);
+   }
+
+   /**
+    * remove requestedAttribute 
+    * <p>
+    * Objects of the following type(s) are allowed in the list
+    * {@link RequestedAttributeType }
+    * 
+    */
+   public void removeRequestedAttribute( RequestedAttributeType req ) 
+   {
+      this.requestedAttribute.remove(req);
+   }
+   
+    
+   /**
     * Gets the value of the serviceName property.
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AuthnAuthorityDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AuthnAuthorityDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/AuthnAuthorityDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -36,22 +36,61 @@
       super(protocolSupport); 
    }
 
+   /**
+    * Add authn query service
+    * @param endpoint
+    */
    public void addAuthnQueryService( EndpointType endpoint )
    {
       this.authnQueryService.add(endpoint);
    }
 
+   /**
+    * Add assertion id request service
+    * @param endpoint
+    */
    public void addAssertionIDRequestService( EndpointType endpoint )
    {
       this.assertionIDRequestService.add(endpoint);
    }
 
+   /**
+    * Add name id format
+    * @param str
+    */
    public void addNameIDFormat( String str )
    {
       this.nameIDFormat.add(str);
    }
+   
+   /**
+    * Remove authn query service
+    * @param endpoint
+    */
+   public void removeAuthnQueryService( EndpointType endpoint )
+   {
+      this.authnQueryService.remove(endpoint);
+   }
 
    /**
+    * remove assertion id request service
+    * @param endpoint
+    */
+   public void removeAssertionIDRequestService( EndpointType endpoint )
+   {
+      this.assertionIDRequestService.remove(endpoint);
+   }
+
+   /**
+    * remove name id format
+    * @param str
+    */
+   public void removeNameIDFormat( String str )
+   {
+      this.nameIDFormat.remove(str);
+   }
+
+   /**
     * Gets the value of the authnQueryService property.
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ContactType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ContactType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ContactType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -143,12 +143,37 @@
       this.surName = value;
    }
 
-
+   /**
+    * Add an email address
+    * @param email
+    */
    public void addEmailAddress( String email )
    {
       this.emailAddress.add(email);
    }
 
+   /**
+    * remove a telephone
+    * @param tel
+    */
+   public void removeTelephone( String tel )
+   {
+      this.telephoneNumber.remove(tel);
+   }
+   
+   /**
+    * remove an email address
+    * @param email
+    */
+   public void removeEmailAddress( String email )
+   {
+      this.emailAddress.remove(email);
+   }
+
+   /**
+    * Add a telephone
+    * @param tel
+    */
    public void addTelephone( String tel )
    {
       this.telephoneNumber.add(tel);

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EndpointType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EndpointType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EndpointType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -45,6 +45,24 @@
       this.binding = binding;
       this.location = location;
    }
+   
+   /**
+    * Add an object
+    * @param obj
+    */
+   public void addObject( Object obj )
+   {
+      this.any.add(obj);
+   }
+   
+   /**
+    * remove an object
+    * @param obj
+    */
+   public void removeObject( Object obj )
+   {
+      this.any.remove(obj);
+   }
 
    /**
     * Gets the value of the any property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntitiesDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntitiesDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntitiesDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -101,11 +101,25 @@
       this.extensions = value;
    }
 
+   /**
+    * Add an entity descriptor
+    * @param obj
+    */
    public void addEntityDescriptor( Object obj )
    {
       this.entityDescriptor.add( obj );
    }
+   
    /**
+    * Remove an entity descriptor
+    * @param obj
+    */
+   public void removeEntityDescriptor( Object obj )
+   {
+      this.entityDescriptor.remove( obj );
+   }
+   
+   /**
     * Gets the value of the entityDescriptorOrEntitiesDescriptor property.
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntityDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntityDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/EntityDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -215,15 +215,32 @@
       this.extensions = value;
    }
 
+   /**
+    * Get a read only list of choice types
+    * @return
+    */
    public List<EDTChoiceType> getChoiceType()
    {
       return Collections.unmodifiableList( choiceType );
    }
 
+   /**
+    * Add a choice type
+    * @param choiceType
+    */
    public void addChoiceType(EDTChoiceType choiceType)
    {
       this.choiceType.add( choiceType );
    }
+   
+   /**
+    * Remove a choice type
+    * @param choiceType
+    */
+   public void removeChoiceType(EDTChoiceType choiceType)
+   {
+      this.choiceType.remove( choiceType );
+   }
 
    /**
     * Gets the value of the organization property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ExtensionsType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ExtensionsType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/ExtensionsType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -30,10 +30,23 @@
 {
    protected List<Object> any = new ArrayList<Object>();
 
-   public void addAny( Object obj )
+   /**
+    * Add an object
+    * @param obj
+    */
+   public void addObject( Object obj )
    {
       this.any.add(obj);
    }
+   
+   /**
+    * Remove an object
+    * @param obj
+    */
+   public void removeObject( Object obj )
+   {
+      this.any.remove(obj);
+   }
 
    /**
     * Gets the value of the any property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/IDPSSODescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/IDPSSODescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/IDPSSODescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -51,32 +51,98 @@
       super(protocolSupport); 
    }
 
+   /**
+    * Add a SSO service
+    * @param endpt
+    */
    public void addSingleSignOnService( EndpointType endpt)
    {
       this.singleSignOnService.add(endpt);
    }
 
+   /**
+    * Add name id mapping service
+    * @param endpt
+    */
    public void addNameIDMappingService( EndpointType endpt )
    {
       this.nameIDMappingService.add(endpt);
    }
 
+   /**
+    * Add assertion id request service
+    * @param endpt
+    */
    public void addAssertionIDRequestService( EndpointType endpt )
    {
       this.assertionIDRequestService.add(endpt);
    }
 
+   /**
+    * Add attribute profile
+    * @param str
+    */
    public void addAttributeProfile( String str )
    {
       this.attributeProfile.add( str );
    }  
 
+   /**
+    * Add attribute
+    * @param att
+    */
    public void addAttribute( AttributeType att )
    {
       this.attribute.add(att);
    } 
+   
+   
+   /**
+    * Remove a SSO service
+    * @param endpt
+    */
+   public void removeSingleSignOnService( EndpointType endpt)
+   {
+      this.singleSignOnService.remove(endpt);
+   }
 
    /**
+    * remove name id mapping service
+    * @param endpt
+    */
+   public void removeNameIDMappingService( EndpointType endpt )
+   {
+      this.nameIDMappingService.remove(endpt);
+   }
+
+   /**
+    * remove assertion id request service
+    * @param endpt
+    */
+   public void removeAssertionIDRequestService( EndpointType endpt )
+   {
+      this.assertionIDRequestService.remove(endpt);
+   }
+
+   /**
+    * Add attribute profile
+    * @param str
+    */
+   public void removeAttributeProfile( String str )
+   {
+      this.attributeProfile.remove( str );
+   }  
+
+   /**
+    * Add attribute
+    * @param att
+    */
+   public void removeAttribute( AttributeType att )
+   {
+      this.attribute.remove(att);
+   } 
+
+   /**
     * Gets the value of the singleSignOnService property.
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/KeyDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/KeyDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/KeyDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -62,10 +62,23 @@
       this.keyInfo = value;
    }
 
+   /**
+    * Add encryption method type
+    * @param e
+    */
    public void addEncryptionMethod( EncryptionMethodType e )
    {
       this.encryptionMethod.add( e );
    }
+   
+   /**
+    * Remove encryption method type
+    * @param e
+    */
+   public void removeEncryptionMethod( EncryptionMethodType e )
+   {
+      this.encryptionMethod.remove( e );
+   }
 
    /**
     * Gets the value of the encryptionMethod property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/OrganizationType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/OrganizationType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/OrganizationType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -38,22 +38,61 @@
 
    protected List<LocalizedURIType> organizationURL = new ArrayList<LocalizedURIType>();
 
+   /**
+    * Add an organization name
+    * @param name
+    */
    public void addOrganizationName( LocalizedNameType name )
    {
       this.organizationName.add(name);
    }
 
+   /**
+    * Add organization display name
+    * @param name
+    */
    public void addOrganizationDisplayName( LocalizedNameType name )
    {
       this.organizationDisplayName.add(name);
    }
 
+   /**
+    * Add organization url
+    * @param uri
+    */
    public void addOrganizationURL( LocalizedURIType uri )
    {
       this.organizationURL.add(uri);
    }
+   
+   /**
+    * remove an organization name
+    * @param name
+    */
+   public void removeOrganizationName( LocalizedNameType name )
+   {
+      this.organizationName.remove(name);
+   }
 
    /**
+    * remove organization display name
+    * @param name
+    */
+   public void removeOrganizationDisplayName( LocalizedNameType name )
+   {
+      this.organizationDisplayName.remove(name);
+   }
+
+   /**
+    * remove organization url
+    * @param uri
+    */
+   public void removeOrganizationURL( LocalizedURIType uri )
+   {
+      this.organizationURL.remove(uri);
+   }
+
+   /**
     * Gets the value of the extensions property.
     * 
     * @return

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/PDPDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/PDPDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/PDPDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -40,22 +40,61 @@
       super(protocolSupport); 
    }
 
+   /**
+    * Add authorization service
+    * @param endpt
+    */
    public void addAuthZService( EndpointType endpt )
    {
       this.authzService.add(endpt);
    }
 
+   /**
+    * Add assertion id request service
+    * @param endpt
+    */
    public void addAssertionIDRequestService( EndpointType endpt )
    {
       this.assertionIDRequestService.add(endpt);
    }
 
+   /**
+    * Add Name ID Format
+    * @param str
+    */
    public void addNameIDFormat( String str )
    {
       this.nameIDFormat.add(str);
    }
+   
+   /**
+    * remove authorization service
+    * @param endpt
+    */
+   public void removeAuthZService( EndpointType endpt )
+   {
+      this.authzService.remove(endpt);
+   }
 
    /**
+    * remove assertion id request service
+    * @param endpt
+    */
+   public void removeAssertionIDRequestService( EndpointType endpt )
+   {
+      this.assertionIDRequestService.remove(endpt);
+   }
+
+   /**
+    * remove Name ID Format
+    * @param str
+    */
+   public void removeNameIDFormat( String str )
+   {
+      this.nameIDFormat.remove(str);
+   }
+
+   /**
     * Gets the value of the authzService property.
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/RoleDescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/RoleDescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/RoleDescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -56,15 +56,41 @@
       protocolSupportEnumeration.addAll( protocolSupport );
    }
    
+   /**
+    * Add key descriptor
+    * @param keyD
+    */
    public void addKeyDescriptor( KeyDescriptorType keyD )
    {
       this.keyDescriptor.add(keyD);
    }
    
+   /**
+    * Add contact 
+    * @param contact
+    */
    public void addContactPerson( ContactType contact )
    {
       this.contactPerson.add(contact);
    }
+   
+   /**
+    * remove key descriptor
+    * @param keyD
+    */
+   public void removeKeyDescriptor( KeyDescriptorType keyD )
+   {
+      this.keyDescriptor.remove(keyD);
+   }
+   
+   /**
+    * remove contact 
+    * @param contact
+    */
+   public void removeContactPerson( ContactType contact )
+   {
+      this.contactPerson.remove(contact);
+   }
      
    /**
     * Gets the value of the signature property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SPSSODescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SPSSODescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SPSSODescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -39,6 +39,42 @@
    {
       super(protocolSupport); 
    }
+   
+   /**
+    * Add an Assertion Consumer Service
+    * @param assertionConsumer an endpoint of type {@link IndexedEndpointType}
+    */
+   public void addAssertionConsumerService( IndexedEndpointType assertionConsumer )
+   {
+      this.assertionConsumerService.add( assertionConsumer );
+   }
+   
+   /**
+    * Add an attribute consumer
+    * @param attributeConsumer an instance of type {@link AttributeConsumingServiceType}
+    */
+   public void addAttributeConsumerService( AttributeConsumingServiceType attributeConsumer )
+   {
+      this.attributeConsumingService.add( attributeConsumer );
+   }
+   
+   /**
+    * Remove an Assertion Consumer Service
+    * @param assertionConsumer an endpoint of type {@link IndexedEndpointType}
+    */
+   public void removeAssertionConsumerService( IndexedEndpointType assertionConsumer )
+   {
+      this.assertionConsumerService.remove( assertionConsumer );
+   }
+   
+   /**
+    * Remove an attribute consumer
+    * @param attributeConsumer an instance of type {@link AttributeConsumingServiceType}
+    */
+   public void removeAttributeConsumerService( AttributeConsumingServiceType attributeConsumer )
+   {
+      this.attributeConsumingService.remove( attributeConsumer );
+   }
 
    /**
     * Gets the value of the assertionConsumerService property.
@@ -50,6 +86,8 @@
    {
       return Collections.unmodifiableList( this.assertionConsumerService );
    }
+   
+   
 
    /**
     * Gets the value of the attributeConsumingService property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SSODescriptorType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SSODescriptorType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/SSODescriptorType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -40,27 +40,79 @@
       super(protocolSupport); 
    }
 
+   /**
+    * Add SLO Service
+    * @param endpt
+    */
    public void addSingleLogoutService( EndpointType endpt )
    {
       this.singleLogoutService.add(endpt);
    }
 
+   /**
+    * Add atrifact resolution service
+    * @param i
+    */
    public void addArtifactResolutionService( IndexedEndpointType i )
    {
       this.artifactResolutionService.add(i);
    }
 
+   /**
+    * Add manage name id service
+    * @param end
+    */
    public void addManageNameIDService( EndpointType end )
    {
       this.manageNameIDService.add(end);
    }
 
+   /**
+    * Add Name ID Format
+    * @param s
+    */
    public void addNameIDFormat( String s )
    {
       this.nameIDFormat.add(s);
    }
+   
+   /**
+    * remove SLO Service
+    * @param endpt
+    */
+   public void removeSingleLogoutService( EndpointType endpt )
+   {
+      this.singleLogoutService.remove(endpt);
+   }
 
    /**
+    * remove atrifact resolution service
+    * @param i
+    */
+   public void removeArtifactResolutionService( IndexedEndpointType i )
+   {
+      this.artifactResolutionService.remove(i);
+   }
+
+   /**
+    * remove manage name id service
+    * @param end
+    */
+   public void removeManageNameIDService( EndpointType end )
+   {
+      this.manageNameIDService.remove(end);
+   }
+
+   /**
+    * remove Name ID Format
+    * @param s
+    */
+   public void removeNameIDFormat( String s )
+   {
+      this.nameIDFormat.remove(s);
+   }
+
+   /**
     * Gets the value of the artifactResolutionService property. 
     * <p>
     * Objects of the following type(s) are allowed in the list

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/TypeWithOtherAttributes.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/TypeWithOtherAttributes.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/metadata/TypeWithOtherAttributes.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -36,12 +36,27 @@
 {
    protected Map<QName, String> otherAttributes = new HashMap<QName, String>();
    
+   /**
+    * Add other attribute
+    * @param qame
+    * @param value
+    */
    public void addOtherAttribute( QName qame, String value )
    {
       otherAttributes.put(qame, value);
    }
    
    /**
+    * Remove other attribute
+    * @param qame
+    * @param value
+    */
+   public void removeOtherAttribute( QName qame )
+   {
+      otherAttributes.remove( qame );
+   }
+   
+   /**
     * Gets a map that contains attributes that aren't bound to any typed property on this class.
     *  
     * 

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AssertionIDRequestType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AssertionIDRequestType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AssertionIDRequestType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -57,10 +57,23 @@
       super(id, version, instant);
    }
 
+   /**
+    * Add assertion id reference
+    * @param id
+    */
    public void addAssertionIDRef( String id )
    {
       assertionIDRef.add( id );
    }
+   
+   /**
+    * remove assertion id reference
+    * @param id
+    */
+   public void removeAssertionIDRef( String id )
+   {
+      assertionIDRef.remove( id );
+   }
 
    /**
     * Gets the value of the assertionIDRef property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AttributeQueryType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AttributeQueryType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AttributeQueryType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -60,12 +60,25 @@
       super(id, version, instant); 
    }
 
+   /**
+    * Add an attribute
+    * @param att
+    */
    public void add( AttributeType att )
    {
       this.attribute.add(att);
    }
    
    /**
+    * Remove an attribute
+    * @param att
+    */
+   public void remove( AttributeType att )
+   {
+      this.attribute.remove(att);
+   }
+   
+   /**
     * Gets the value of the attribute property. 
     */
    public List<AttributeType> getAttribute() 

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AuthzDecisionQueryType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AuthzDecisionQueryType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/AuthzDecisionQueryType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -63,10 +63,23 @@
       super(id, version, instant); 
    }
 
+   /**
+    * Add an action
+    * @param act
+    */
    public void addAction( ActionType act )
    {
       this.action.add(act);
    }
+   
+   /**
+    * Remove an action
+    * @param act
+    */
+   public void removeAction( ActionType act )
+   {
+      this.action.remove(act);
+   }
 
    /**
     * Gets the value of the action property.  

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ExtensionsType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ExtensionsType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ExtensionsType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -48,10 +48,23 @@
 { 
    protected List<Object> any = new ArrayList<Object>();
 
+   /**
+    * Add an extension
+    * @param extension
+    */
    public void addExtension( Object extension )
    {
       any.add( extension );
    }
+   
+   /**
+    * Remove an extension
+    * @param extension
+    */
+   public void removeExtension( Object extension )
+   {
+      any.remove( extension );
+   }
 
    /**
     * Gets the value of the any property. 

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/IDPListType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/IDPListType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/IDPListType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -52,10 +52,23 @@
    protected List<IDPEntryType> idpEntry = new ArrayList<IDPEntryType>(); 
    protected URI getComplete;
 
+   /**
+    * Add an idp entry
+    * @param entry
+    */
    public void addIDPEntry( IDPEntryType entry )
    {
       this.idpEntry.add(entry);
    }
+   
+   /**
+    * Remove an idp entry
+    * @param entry
+    */
+   public void removeIDPEntry( IDPEntryType entry )
+   {
+      this.idpEntry.remove(entry);
+   }
 
    /**
     * Gets the value of the idpEntry property. 

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/LogoutRequestType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/LogoutRequestType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/LogoutRequestType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -146,10 +146,23 @@
         this.encryptedID = value;
     }
     
+    /**
+     * Add session index
+     * @param index
+     */
     public void addSessionIndex( String index )
     {
        this.sessionIndex.add( index );
     }
+    
+    /**
+     * Remove session index
+     * @param index
+     */
+    public void removeSessionIndex( String index )
+    {
+       this.sessionIndex.remove( index );
+    }
 
     /**
      * Gets the value of the sessionIndex property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/RequestedAuthnContextType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/RequestedAuthnContextType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/RequestedAuthnContextType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -52,17 +52,43 @@
    protected List<String> authnContextDeclRef = new ArrayList<String>();
    protected AuthnContextComparisonType comparison;
 
+   /**
+    * Add an authn Context class ref
+    * @param str
+    */
    public void addAuthnContextClassRef( String str )
    {
       this.authnContextClassRef.add(str);
    }
 
+   /**
+    * Add authn context decl ref
+    * @param str
+    */
    public void addAuthnContextDeclRef( String str )
    {
       this.authnContextDeclRef.add(str);
    }
+   
+   /**
+    * Remove an authn Context class ref
+    * @param str
+    */
+   public void removeAuthnContextClassRef( String str )
+   {
+      this.authnContextClassRef.remove(str);
+   }
 
    /**
+    * remove authn context decl ref
+    * @param str
+    */
+   public void removeAuthnContextDeclRef( String str )
+   {
+      this.authnContextDeclRef.remove(str);
+   }
+
+   /**
     * Gets the value of the authnContextClassRef property. 
     * 
     */

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ResponseType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ResponseType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ResponseType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -53,10 +53,23 @@
 { 
    protected List<RTChoiceType> assertions = new ArrayList<ResponseType.RTChoiceType>();
 
+   /**
+    * Add an assertion
+    * @param choice
+    */
    public void addAssertion( RTChoiceType choice )
    {
       assertions.add(choice);
    }
+   
+   /**
+    * Remove an assertion
+    * @param choice
+    */
+   public void removeAssertion( RTChoiceType choice )
+   {
+      assertions.remove(choice);
+   }
 
    /**
     * Gets a read only list of assertions

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ScopingType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ScopingType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/ScopingType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -82,14 +82,7 @@
 
     /**
      * Gets the value of the requesterID property.
-     * 
      * <p>
-     * This accessor method returns a reference to the live list,
-     * not a snapshot. Therefore any modification you make to the
-     * returned list will be present inside the JAXB object.
-     * This is why there is not a <CODE>set</CODE> method for the requesterID property.
-     * 
-     * <p>
      * For example, to add a new item, do as follows:
      * <pre>
      *    getRequesterID().add(newItem);
@@ -107,10 +100,23 @@
         return Collections.unmodifiableList( this.requesterID );
     }
     
+    /**
+     * Add requester id
+     * @param uri
+     */
     public void addRequesterID( URI uri )
     {
        this.requesterID.add( uri );
     }
+    
+    /**
+     * Remove requester id
+     * @param uri
+     */
+    public void removeRequesterID( URI uri )
+    {
+       this.requesterID.remove( uri );
+    }
 
     /**
      * Gets the value of the proxyCount property.

Modified: federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/StatusDetailType.java
===================================================================
--- federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/StatusDetailType.java	2011-01-27 12:40:34 UTC (rev 672)
+++ federation/trunk/picketlink-fed-model/src/main/java/org/picketlink/identity/federation/newmodel/saml/v2/protocol/StatusDetailType.java	2011-01-27 16:42:24 UTC (rev 673)
@@ -49,13 +49,26 @@
 {
    protected List<Object> any = new ArrayList<Object>();
 
+   /**
+    * Add status detail
+    * @param obj
+    */
    public void addStatusDetail( Object obj )
    {
       this.any.add( obj );
    }
+   
+   /**
+    * Remove status detail
+    * @param obj
+    */
+   public void removeStatusDetail( Object obj )
+   {
+      this.any.remove( obj );
+   }
 
    /**
-    * Gets the value of the any property. 
+    * Gets the value of the any property.  Read-Only list
     */
    public List<Object> getAny() 
    {



More information about the jboss-cvs-commits mailing list