[jboss-cvs] JBossAS SVN: r71708 - projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl.

[jboss-cvs-commits at lists.jboss.org]

Author: anil.saldhana at jboss.com
Date: 2008-04-03 14:12:30 -0400 (Thu, 03 Apr 2008)
New Revision: 71708

Modified:
   projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeImpl.java
   projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeMarshaller.java
Log:
minor refactor

Modified: projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeImpl.java
===================================================================
--- projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeImpl.java	2008-04-03 18:10:59 UTC (rev 71707)
+++ projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeImpl.java	2008-04-03 18:12:30 UTC (rev 71708)
@@ -121,7 +121,7 @@
             XMLConstants.XSI_NS, "type", XMLConstants.XSI_PREFIX);
       xsiAttr.setTextContent("xacml-samlp:XACMLAuthzDecisionStatement");
       element.setAttributeNodeNS(xsiAttr);  
-     
+      
       return element;
    }   
 }
\ No newline at end of file

Modified: projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeMarshaller.java
===================================================================
--- projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeMarshaller.java	2008-04-03 18:10:59 UTC (rev 71707)
+++ projects/security/security-xacml/trunk/jboss-xacml-saml/src/main/java/org/jboss/security/xacml/saml/integration/opensaml/impl/XACMLAuthzDecisionStatementTypeMarshaller.java	2008-04-03 18:12:30 UTC (rev 71708)
@@ -29,6 +29,7 @@
 import org.opensaml.xml.XMLObject;
 import org.opensaml.xml.io.MarshallingException;
 import org.opensaml.xml.util.XMLHelper;
+import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 import org.w3c.dom.Node;
 
@@ -46,7 +47,8 @@
    {
       XACMLAuthzDecisionStatementType xacmlType = (XACMLAuthzDecisionStatementType) xmlObject;
       
-      Element xacmlDecisionElement = xacmlType.asElement(parentElement.getOwnerDocument());
+      Document ownerDocument = parentElement.getOwnerDocument();
+      Element xacmlDecisionElement = xacmlType.asElement(ownerDocument);
       
       parentElement.appendChild(xacmlDecisionElement);
       
@@ -56,7 +58,7 @@
          Node responseRoot = responseContext.getDocumentElement();
          if(responseRoot != null)
          {
-            XMLHelper.adoptElement((Element) responseRoot, parentElement.getOwnerDocument());  
+            XMLHelper.adoptElement((Element) responseRoot, ownerDocument);  
             xacmlDecisionElement.appendChild(responseRoot);
          }
          else
@@ -64,7 +66,7 @@
             try
             {
                Element elem = JBossXACMLUtil.getResponseContextElement(responseContext);
-               XMLHelper.adoptElement(elem, parentElement.getOwnerDocument());  
+               XMLHelper.adoptElement(elem, ownerDocument);  
                xacmlDecisionElement.appendChild(elem);
             }
             catch (Exception e)
@@ -80,7 +82,7 @@
          Node requestRoot = requestContext.getDocumentElement();
          if(requestRoot != null)
          { 
-            XMLHelper.adoptElement((Element) requestRoot, parentElement.getOwnerDocument());  
+            XMLHelper.adoptElement((Element) requestRoot, ownerDocument);  
             xacmlDecisionElement.appendChild(requestRoot);
          } 
       }