You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cxf.apache.org by dk...@apache.org on 2009/06/30 21:38:17 UTC

svn commit: r789896 - /cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java

Author: dkulp
Date: Tue Jun 30 19:38:17 2009
New Revision: 789896

URL: http://svn.apache.org/viewvc?rev=789896&view=rev
Log:
CXF-2292 - don't map faults on one way ops

Modified:
    cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java

Modified: cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java?rev=789896&r1=789895&r2=789896&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java (original)
+++ cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java Tue Jun 30 19:38:17 2009
@@ -1381,7 +1381,8 @@
             }
         }
 
-        if (hasOutMessage(method)) {
+        boolean hasOut = hasOutMessage(method);
+        if (hasOut) {
             // Setup the output message
             MessageInfo outMsg = op.createMessage(createOutputMessageName(op, method),
                                                   MessageInfo.Type.OUTPUT);
@@ -1459,7 +1460,10 @@
         //when doing java->wsdl->java
         setParameterOrder(method, paramClasses, op);
 
-        initializeFaults(intf, op, method);
+        if (hasOut) {
+            // Faults are only valid if not a one-way operation
+            initializeFaults(intf, op, method);
+        }
     }
     
     private void setParameterOrder(Method method, Class[] paramClasses, OperationInfo op) {