tomj 2002/09/17 12:54:54
Modified: java/src/org/apache/axis/message RPCElement.java
Log:
Improve comments, particularly one that asks me to clarify :-)
Revision Changes Path
1.73 +5 -7 xml-axis/java/src/org/apache/axis/message/RPCElement.java
Index: RPCElement.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/message/RPCElement.java,v
retrieving revision 1.72
retrieving revision 1.73
diff -u -r1.72 -r1.73
--- RPCElement.java 9 Sep 2002 19:30:46 -0000 1.72
+++ RPCElement.java 17 Sep 2002 19:54:53 -0000 1.73
@@ -116,10 +116,8 @@
}
if (operations != null && operations.length > 0) {
- // IF we're doc/literal... we can't count on the element name
- // being the method name.
- elementIsFirstParam = (operations[0].getStyle() ==
- Style.DOCUMENT);
+ // if we're document style, the top level element is important
+ elementIsFirstParam = (operations[0].getStyle() == Style.DOCUMENT);
}
this.operations = operations;
@@ -286,10 +284,10 @@
msgContext.getOperationStyle() == Style.RPC ||
msgContext.getOperationStyle() == Style.WRAPPED);
- // I don't quite understand why this is necessary, but when
- // I have MIME and a no-param document WSDL, if I don't check
+ // When I have MIME and a no-param document WSDL, if I don't check
// for no params here, the server chokes with "can't find Body".
- // Tom, maybe you have some insight?
+ // because it will be looking for the enclosing element always
+ // found in an RPC-style (and wrapped) request
boolean noParams = params.size() == 0;
if (isRPC || noParams) {
