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/05/07 19:19:31 UTC
svn commit: r772714 -
/cxf/trunk/rt/bindings/xml/src/main/java/org/apache/cxf/binding/xml/XMLFormatValidator.java
Author: dkulp
Date: Thu May 7 17:19:30 2009
New Revision: 772714
URL: http://svn.apache.org/viewvc?rev=772714&view=rev
Log:
[CXF-2201] Oneway ops confuse the XML validator
Modified:
cxf/trunk/rt/bindings/xml/src/main/java/org/apache/cxf/binding/xml/XMLFormatValidator.java
Modified: cxf/trunk/rt/bindings/xml/src/main/java/org/apache/cxf/binding/xml/XMLFormatValidator.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/bindings/xml/src/main/java/org/apache/cxf/binding/xml/XMLFormatValidator.java?rev=772714&r1=772713&r2=772714&view=diff
==============================================================================
--- cxf/trunk/rt/bindings/xml/src/main/java/org/apache/cxf/binding/xml/XMLFormatValidator.java (original)
+++ cxf/trunk/rt/bindings/xml/src/main/java/org/apache/cxf/binding/xml/XMLFormatValidator.java Thu May 7 17:19:30 2009
@@ -82,8 +82,11 @@
if (inExtensors != null) {
itIn = inExtensors.iterator();
}
- if (findXMLFormatRootNode(itIn, bo, path + "-input")) {
- // Input check correct, continue to check output binding
+ if (!findXMLFormatRootNode(itIn, bo, path + "-input")) {
+ return false;
+ }
+ // Input check correct, continue to check output binding
+ if (op.getOutput() != null) {
needRootNode = false;
if (op.getOutput().getMessageParts().size() == 0
|| op.getOutput().getMessageParts().size() > 1) {
@@ -102,8 +105,6 @@
return false;
}
}
- } else {
- return false;
}
}
}