[java-opensaml COMMIT] /trunk/opensaml-saml-api/src/main/java/org/opensaml/saml/common/AbstractSignableSAMLObject.java

[noreply at shibboleth.net]

Author: putmanb
Date: Thu Jun 20 03:01:48 2013
New Revision: 3393

URL: http://svn.shibboleth.net/view/java-opensaml?rev=3393&view=rev
Log:
Minor tweak: in abstract signable SAMLObject, only auto-add SAML content reference if there isn't already one there.

Modified:
    trunk/opensaml-saml-api/src/main/java/org/opensaml/saml/common/AbstractSignableSAMLObject.java

Modified: trunk/opensaml-saml-api/src/main/java/org/opensaml/saml/common/AbstractSignableSAMLObject.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-saml-api/src/main/java/org/opensaml/saml/common/AbstractSignableSAMLObject.java?rev=3393&r1=3392&r2=3393&view=diff
==============================================================================
--- trunk/opensaml-saml-api/src/main/java/org/opensaml/saml/common/AbstractSignableSAMLObject.java (original)
+++ trunk/opensaml-saml-api/src/main/java/org/opensaml/saml/common/AbstractSignableSAMLObject.java Thu Jun 20 03:01:48 2013
@@ -56,7 +56,7 @@
      * changed prior to marshalling this object.
      */
     public void setSignature(Signature newSignature) {
-        if(newSignature != null){
+        if(newSignature != null && newSignature.getContentReferences().isEmpty()) {
             newSignature.getContentReferences().add(new SAMLObjectContentReference(this));
         }
         super.setSignature(newSignature);