[jbossws-commits] JBossWS SVN: r3951 - branches/jbossws-1.2.0.GA_JBWS-1610/jbossws-core/src/main/java/org/jboss/ws/core/server.

[jbossws-commits at lists.jboss.org]

Author: david.boeren
Date: 2007-07-19 10:21:55 -0400 (Thu, 19 Jul 2007)
New Revision: 3951

Modified:
   branches/jbossws-1.2.0.GA_JBWS-1610/jbossws-core/src/main/java/org/jboss/ws/core/server/AbstractServiceEndpointInvoker.java
Log:
Re-adding the fix for Amberpoint case 15726

Modified: branches/jbossws-1.2.0.GA_JBWS-1610/jbossws-core/src/main/java/org/jboss/ws/core/server/AbstractServiceEndpointInvoker.java
===================================================================
--- branches/jbossws-1.2.0.GA_JBWS-1610/jbossws-core/src/main/java/org/jboss/ws/core/server/AbstractServiceEndpointInvoker.java	2007-07-19 12:59:02 UTC (rev 3950)
+++ branches/jbossws-1.2.0.GA_JBWS-1610/jbossws-core/src/main/java/org/jboss/ws/core/server/AbstractServiceEndpointInvoker.java	2007-07-19 14:21:55 UTC (rev 3951)
@@ -150,20 +150,14 @@
             msgContext.setOperationMetaData(opMetaData);
             oneway = opMetaData.isOneWay();
 
-            // Unbind the request message
-            epInv = binding.unbindRequestMessage(opMetaData, reqMessage);
+            // We no longer call unbindRequestMessage here, per case 15726
          }
 
          handlersPass = handlersPass && callRequestHandlerChain(seInfo, HandlerType.ENDPOINT);
          handlersPass = handlersPass && callRequestHandlerChain(seInfo, HandlerType.POST);
 
-         // Check if protocol handlers modified the payload
-         if (msgContext.isModified() == true)
-         {
-            if (log.isDebugEnabled())
-               log.debug("Handler modified body payload, unbind message again");
-            epInv = binding.unbindRequestMessage(opMetaData, reqMessage);
-         }
+         // Unbind the request message
+         epInv = binding.unbindRequestMessage(opMetaData, reqMessage);
 
          if (handlersPass)
          {