svn commit: r558683 - /webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/policy/SandeshaPolicyBean.java

[sanka]

Author: sanka
Date: Mon Jul 23 03:31:34 2007
New Revision: 558683

URL: http://svn.apache.org/viewvc?view=rev&rev=558683
Log:
Fixed: NPE is thrown when tries to serialize the policy from a partially built 
SandeshaPolicyBean.


Modified:
    
webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/policy/SandeshaPolicyBean.java

Modified: 
webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/policy/SandeshaPolicyBean.java
URL: 
http://svn.apache.org/viewvc/webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/policy/SandeshaPolicyBean.java?view=diff&rev=558683&r1=558682&r2=558683
==============================================================================
--- 
webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/policy/SandeshaPolicyBean.java
 (original)
+++ 
webservices/sandesha/trunk/java/modules/core/src/main/java/org/apache/sandesha2/policy/SandeshaPolicyBean.java
 Mon Jul 23 03:31:34 2007
@@ -292,20 +292,27 @@
                        
writer.writeCharacters(Long.toString(getInactivityTimeoutInterval()));
                        writer.writeEndElement();
                        
+            
+            // FIXME : Assign default values if inactivityTimeoutMeasure is 
set to null
+            if (inactivityTimeoutMeasure != null && 
inactivityTimeoutMeasure.trim().length() > 0) {
                        // <wsrm:InactivityTimeoutMeasure />
                        writer.writeStartElement(prefix, 
Sandesha2Constants.Assertions.Q_ELEM_INACTIVITY_TIMEOUT_MEASURES.getLocalPart(),
 namespaceURI);
                        writer.writeCharacters(inactivityTimeoutMeasure);
                        writer.writeEndElement();
+            }
 
-                       // <wsrm:SequenceRemovalTimeout />
+            // <wsrm:SequenceRemovalTimeout />
                        writer.writeStartElement(prefix, 
Sandesha2Constants.Assertions.Q_ELEM_SEQUENCE_REMOVAL_TIMEOUT.getLocalPart(), 
namespaceURI);
                        
writer.writeCharacters(Long.toString(getSequenceRemovalTimeoutInterval()));
                        writer.writeEndElement();
                        
                        // <wsrm:SequenceRemovalTimeoutMeasure />
+            // FIXME : Assign default values if sequenceRemovalTimeoutMeasure 
is set to null
+            if (sequenceRemovalTimeoutMeasure != null && 
sequenceRemovalTimeoutMeasure.trim().length() > 0) {
                        writer.writeStartElement(prefix, 
Sandesha2Constants.Assertions.Q_ELEM_SEQUENCE_REMOVAL_TIMEOUT_MEASURES.getLocalPart(),
 namespaceURI);
                        writer.writeCharacters(sequenceRemovalTimeoutMeasure);
                        writer.writeEndElement();
+            }
 
                        // <wsrm:InvokeInOrder />
                        writer.writeStartElement(prefix, 
Sandesha2Constants.Assertions.Q_ELEM_INVOKE_INORDER.getLocalPart(), 
namespaceURI);



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]