You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@axis.apache.org by sa...@apache.org on 2012/02/28 14:13:46 UTC

svn commit: r1294631 - /axis/axis2/java/transports/trunk/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java

Author: sagara
Date: Tue Feb 28 13:13:46 2012
New Revision: 1294631

URL: http://svn.apache.org/viewvc?rev=1294631&view=rev
Log:
Applied patch for TRANSPORTS-47.

Modified:
    axis/axis2/java/transports/trunk/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java

Modified: axis/axis2/java/transports/trunk/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java
URL: http://svn.apache.org/viewvc/axis/axis2/java/transports/trunk/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java?rev=1294631&r1=1294630&r2=1294631&view=diff
==============================================================================
--- axis/axis2/java/transports/trunk/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java (original)
+++ axis/axis2/java/transports/trunk/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSMessageReceiver.java Tue Feb 28 13:13:46 2012
@@ -192,23 +192,19 @@ public class JMSMessageReceiver {
             msgContext.setProperty(BaseConstants.USER_TRANSACTION, ut);
         }
 
-        try {
-            jmsListener.handleIncomingMessage(
+        jmsListener.handleIncomingMessage(
                 msgContext,
                 JMSUtils.getTransportHeaders(message),
                 soapAction,
                 contentTypeInfo.getContentType());
 
-        } finally {
-
-            Object o = msgContext.getProperty(BaseConstants.SET_ROLLBACK_ONLY);
-            if (o != null) {
-                if ((o instanceof Boolean && ((Boolean) o)) ||
+        Object o = msgContext.getProperty(BaseConstants.SET_ROLLBACK_ONLY);
+        if (o != null) {
+            if ((o instanceof Boolean && ((Boolean) o)) ||
                     (o instanceof String && Boolean.valueOf((String) o))) {
-                    return false;
-                }
+                return false;
             }
-            return true;
         }
+        return true;
     }
 }