You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2009/03/12 13:21:35 UTC

svn commit: r752843 - in /servicemix/components/bindings/servicemix-jms/trunk/src: main/java/org/apache/servicemix/jms/endpoints/ test/java/org/apache/servicemix/jms/

Author: gnodet
Date: Thu Mar 12 12:21:35 2009
New Revision: 752843

URL: http://svn.apache.org/viewvc?rev=752843&view=rev
Log:
SMXCOMP-44: default value should be true for rollbackOnError to restore compatibility

Modified:
    servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/DefaultConsumerMarshaler.java
    servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/JmsSoapConsumerMarshaler.java
    servicemix/components/bindings/servicemix-jms/trunk/src/test/java/org/apache/servicemix/jms/JmsProviderConsumerEndpointTest.java

Modified: servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/DefaultConsumerMarshaler.java
URL: http://svn.apache.org/viewvc/servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/DefaultConsumerMarshaler.java?rev=752843&r1=752842&r2=752843&view=diff
==============================================================================
--- servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/DefaultConsumerMarshaler.java (original)
+++ servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/DefaultConsumerMarshaler.java Thu Mar 12 12:21:35 2009
@@ -53,7 +53,7 @@
 public class DefaultConsumerMarshaler extends AbstractJmsMarshaler implements JmsConsumerMarshaler {
     
     private URI mep;
-    private boolean rollbackOnError;
+    private boolean rollbackOnError = true;
 
     public DefaultConsumerMarshaler() {
         this.mep = JbiConstants.IN_ONLY;

Modified: servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/JmsSoapConsumerMarshaler.java
URL: http://svn.apache.org/viewvc/servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/JmsSoapConsumerMarshaler.java?rev=752843&r1=752842&r2=752843&view=diff
==============================================================================
--- servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/JmsSoapConsumerMarshaler.java (original)
+++ servicemix/components/bindings/servicemix-jms/trunk/src/main/java/org/apache/servicemix/jms/endpoints/JmsSoapConsumerMarshaler.java Thu Mar 12 12:21:35 2009
@@ -44,7 +44,7 @@
     private Binding<?> binding;
     private boolean useJbiWrapper = true;
     private Policy[] policies;
-    private boolean rollbackOnError;
+    private boolean rollbackOnError = true;
 
     /**
      * @return the binding

Modified: servicemix/components/bindings/servicemix-jms/trunk/src/test/java/org/apache/servicemix/jms/JmsProviderConsumerEndpointTest.java
URL: http://svn.apache.org/viewvc/servicemix/components/bindings/servicemix-jms/trunk/src/test/java/org/apache/servicemix/jms/JmsProviderConsumerEndpointTest.java?rev=752843&r1=752842&r2=752843&view=diff
==============================================================================
--- servicemix/components/bindings/servicemix-jms/trunk/src/test/java/org/apache/servicemix/jms/JmsProviderConsumerEndpointTest.java (original)
+++ servicemix/components/bindings/servicemix-jms/trunk/src/test/java/org/apache/servicemix/jms/JmsProviderConsumerEndpointTest.java Thu Mar 12 12:21:35 2009
@@ -117,6 +117,7 @@
         endpoint.setEndpoint("endpoint");
         DefaultConsumerMarshaler marshaler = new DefaultConsumerMarshaler();
         marshaler.setMep(new URI("http://www.w3.org/2004/08/wsdl/in-out"));
+        marshaler.setRollbackOnError(false);
         endpoint.setMarshaler(marshaler);
         endpoint.setListenerType("simple");
         endpoint.setConnectionFactory(connFactory);