cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Glen Mazza <glen.ma...@verizon.net>
Subject Re: svn commit: r583877 - in /incubator/cxf/trunk: rt/core/src/main/java/org/apache/cxf/interceptor/ rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/ rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/ rt/frontend/jaxws/src/main/java/o...
Date Fri, 12 Oct 2007 02:05:09 GMT
Am Donnerstag, den 11.10.2007, 16:53 +0000 schrieb dkulp@apache.org:
> Author: dkulp
> Date: Thu Oct 11 09:53:40 2007
> New Revision: 583877

> Modified:
> incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java
> URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java?rev=583877&r1=583876&r2=583877&view=diff
> ==============================================================================
> --- incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java
(original)
> +++ incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/service/factory/ReflectionServiceFactoryBean.java
Thu Oct 11 09:53:40 2007
> @@ -903,7 +903,19 @@
>      }
>  
>      protected void createOutputWrappedMessageParts(OperationInfo op, Method method,
MessageInfo outMsg) {
> -        MessagePartInfo part = outMsg.addMessagePart("result");
> +        String partName = null;
> +        for (Iterator itr = serviceConfigurations.iterator(); itr.hasNext();) {
> +            AbstractServiceConfiguration c = (AbstractServiceConfiguration)itr.next();
> +            partName = c.getResponseWrapperPartName(op, method);
> +            if (partName != null) {
> +                break;
> +            }
> +        }
> +        if (partName == null) {
> +            partName = "parameters";
> +        }

I may be misunderstanding the code here, but the WSDL schema[1] seems to
say that partName is required, and if you don't have it, the WSDL is no
good--it should be rejected.  Are you sure you want to be adding
"parameters" in under that case?

Glen

[1] http://www.w3.org/TR/wsdl#A4.1



Mime
View raw message