[jboss-cvs] JBossAS SVN: r111093 - projects/security/security-xacml/trunk/jboss-xacml/src/main/java/org/jboss/security/xacml/core.

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

Author: anil.saldhana at jboss.com
Date: 2011-04-01 11:32:56 -0400 (Fri, 01 Apr 2011)
New Revision: 111093

Modified:
   projects/security/security-xacml/trunk/jboss-xacml/src/main/java/org/jboss/security/xacml/core/JBossResponseContext.java
Log:
SECURITY-561: bring the obligations type outside

Modified: projects/security/security-xacml/trunk/jboss-xacml/src/main/java/org/jboss/security/xacml/core/JBossResponseContext.java
===================================================================
--- projects/security/security-xacml/trunk/jboss-xacml/src/main/java/org/jboss/security/xacml/core/JBossResponseContext.java	2011-04-01 15:24:14 UTC (rev 111092)
+++ projects/security/security-xacml/trunk/jboss-xacml/src/main/java/org/jboss/security/xacml/core/JBossResponseContext.java	2011-04-01 15:32:56 UTC (rev 111093)
@@ -153,16 +153,19 @@
          Set<Obligation> obligationsSet = result.getObligations();
          if(obligationsSet != null)
          {
+            ObligationsType obligationsType = new ObligationsType();
             for(Obligation obl:obligationsSet)
             {
                ObligationType obType = new ObligationType();
                obType.setObligationId(obl.getId().toASCIIString());
                obType.setFulfillOn(EffectType.fromValue(Result.DECISIONS[obl.getFulfillOn()]));
-            
-               ObligationsType obligationsType = new ObligationsType();
+               
                obligationsType.getObligation().add(obType);
-               resultType.setObligations(obligationsType);  
             }
+            if( obligationsSet.size() > 0 )
+            {
+               resultType.setObligations(obligationsType);
+            }
          }
       }
       return resultType;