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 2013/10/08 22:59:33 UTC
svn commit: r1530419 - in /cxf/trunk: core/
core/src/main/java/org/apache/cxf/service/model/
rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/
rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/
Author: dkulp
Date: Tue Oct 8 20:59:32 2013
New Revision: 1530419
URL: http://svn.apache.org/r1530419
Log:
[CXF-5306] If an out of band header part has the same name as a regular part, there can be confusion as to what get's written out where.
Modified:
cxf/trunk/core/pom.xml
cxf/trunk/core/src/main/java/org/apache/cxf/service/model/AbstractMessageContainer.java
cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/WrapperClassInInterceptor.java
Modified: cxf/trunk/core/pom.xml
URL: http://svn.apache.org/viewvc/cxf/trunk/core/pom.xml?rev=1530419&r1=1530418&r2=1530419&view=diff
==============================================================================
--- cxf/trunk/core/pom.xml (original)
+++ cxf/trunk/core/pom.xml Tue Oct 8 20:59:32 2013
@@ -311,15 +311,6 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
- <executions>
- <execution>
- <id>package</id>
- <phase>package</phase>
- <goals>
- <goal>jar</goal>
- </goals>
- </execution>
- </executions>
<configuration>
<detectOfflineLinks>false</detectOfflineLinks>
<header>Apache CXF API</header>
@@ -374,6 +365,15 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
+ <executions>
+ <execution>
+ <id>package</id>
+ <phase>package</phase>
+ <goals>
+ <goal>jar</goal>
+ </goals>
+ </execution>
+ </executions>
</plugin>
</plugins>
</build>
@@ -389,6 +389,23 @@
<properties>
<buildnumber.connection.url>scm:git:http://</buildnumber.connection.url>
</properties>
+ <build>
+ <plugins>
+ <plugin>
+ <groupId>org.apache.maven.plugins</groupId>
+ <artifactId>maven-javadoc-plugin</artifactId>
+ <executions>
+ <execution>
+ <id>package</id>
+ <phase>package</phase>
+ <goals>
+ <goal>jar</goal>
+ </goals>
+ </execution>
+ </executions>
+ </plugin>
+ </plugins>
+ </build>
</profile>
</profiles>
Modified: cxf/trunk/core/src/main/java/org/apache/cxf/service/model/AbstractMessageContainer.java
URL: http://svn.apache.org/viewvc/cxf/trunk/core/src/main/java/org/apache/cxf/service/model/AbstractMessageContainer.java?rev=1530419&r1=1530418&r2=1530419&view=diff
==============================================================================
--- cxf/trunk/core/src/main/java/org/apache/cxf/service/model/AbstractMessageContainer.java (original)
+++ cxf/trunk/core/src/main/java/org/apache/cxf/service/model/AbstractMessageContainer.java Tue Oct 8 20:59:32 2013
@@ -155,11 +155,12 @@ public abstract class AbstractMessageCon
*/
public MessagePartInfo getMessagePart(QName name) {
MessagePartInfo mpi = messageParts.get(name);
- if (mpi == null) {
- for (MessagePartInfo mpi2 : messageParts.values()) {
- if (name.equals(mpi2.getConcreteName())) {
- return mpi2;
- }
+ if (mpi != null) {
+ return mpi;
+ }
+ for (MessagePartInfo mpi2 : messageParts.values()) {
+ if (name.equals(mpi2.getConcreteName())) {
+ return mpi2;
}
}
for (MessagePartInfo mpi2 : getOutOfBandParts()) {
Modified: cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java?rev=1530419&r1=1530418&r2=1530419&view=diff
==============================================================================
--- cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java (original)
+++ cxf/trunk/rt/bindings/soap/src/main/java/org/apache/cxf/binding/soap/SoapBindingFactory.java Tue Oct 8 20:59:32 2013
@@ -581,7 +581,7 @@ public class SoapBindingFactory extends
}
if (minfo == null) {
- minfo = new MessageInfo(null, type, msg.getQName());
+ minfo = new MessageInfo(bop.getOperationInfo(), type, msg.getQName());
}
buildMessage(minfo, msg, schemas, nextId, partName);
@@ -609,8 +609,6 @@ public class SoapBindingFactory extends
}
}
-
-
if (isInput) {
minfo = unwrapped.getInput();
type = MessageInfo.Type.INPUT;
@@ -639,7 +637,7 @@ public class SoapBindingFactory extends
throw new RuntimeException("Problem with WSDL: part element in message "
+ msg.getQName().getLocalPart()
+ " does not specify a name.");
- }
+ }
QName pqname = new QName(minfo.getName().getNamespaceURI(), part.getName());
MessagePartInfo pi = minfo.getMessagePart(pqname);
if (pi != null
Modified: cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/WrapperClassInInterceptor.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/WrapperClassInInterceptor.java?rev=1530419&r1=1530418&r2=1530419&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/WrapperClassInInterceptor.java (original)
+++ cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/interceptors/WrapperClassInInterceptor.java Tue Oct 8 20:59:32 2013
@@ -139,10 +139,10 @@ public class WrapperClassInInterceptor e
for (MessagePartInfo part : messageInfo.getMessageParts()) {
if (Boolean.TRUE.equals(part.getProperty(ReflectionServiceFactoryBean.HEADER))) {
MessagePartInfo mpi = wrappedMessageInfo.getMessagePart(part.getName());
- if (lst.hasValue(mpi)) {
+ if (mpi != null && lst.hasValue(mpi)) {
count++;
newParams.put(part, lst.get(mpi));
- } else if (mpi.getTypeClass() == null) {
+ } else if (mpi == null || mpi.getTypeClass() == null) {
//header, but not mapped to a param on the method
if (removes == null) {
removes = new ArrayList<Integer>();