cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dk...@apache.org
Subject svn commit: r1163269 - in /cxf/trunk/rt/frontend/jaxws/src: main/java/org/apache/cxf/jaxws/JAXWSMethodInvoker.java main/java/org/apache/cxf/jaxws/context/WebServiceContextImpl.java test/java/org/apache/cxf/jaxws/context/WebServiceContextImplTest.java
Date Tue, 30 Aug 2011 15:56:04 GMT
Author: dkulp
Date: Tue Aug 30 15:56:03 2011
New Revision: 1163269

URL: http://svn.apache.org/viewvc?rev=1163269&view=rev
Log:
[CXF-3758] Some work to make sure the WSC threadlocal is really clear.

Modified:
    cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JAXWSMethodInvoker.java
    cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/context/WebServiceContextImpl.java
    cxf/trunk/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/context/WebServiceContextImplTest.java

Modified: cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JAXWSMethodInvoker.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JAXWSMethodInvoker.java?rev=1163269&r1=1163268&r2=1163269&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JAXWSMethodInvoker.java
(original)
+++ cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JAXWSMethodInvoker.java
Tue Aug 30 15:56:03 2011
@@ -66,9 +66,10 @@ public class JAXWSMethodInvoker extends 
             updateHeader(exchange, ctx);
             throw f;
         } finally {
-            addHandlerProperties(ctx, handlerScopedStuff);
             //clear the WebServiceContextImpl's ThreadLocal variable
             WebServiceContextImpl.clear();
+            
+            addHandlerProperties(ctx, handlerScopedStuff);
         }
         return res;
     }

Modified: cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/context/WebServiceContextImpl.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/context/WebServiceContextImpl.java?rev=1163269&r1=1163268&r2=1163269&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/context/WebServiceContextImpl.java
(original)
+++ cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/context/WebServiceContextImpl.java
Tue Aug 30 15:56:03 2011
@@ -46,10 +46,6 @@ public class WebServiceContextImpl imple
     public WebServiceContextImpl() { 
     }
 
-    public WebServiceContextImpl(MessageContext ctx) { 
-        setMessageContext(ctx);
-    } 
-
     // Implementation of javax.xml.ws.WebServiceContext
 
     public final MessageContext getMessageContext() {
@@ -123,7 +119,7 @@ public class WebServiceContextImpl imple
     }
 
     public static void clear() {
-        context.set(null);
+        context.remove();
     }
 
 }

Modified: cxf/trunk/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/context/WebServiceContextImplTest.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/context/WebServiceContextImplTest.java?rev=1163269&r1=1163268&r2=1163269&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/context/WebServiceContextImplTest.java
(original)
+++ cxf/trunk/rt/frontend/jaxws/src/test/java/org/apache/cxf/jaxws/context/WebServiceContextImplTest.java
Tue Aug 30 15:56:03 2011
@@ -34,15 +34,7 @@ public class WebServiceContextImplTest e
     @After
     public void tearDown() { 
         WebServiceContextImpl.clear();
-    } 
-
-    @Test
-    public void testConstructor() { 
-        MessageImpl msg = new MessageImpl();
-        WrappedMessageContext msgCtx = new WrappedMessageContext(msg); 
-        WebServiceContextImpl ctx = new WebServiceContextImpl(msgCtx);
-        assertSame(msgCtx, ctx.getMessageContext());
-    }     
+    }    
 
     @Test
     public void testGetSetMessageContext() { 



Mime
View raw message