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;
     }
 }


Reply via email to