Author: antelder
Date: Thu Nov  3 14:45:25 2011
New Revision: 1197162

URL: http://svn.apache.org/viewvc?rev=1197162&view=rev
Log:
TUSCANY-3972 JMSBIndingProcessor.writeDestinationProperties should handle 
Destination Type

Modified:
    
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
    
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java

Modified: 
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java?rev=1197162&r1=1197161&r2=1197162&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
 Thu Nov  3 14:45:25 2011
@@ -1421,11 +1421,10 @@ public class JMSBindingProcessor extends
             writer.writeAttribute("jndiName", destinationName);            
         }
 
-        // Type not handled yet
-        // String destinationType = jmsBinding.getDestinationType();
-        // if ( destinationType != null && destinationType.length() > 0) {
-        //     writer.writeAttribute("type", destinationType);            
-        // }
+        String destinationType = jmsBinding.getDestinationType();
+        if ( destinationType != null && destinationType.length() > 0) {
+            writer.writeAttribute("type", destinationType);            
+        }
 
         String destinationCreate = jmsBinding.getDestinationCreate();
         if ( destinationCreate != null && destinationCreate.length() > 0 &&

Modified: 
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java?rev=1197162&r1=1197161&r2=1197162&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java
 Thu Nov  3 14:45:25 2011
@@ -404,7 +404,7 @@ public class JMSBindingProcessorWriteTes
                        "<component name=\"HelloWorldComponent\">" + 
                            "<service name=\"HelloWorldService\">" +
                              "<binding.jms><operationProperties name=\"JKL\" 
/> " +
-                               "<destination jndiName=\"AAA\">" +
+                               "<destination jndiName=\"AAA\" type=\"queue\">" 
+
                                  "<property name=\"AAAProp\" 
type=\"string\"></property>" +
                                "</destination> " +
                                "<connectionFactory jndiName=\"ABC\" /> "+


Reply via email to