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) {



Reply via email to