You know what, forget that message. That was apparently jetlag talking. I hadn't really looked closely enough at this, and upon doing so I'm fine with it - the data really was being used all over the place. Ignore the man across the ocean, please. (no, not Glyn!)
--G ----- Original Message ----- From: "Glen Daniels" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Tuesday, June 11, 2002 3:38 PM Subject: Re: cvs commit: xml-axis/java/src/org/apache/axis/client Call.java > I'm not going to -1 this, but don't think it's a very good idea. We have > basically taken a whole bunch of data which was previously described close to > the actual package in which it is used, and dropped it all willy-nilly into a > single place just because it happens to be a type with a string representation. > Yuk (or "Le Yuk", as they say here in Paris). > > --Glen > > ----- Original Message ----- > From: <[EMAIL PROTECTED]> > To: <[EMAIL PROTECTED]> > Sent: Tuesday, June 11, 2002 3:26 PM > Subject: cvs commit: xml-axis/java/src/org/apache/axis/client Call.java > > > > rsitze 2002/06/11 06:26:16 > > > > Modified: java/src/org/apache/axis/description ServiceDesc.java > > OperationDesc.java > > java/src/org/apache/axis/encoding > > SerializationContextImpl.java > > java/src/org/apache/axis/wsdl/toJava Emitter.java > > JavaDeployWriter.java > > java/test/wsdl Wsdl2javaAntTask.java > > java/src/org/apache/axis/deployment/wsdd WSDDService.java > > java/src/org/apache/axis/message BodyBuilder.java > > RPCElement.java > > java/src/org/apache/axis/handlers/soap SOAPService.java > > java/src/org/apache/axis/providers/java JavaProvider.java > > java/src/org/apache/axis/handlers JWSProcessor.java > > java/src/org/apache/axis MessageContext.java > > java/src/org/apache/axis/wsdl WSDL2Java.java > > java/src/org/apache/axis/utils axisNLS.properties > > java/src/org/apache/axis/client Call.java > > Added: java/src/org/apache/axis/enum Style.java Scope.java > > Enum.java > > Log: > > Introduce enumerations for Style & Scope integer constants. > > Allows access to 'String' representation of same, conversion > > between, better type safetly, and (bottom line) consolidation > > of these functions to one point. > > > > Revision Changes Path > > 1.25 +20 -52 > xml-axis/java/src/org/apache/axis/description/ServiceDesc.java > > > > Index: ServiceDesc.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/description/ServiceDesc.java,v > > retrieving revision 1.24 > > retrieving revision 1.25 > > diff -u -r1.24 -r1.25 > > --- ServiceDesc.java 4 Jun 2002 20:34:02 -0000 1.24 > > +++ ServiceDesc.java 11 Jun 2002 13:26:14 -0000 1.25 > > @@ -54,21 +54,26 @@ > > */ > > package org.apache.axis.description; > > > > +import java.lang.reflect.InvocationTargetException; > > +import java.lang.reflect.Method; > > +import java.lang.reflect.Modifier; > > +import java.util.ArrayList; > > +import java.util.Collection; > > +import java.util.HashMap; > > +import java.util.Iterator; > > +import java.util.List; > > + > > +import javax.xml.rpc.holders.Holder; > > +import javax.xml.rpc.namespace.QName; > > + > > +import org.apache.axis.encoding.TypeMapping; > > +import org.apache.axis.enum.*; > > import org.apache.axis.utils.JavaUtils; > > import org.apache.axis.utils.bytecode.ExtractorFactory; > > -import org.apache.axis.encoding.TypeMapping; > > import org.apache.axis.wsdl.Skeleton; > > import org.apache.commons.logging.Log; > > import org.apache.commons.logging.LogFactory; > > > > -import javax.xml.rpc.namespace.QName; > > -import javax.xml.rpc.holders.Holder; > > -import java.util.*; > > -import java.lang.reflect.Method; > > -import java.lang.reflect.InvocationTargetException; > > -import java.lang.reflect.Field; > > -import java.lang.reflect.Modifier; > > - > > /** > > * A ServiceDesc is an abstract description of a service. > > * > > @@ -82,43 +87,6 @@ > > protected static Log log = > > LogFactory.getLog(ServiceDesc.class.getName()); > > > > - public static final int STYLE_RPC = 0; > > - public static final int STYLE_DOCUMENT = 1; > > - public static final int STYLE_WRAPPED = 2; > > - public static final int STYLE_MESSAGE = 3; > > - > > - private static final String[] styleStrings = { "rpc", "document", > "wrapped", "message" }; > > - > > - /** > > - * Utility function to return a string representation of a style > > - * constant. > > - */ > > - public static String getStringFromStyle(int style) > > - { > > - return (style >= STYLE_RPC && style <= STYLE_MESSAGE) ? > styleStrings[style] : null; > > - } > > - > > - /** > > - * Utility function to convert string to operation style constants > > - * > > - * @param operationStyle "rpc", "document", "wrapped", or "message" > > - * @return either STYLE_RPC, STYLE_DOCUMENT or STYLE_WRAPPED (all > defined > > - * in org.apache.axis.description.ServiceDesc) > > - */ > > - public static int getStyleFromString(String operationStyle) > > - { > > - for (int idx = 0; idx <= styleStrings.length; idx++) > > - if (styleStrings[idx].equalsIgnoreCase(operationStyle)) > > - return idx; > > - > > - // Not one of the recognized values. We're going to return RPC > > - // as the default, but log an error. > > - log.error(JavaUtils.getMessage("badStyle", operationStyle)); > > - > > - return ServiceDesc.STYLE_RPC; > > - } > > - > > - > > /** The name of this service */ > > private String name = null; > > > > @@ -130,7 +98,7 @@ > > private List disallowedMethods = null; > > > > /** Style */ > > - private int style = STYLE_RPC; > > + private Style style = Style.RPC; > > > > /** Implementation class */ > > private Class implClass = null; > > @@ -184,11 +152,11 @@ > > public ServiceDesc() { > > } > > > > - public int getStyle() { > > + public Style getStyle() { > > return style; > > } > > > > - public void setStyle(int style) { > > + public void setStyle(Style style) { > > this.style = style; > > } > > > > @@ -203,7 +171,7 @@ > > */ > > public boolean isWrapped() > > { > > - return ((style == STYLE_RPC) || (style == STYLE_WRAPPED)); > > + return ((style == Style.RPC) || (style == Style.WRAPPED)); > > } > > > > public String getWSDLFile() { > > @@ -384,7 +352,7 @@ > > { > > // If we're MESSAGE style, we should only have a single operation, > > // to which we'll pass any XML we receive. > > - if (style == STYLE_MESSAGE) { > > + if (style == Style.MESSAGE) { > > return new OperationDesc [] { > (OperationDesc)operations.get(0) }; > > } > > > > @@ -396,7 +364,7 @@ > > ArrayList overloads = (ArrayList)qname2OperationsMap.get(qname); > > > > if (overloads == null) { > > - if ((style == STYLE_RPC) && (name2OperationsMap != null)) { > > + if ((style == Style.RPC) && (name2OperationsMap != null)) { > > // Try ignoring the namespace....? > > overloads = > (ArrayList)name2OperationsMap.get(qname.getLocalPart()); > > } > > > > > > > > 1.11 +8 -6 > xml-axis/java/src/org/apache/axis/description/OperationDesc.java > > > > Index: OperationDesc.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/description/OperationDesc.java,v > > retrieving revision 1.10 > > retrieving revision 1.11 > > diff -u -r1.10 -r1.11 > > --- OperationDesc.java 1 May 2002 01:50:50 -0000 1.10 > > +++ OperationDesc.java 11 Jun 2002 13:26:14 -0000 1.11 > > @@ -58,6 +58,8 @@ > > import java.util.ArrayList; > > import java.util.Iterator; > > import java.lang.reflect.Method; > > + > > +import org.apache.axis.enum.Style; > > import org.apache.commons.logging.Log; > > import org.apache.commons.logging.LogFactory; > > > > @@ -89,7 +91,7 @@ > > private Method method; > > > > /** This operation's style. If null, we default to our parent's */ > > - private Integer style; > > + private Style style = null; > > > > /** The number of "in" params (i.e. IN or INOUT) for this operation */ > > private int numInParams = 0; > > @@ -182,25 +184,25 @@ > > this.soapAction = soapAction; > > } > > > > - public void setStyle(int style) > > + public void setStyle(Style style) > > { > > - this.style = new Integer(style); > > + this.style = style; > > } > > > > /** > > * Return the style of the operation, defaulting to the parent > > * ServiceDesc's style if we don't have one explicitly set. > > */ > > - public int getStyle() > > + public Style getStyle() > > { > > if (style == null) { > > if (parent != null) { > > return parent.getStyle(); > > } > > - return ServiceDesc.STYLE_RPC; // Default > > + return Style.DEFAULT; // Default > > } > > > > - return style.intValue(); > > + return style; > > } > > > > public void addParameter(ParameterDesc param) > > > > > > > > 1.1 xml-axis/java/src/org/apache/axis/enum/Style.java > > > > Index: Style.java > > =================================================================== > > package org.apache.axis.enum; > > > > import javax.xml.rpc.namespace.QName; > > > > import org.apache.axis.Constants; > > import org.apache.axis.deployment.wsdd.WSDDConstants; > > > > > > > > /** > > * @author rsitze > > */ > > public class Style extends Enum { > > private static final Type type = new Type(); > > > > public static final String RPC_STR = "rpc"; > > public static final String DOCUMENT_STR = "document"; > > public static final String WRAPPED_STR = "wrapped"; > > public static final String MESSAGE_STR = "message"; > > > > public static final Style RPC = type.getStyle(RPC_STR); > > public static final Style DOCUMENT = type.getStyle(DOCUMENT_STR); > > public static final Style WRAPPED = type.getStyle(WRAPPED_STR); > > public static final Style MESSAGE = type.getStyle(MESSAGE_STR); > > > > public static final Style DEFAULT = RPC; > > > > static { type.setDefault(DEFAULT); } > > > > > > private QName provider; > > private String encoding; > > > > > > // public int getValue(); > > // public String getName(); > > // public Type getType(); > > > > > > public static Style getDefault() { return (Style)type.getDefault(); } > > > > public final QName getProvider() { return provider; } > > public final String getEncoding() { return encoding; } > > > > public static final Style getStyle(int style) { > > return type.getStyle(style); > > } > > > > public static final Style getStyle(String style) { > > return type.getStyle(style); > > } > > > > public static final Style getStyle(String style, Style dephault) { > > return type.getStyle(style, dephault); > > } > > > > public static final boolean isValid(String style) { > > return type.isValid(style); > > } > > > > public static final int size() { > > return type.size(); > > } > > > > public static final String[] getStyles() { > > return type.getEnumNames(); > > } > > > > public static class Type extends Enum.Type { > > private Type() { > > super("style", new Enum[] { > > new Style(0, RPC_STR, > > WSDDConstants.QNAME_JAVARPC_PROVIDER, > > Constants.URI_DEFAULT_SOAP_ENC), > > new Style(1, DOCUMENT_STR, > > WSDDConstants.QNAME_JAVARPC_PROVIDER, > > Constants.URI_LITERAL_ENC), > > new Style(2, WRAPPED_STR, > > WSDDConstants.QNAME_JAVARPC_PROVIDER, > > Constants.URI_LITERAL_ENC), > > new Style(3, MESSAGE_STR, > > WSDDConstants.QNAME_JAVAMSG_PROVIDER, > > Constants.URI_LITERAL_ENC) > > }); > > } > > > > public final Style getStyle(int style) { > > return (Style)this.getEnum(style); > > } > > > > public final Style getStyle(String style) { > > return (Style)this.getEnum(style); > > } > > > > public final Style getStyle(String style, Style dephault) { > > return (Style)this.getEnum(style, dephault); > > } > > > > // public final String getName(); > > // public boolean isValid(String enumName); > > // public final int size(); > > // public final String[] getEnumNames(); > > // public final Enum getEnum(int enum); > > // public final Enum getEnum(String enumName); > > // public final Enum getEnum(String enumName, Enum dephault); > > } > > > > private Style(int value, String name, QName provider, String encoding) > { > > super(type, value, name); > > this.provider = provider; > > this.encoding = encoding; > > } > > }; > > > > > > > > 1.1 xml-axis/java/src/org/apache/axis/enum/Scope.java > > > > Index: Scope.java > > =================================================================== > > package org.apache.axis.enum; > > > > import javax.xml.rpc.namespace.QName; > > > > import org.apache.axis.Constants; > > import org.apache.axis.deployment.wsdd.WSDDConstants; > > > > > > /** > > * @author rsitze > > */ > > public class Scope extends Enum { > > private static final Type type = new Type(); > > > > public static final String REQUEST_STR = "Request"; > > public static final String APPLICATION_STR = "Application"; > > public static final String SESSION_STR = "Session"; > > > > public static final Scope REQUEST = type.getScope(REQUEST_STR); > > public static final Scope APPLICATION = type.getScope(APPLICATION_STR); > > public static final Scope SESSION = type.getScope(SESSION_STR); > > > > public static final Scope DEFAULT = REQUEST; > > > > > > static { type.setDefault(DEFAULT); } > > > > > > // public int getValue(); > > // public String getName(); > > // public Type getType(); > > > > public static Scope getDefault() { return (Scope)type.getDefault(); } > > > > public static final Scope getScope(int scope) { > > return type.getScope(scope); > > } > > > > public static final Scope getScope(String scope) { > > return type.getScope(scope); > > } > > > > public static final Scope getScope(String scope, Scope dephault) { > > return type.getScope(scope, dephault); > > } > > > > public static final boolean isValid(String scope) { > > return type.isValid(scope); > > } > > > > public static final int size() { > > return type.size(); > > } > > > > public static final String[] getScopes() { > > return type.getEnumNames(); > > } > > > > public static class Type extends Enum.Type { > > private Type() { > > super("scope", new Enum[] { > > new Scope(0, REQUEST_STR), > > new Scope(1, APPLICATION_STR), > > new Scope(2, SESSION_STR) > > }); > > } > > > > public final Scope getScope(int scope) { > > return (Scope)this.getEnum(scope); > > } > > > > public final Scope getScope(String scope) { > > return (Scope)this.getEnum(scope); > > } > > > > public final Scope getScope(String scope, Scope dephault) { > > return (Scope)this.getEnum(scope, dephault); > > } > > > > // public final String getName(); > > // public boolean isValid(String enumName); > > // public final int size(); > > // public final String[] getEnumNames(); > > // public final Enum getEnum(int enum); > > // public final Enum getEnum(String enumName); > > // public final Enum getEnum(String enumName, Enum dephault); > > } > > > > private Scope(int value, String name) { > > super(type, value, name); > > } > > }; > > > > > > > > 1.1 xml-axis/java/src/org/apache/axis/enum/Enum.java > > > > Index: Enum.java > > =================================================================== > > package org.apache.axis.enum; > > > > import java.util.Vector; > > > > import org.apache.axis.utils.JavaUtils; > > import org.apache.commons.logging.Log; > > import org.apache.commons.logging.LogFactory; > > > > /** > > * General support for 'enumerated' data types. > > * Name searches are case insensitive. > > * > > * @author Richard Sitze ([EMAIL PROTECTED]) > > */ > > public abstract class Enum { > > protected static Log log = > > LogFactory.getLog(Enum.class.getName()); > > > > private final Type type; > > public final int value; > > public final String name; > > > > protected Enum(Type type, int value, String name) { > > this.type = type; > > this.value = value; > > this.name = name.intern(); > > } > > > > public final int getValue() { return value; } > > public final String getName() { return name; } > > public final Type getType() { return type; } > > > > public final boolean equals(Object obj) { > > return (obj != null && obj instanceof Enum) > > ? _equals((Enum)obj) > > : false; > > } > > > > public final boolean equals(Enum obj) { > > return (obj != null) ? _equals(obj) : false; > > } > > > > /** > > * The 'equals' logic assumes that there is a one-to-one > > * relationship between value & name. If this isn't true, > > * then expect to be confused when using this class with > > * Collections. > > */ > > private final boolean _equals(Enum obj) { > > return (//obj.name == name && // names are internalized > > obj.type == type && > > obj.value == value); > > } > > > > public static abstract class Type { > > private final String name; > > private final Enum[] enums; > > private Enum dephault = null; > > > > protected Type(String name, Enum[] enums) { > > this.name = name.intern(); > > this.enums = enums; > > } > > > > protected void setDefault(Enum dephault) { > > this.dephault = dephault; > > } > > > > public Enum getDefault() { > > return dephault; > > } > > > > public final String getName() { > > return name; > > } > > > > public final boolean isValid(String enumName) { > > for (int enum = 0; enum < enums.length; enum++) { > > if (enums[enum].getName().equalsIgnoreCase(enumName)) > > return true; > > } > > > > return false; > > } > > > > public final int size() { > > return enums.length; > > } > > > > /** > > * Returns array of names for enumerated values > > */ > > public final String[] getEnumNames() { > > String[] nms = new String[ size() ]; > > > > for (int idx = 0; idx < enums.length; idx++) > > nms[idx] = enums[idx].getName(); > > > > return nms; > > } > > > > /** > > * Returns name of enumerated value > > */ > > public final Enum getEnum(int enum) { > > return (enum >= 0 && enum < enums.length) ? enums[enum] : > null; > > } > > > > /** > > * Returns enumerated value of name > > */ > > public final Enum getEnum(String enumName) { > > Enum e = getEnum(enumName, null); > > > > if (e == null) { > > log.error(JavaUtils.getMessage("badEnum02", name, > enumName)); > > } > > > > return e; > > } > > > > /** > > * Returns enumerated value of name > > * > > * For large sets of enumerated values, a HashMap could > > * be used to retrieve. It's not clear if there is any > > * benefit for small (3 to 4) sets, as used now. > > */ > > public final Enum getEnum(String enumName, Enum dephault) { > > if (enumName != null && enumName.length() > 0) { > > for (int enum = 0; enum < enums.length; enum++) { > > Enum e = enums[enum]; > > if (e.getName().equalsIgnoreCase(enumName)) > > return e; > > } > > } > > > > return dephault; > > } > > } > > } > > > > > > > > 1.31 +2 -2 > xml-axis/java/src/org/apache/axis/encoding/SerializationContextImpl.java > > > > Index: SerializationContextImpl.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/encoding/SerializationContextImpl.j > ava,v > > retrieving revision 1.30 > > retrieving revision 1.31 > > diff -u -r1.30 -r1.31 > > --- SerializationContextImpl.java 31 May 2002 19:08:07 -0000 1.30 > > +++ SerializationContextImpl.java 11 Jun 2002 13:26:14 -0000 1.31 > > @@ -61,7 +61,7 @@ > > import org.apache.axis.MessageContext; > > import org.apache.axis.soap.SOAPConstants; > > import org.apache.axis.wsdl.symbolTable.SymbolTable; > > -import org.apache.axis.description.ServiceDesc; > > +import org.apache.axis.enum.Style; > > import org.apache.axis.handlers.soap.SOAPService; > > import org.apache.axis.attachments.Attachments; > > import org.apache.axis.client.Call; > > @@ -229,7 +229,7 @@ > > // send xsi:type, and don't do multiref in that case. > > SOAPService service = msgContext.getService(); > > if (service != null) { > > - if (service.getStyle() != ServiceDesc.STYLE_RPC) { > > + if (service.getStyle() != Style.RPC) { > > sendXSIType = false; > > doMultiRefs = false; > > } > > > > > > > > 1.41 +12 -9 > xml-axis/java/src/org/apache/axis/wsdl/toJava/Emitter.java > > > > Index: Emitter.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/toJava/Emitter.java,v > > retrieving revision 1.40 > > retrieving revision 1.41 > > diff -u -r1.40 -r1.41 > > --- Emitter.java 6 Jun 2002 13:36:19 -0000 1.40 > > +++ Emitter.java 11 Jun 2002 13:26:14 -0000 1.41 > > @@ -71,7 +71,7 @@ > > import org.apache.axis.encoding.DefaultTypeMappingImpl; > > import org.apache.axis.encoding.TypeMapping; > > > > -import org.apache.axis.providers.java.JavaProvider; > > +import org.apache.axis.enum.Scope; > > > > import org.apache.axis.utils.JavaUtils; > > > > @@ -92,6 +92,10 @@ > > * @author Steve Graham ([EMAIL PROTECTED]) > > */ > > public class Emitter extends Parser { > > +// public static final byte SCOPE_NOT_EXPLICIT = (byte)0xFF; > > +// public static final byte SCOPE_APPLICATION = > (byte)Scope.APPLICATION.getValue(); > > +// public static final byte SCOPE_REQUEST = > (byte)Scope.REQUEST.getValue(); > > +// public static final byte SCOPE_SESSION = > (byte)Scope.SESSION.getValue(); > > > > protected HashMap namespaceMap = new HashMap(); > > protected String typeMappingVersion = "1.2"; > > @@ -106,7 +110,7 @@ > > private boolean bGenerateAll = false; > > private boolean bHelperGeneration = false; > > private String packageName = null; > > - private byte scope = JavaProvider.BYTE_SCOPE_NOT_EXPLICIT; > > + private Scope scope = null; > > private GeneratedFileInfo fileInfo = new GeneratedFileInfo(); > > private HashMap delayedNamespacesMap = new HashMap(); > > private String outputDir = null; > > @@ -229,20 +233,19 @@ > > > > /** > > * Set the scope for the deploy.xml file. > > - * @param scope One of JavaProvider.BYTE_SCOPE_NOT_EXPLICIT, > > - * JavaProvider.BYTE_SCOPE_APPLICATION, > JavaProvider.BYTE_SCOPE_REQUEST, > > - * JavaProvider.BYTE_SCOPE_SESSION. Anything else is equivalent to > > - * BYTE_SCOPE_NOT_EXPLICIT and no explicit scope tag will appear in > > - * deploy.xml. > > + * @param scope One of 'null', > > + * Scope.APPLICATION, Scope.REQUEST, Scope.SESSION. > > + * Anything else is equivalent to 'null' null and no explicit > > + * scope tag will appear in deploy.xml. > > */ > > - public void setScope(byte scope) { > > + public void setScope(Scope scope) { > > this.scope = scope; > > } // setScope > > > > /** > > * Get the scope for the deploy.xml file. > > */ > > - public byte getScope() { > > + public Scope getScope() { > > return scope; > > } // getScope > > > > > > > > > > 1.48 +4 -11 > xml-axis/java/src/org/apache/axis/wsdl/toJava/JavaDeployWriter.java > > > > Index: JavaDeployWriter.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/toJava/JavaDeployWriter.java,v > > retrieving revision 1.47 > > retrieving revision 1.48 > > diff -u -r1.47 -r1.48 > > --- JavaDeployWriter.java 7 Jun 2002 12:45:08 -0000 1.47 > > +++ JavaDeployWriter.java 11 Jun 2002 13:26:14 -0000 1.48 > > @@ -73,8 +73,7 @@ > > import org.apache.axis.Constants; > > > > import org.apache.axis.deployment.wsdd.WSDDConstants; > > - > > -import org.apache.axis.providers.java.JavaProvider; > > +import org.apache.axis.enum.Scope; > > > > import org.apache.axis.utils.JavaUtils; > > > > @@ -353,15 +352,9 @@ > > pw.println(methodList.substring(1) + "\"/>"); > > } > > > > - if (emitter.getScope() == JavaProvider.BYTE_SCOPE_APPLICATION) { > > - pw.println(" <parameter name=\"scope\" > value=\"Application\"/>"); > > - } > > - else if (emitter.getScope() == JavaProvider.BYTE_SCOPE_REQUEST) { > > - pw.println(" <parameter name=\"scope\" > value=\"Request\"/>"); > > - } > > - else if (emitter.getScope() == JavaProvider.BYTE_SCOPE_SESSION) { > > - pw.println(" <parameter name=\"scope\" > value=\"Session\"/>"); > > - } > > + Scope scope = emitter.getScope(); > > + if (scope != null) > > + pw.println(" <parameter name=\"scope\" value=\"" + > scope.getName() + "\"/>"); > > } //writeDeployBinding > > > > /** > > > > > > > > 1.30 +8 -15 xml-axis/java/test/wsdl/Wsdl2javaAntTask.java > > > > Index: Wsdl2javaAntTask.java > > =================================================================== > > RCS file: /home/cvs/xml-axis/java/test/wsdl/Wsdl2javaAntTask.java,v > > retrieving revision 1.29 > > retrieving revision 1.30 > > diff -u -r1.29 -r1.30 > > --- Wsdl2javaAntTask.java 6 Jun 2002 13:35:55 -0000 1.29 > > +++ Wsdl2javaAntTask.java 11 Jun 2002 13:26:14 -0000 1.30 > > @@ -53,10 +53,9 @@ > > */ > > package test.wsdl; > > > > -import org.apache.axis.providers.java.JavaProvider; > > +import org.apache.axis.enum.Scope; > > import org.apache.axis.utils.DefaultAuthenticator; > > import org.apache.axis.wsdl.toJava.Emitter; > > -import org.apache.axis.providers.java.JavaProvider; > > > > import org.apache.tools.ant.BuildException; > > import org.apache.tools.ant.Project; > > @@ -111,21 +110,15 @@ > > // Instantiate the emitter > > Emitter emitter = new Emitter(); > > > > - if > (JavaProvider.OPTION_SCOPE_APPLICATION.equalsIgnoreCase(deployScope)) { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_APPLICATION); > > - } > > - else if > (JavaProvider.OPTION_SCOPE_REQUEST.equalsIgnoreCase(deployScope)) { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_REQUEST); > > - } > > - else if > (JavaProvider.OPTION_SCOPE_SESSION.equalsIgnoreCase(deployScope)) { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_SESSION); > > - } > > - else if ("none".equalsIgnoreCase(deployScope)) { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_NOT_EXPLICIT); > > - } > > - else { > > + Scope scope = Scope.getScope(deployScope, null); > > + if (scope != null) { > > + emitter.setScope(scope); > > + } else if ("none".equalsIgnoreCase(deployScope)) { > > + /* leave default (null, or not-explicit) */; > > + } else { > > log("Unrecognized scope: " + deployScope + ". Ignoring > it.", Project.MSG_VERBOSE); > > } > > + > > if (!namespaceMap.isEmpty()) { > > emitter.setNamespaceMap(namespaceMap); > > } > > > > > > > > 1.69 +11 -22 > xml-axis/java/src/org/apache/axis/deployment/wsdd/WSDDService.java > > > > Index: WSDDService.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/deployment/wsdd/WSDDService.java,v > > retrieving revision 1.68 > > retrieving revision 1.69 > > diff -u -r1.68 -r1.69 > > --- WSDDService.java 4 Jun 2002 20:34:03 -0000 1.68 > > +++ WSDDService.java 11 Jun 2002 13:26:14 -0000 1.69 > > @@ -70,6 +70,7 @@ > > import org.apache.axis.encoding.TypeMappingRegistryImpl; > > import org.apache.axis.encoding.ser.BaseDeserializerFactory; > > import org.apache.axis.encoding.ser.BaseSerializerFactory; > > +import org.apache.axis.enum.Style; > > import org.apache.axis.handlers.soap.SOAPService; > > import org.apache.axis.providers.java.JavaProvider; > > import org.apache.axis.utils.JavaUtils; > > @@ -102,7 +103,7 @@ > > private String descriptionURL; > > > > /** Style - document or RPC (the default) */ > > - private int style = ServiceDesc.STYLE_RPC; > > + private Style style = Style.DEFAULT; > > > > private SOAPService cachedService = null; > > > > @@ -141,16 +142,9 @@ > > > > String styleStr = e.getAttribute(ATTR_STYLE); > > if (styleStr != null && !styleStr.equals("")) { > > - style = ServiceDesc.getStyleFromString(styleStr); > > + style = Style.getStyle(styleStr, Style.DEFAULT); > > desc.setStyle(style); > > - switch (style) { > > - case ServiceDesc.STYLE_MESSAGE: > > - providerQName = WSDDConstants.QNAME_JAVAMSG_PROVIDER; > > - break; > > - default: > > - providerQName = WSDDConstants.QNAME_JAVARPC_PROVIDER; > > - break; > > - } > > + providerQName = style.getProvider(); > > } > > > > String streamStr = e.getAttribute(ATTR_STREAMING); > > @@ -217,9 +211,7 @@ > > Class cls = cl.loadClass(className); > > desc.setImplClass(cls); > > initTMR(); > > - String encStyle = (desc.getStyle() == > ServiceDesc.STYLE_RPC) ? > > - Constants.URI_DEFAULT_SOAP_ENC : ""; > > - desc.setTypeMapping(getTypeMapping(encStyle)); > > + > desc.setTypeMapping(getTypeMapping(desc.getStyle().getEncoding())); > > } catch (Exception ex) { > > } > > } > > @@ -289,7 +281,7 @@ > > /** > > * Get the service style - document or RPC > > */ > > - public int getStyle() { > > + public Style getStyle() { > > return style; > > } > > > > @@ -300,7 +292,7 @@ > > /** > > * Set the service style - document or RPC > > */ > > - public void setStyle(int style) { > > + public void setStyle(Style style) { > > this.style = style; > > } > > > > @@ -394,7 +386,7 @@ > > service.setName(getQName().getLocalPart()); > > service.setOptions(getParametersTable()); > > > > - if (style != ServiceDesc.STYLE_RPC) { > > + if (style != Style.RPC) { > > // No Multirefs/xsi:types > > service.setOption(AxisEngine.PROP_DOMULTIREFS, Boolean.FALSE); > > service.setOption(AxisEngine.PROP_SEND_XSI, Boolean.FALSE); > > @@ -435,10 +427,7 @@ > > // use the style of the service to map doc/lit or rpc/enc > > String encodingStyle = mapping.getEncodingStyle(); > > if (encodingStyle == null) { > > - if (style == ServiceDesc.STYLE_RPC) > > - encodingStyle =Constants.URI_DEFAULT_SOAP_ENC; > > - else > > - encodingStyle = Constants.URI_LITERAL_ENC; // literal > > + encodingStyle = style.getEncoding(); > > } > > TypeMapping tm = (TypeMapping) > tmr.getTypeMapping(encodingStyle); > > TypeMapping df = (TypeMapping) tmr.getDefaultTypeMapping(); > > @@ -495,9 +484,9 @@ > > attrs.addAttribute("", ATTR_PROVIDER, ATTR_PROVIDER, > > "CDATA", > context.qName2String(providerQName)); > > } > > - if (style != ServiceDesc.STYLE_RPC) { > > + if (style != Style.DEFAULT) { > > attrs.addAttribute("", ATTR_STYLE, ATTR_STYLE, > > - "CDATA", > ServiceDesc.getStringFromStyle(style)); > > + "CDATA", style.getName()); > > } > > > > context.startElement(WSDDConstants.QNAME_SERVICE, attrs); > > > > > > > > 1.35 +2 -3 > xml-axis/java/src/org/apache/axis/message/BodyBuilder.java > > > > Index: BodyBuilder.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/message/BodyBuilder.java,v > > retrieving revision 1.34 > > retrieving revision 1.35 > > diff -u -r1.34 -r1.35 > > --- BodyBuilder.java 4 Jun 2002 01:44:01 -0000 1.34 > > +++ BodyBuilder.java 11 Jun 2002 13:26:15 -0000 1.35 > > @@ -63,8 +63,8 @@ > > import org.apache.axis.Constants; > > import org.apache.axis.MessageContext; > > import org.apache.axis.description.OperationDesc; > > -import org.apache.axis.description.ServiceDesc; > > import org.apache.axis.encoding.DeserializationContext; > > +import org.apache.axis.enum.Style; > > import org.apache.commons.logging.Log; > > import org.apache.commons.logging.LogFactory; > > import org.xml.sax.Attributes; > > @@ -152,8 +152,7 @@ > > } else if (!gotRPCElement) { > > if (isRoot && > > (operations == null || > > - (operations[0].getStyle() != > > - ServiceDesc.STYLE_MESSAGE))) { > > + (operations[0].getStyle() != Style.MESSAGE))) { > > gotRPCElement = true; > > > > try { > > > > > > > > 1.62 +6 -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.61 > > retrieving revision 1.62 > > diff -u -r1.61 -r1.62 > > --- RPCElement.java 4 Jun 2002 01:44:01 -0000 1.61 > > +++ RPCElement.java 11 Jun 2002 13:26:15 -0000 1.62 > > @@ -62,6 +62,7 @@ > > import org.apache.axis.description.ServiceDesc; > > import org.apache.axis.encoding.DeserializationContext; > > import org.apache.axis.encoding.SerializationContext; > > +import org.apache.axis.enum.Style; > > import org.apache.axis.handlers.soap.SOAPService; > > import org.apache.axis.utils.JavaUtils; > > import org.apache.axis.wsdl.toJava.Utils; > > @@ -115,7 +116,7 @@ > > // IF we're doc/literal... we can't count on the element name > > // being the method name. > > elementIsFirstParam = (operations[0].getStyle() == > > - ServiceDesc.STYLE_DOCUMENT); > > + Style.DOCUMENT); > > } > > > > this.operations = operations; > > @@ -268,12 +269,10 @@ > > protected void outputImpl(SerializationContext context) throws > Exception > > { > > MessageContext msgContext = context.getMessageContext(); > > - boolean isRPC = true; > > - if (msgContext != null && > > - (msgContext.getOperationStyle() != ServiceDesc.STYLE_RPC) > && > > - (msgContext.getOperationStyle() != > ServiceDesc.STYLE_WRAPPED)) { > > - isRPC = false; > > - } > > + boolean isRPC = > > + (msgContext == null || > > + msgContext.getOperationStyle() == Style.RPC || > > + msgContext.getOperationStyle() == Style.WRAPPED); > > > > if (isRPC) { > > // Set default namespace if appropriate (to avoid prefix > mappings > > > > > > > > 1.65 +3 -2 > xml-axis/java/src/org/apache/axis/handlers/soap/SOAPService.java > > > > Index: SOAPService.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/handlers/soap/SOAPService.java,v > > retrieving revision 1.64 > > retrieving revision 1.65 > > diff -u -r1.64 -r1.65 > > --- SOAPService.java 31 May 2002 19:08:08 -0000 1.64 > > +++ SOAPService.java 11 Jun 2002 13:26:15 -0000 1.65 > > @@ -66,6 +66,7 @@ > > import org.apache.axis.encoding.TypeMappingRegistryImpl; > > import org.apache.axis.encoding.TypeMapping; > > import org.apache.axis.encoding.DefaultTypeMappingImpl; > > +import org.apache.axis.enum.Style; > > import org.apache.axis.providers.java.JavaProvider; > > import org.apache.axis.handlers.BasicHandler; > > import org.apache.axis.message.SOAPEnvelope; > > @@ -257,11 +258,11 @@ > > return true; > > } > > > > - public int getStyle() { > > + public Style getStyle() { > > return serviceDescription.getStyle(); > > } > > > > - public void setStyle(int style) { > > + public void setStyle(Style style) { > > serviceDescription.setStyle(style); > > } > > > > > > > > > > 1.59 +16 -48 > xml-axis/java/src/org/apache/axis/providers/java/JavaProvider.java > > > > Index: JavaProvider.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/providers/java/JavaProvider.java,v > > retrieving revision 1.58 > > retrieving revision 1.59 > > diff -u -r1.58 -r1.59 > > --- JavaProvider.java 9 Jun 2002 02:01:10 -0000 1.58 > > +++ JavaProvider.java 11 Jun 2002 13:26:15 -0000 1.59 > > @@ -68,6 +68,8 @@ > > import org.apache.axis.utils.cache.ClassCache; > > import org.apache.axis.wsdl.fromJava.Emitter; > > import org.apache.axis.encoding.TypeMapping; > > +import org.apache.axis.enum.Style; > > +import org.apache.axis.enum.Scope; > > import org.apache.axis.Constants; > > import org.apache.axis.handlers.soap.SOAPService; > > import org.apache.axis.deployment.wsdd.WSDDConstants; > > @@ -103,33 +105,11 @@ > > > > public static final String OPTION_SCOPE = "scope"; > > > > - // Values for the OPTION_SCOPE > > - public static final String OPTION_SCOPE_REQUEST = "Request"; > > - public static final String OPTION_SCOPE_SESSION = "Session"; > > - public static final String OPTION_SCOPE_APPLICATION = "Application"; > > - public static final String OPTION_SCOPE_DEFAULT = > OPTION_SCOPE_REQUEST; > > - > > - public static final byte BYTE_SCOPE_NOT_EXPLICIT = 0x00; > > - public static final byte BYTE_SCOPE_APPLICATION = 0x01; > > - public static final byte BYTE_SCOPE_REQUEST = 0x10; > > - public static final byte BYTE_SCOPE_SESSION = 0x11; > > - public static final byte BYTE_SCOPE_DEFAULT = BYTE_SCOPE_REQUEST; > > - > > - public static boolean isValidScope(String scope) > > - { > > - return scope == null || > > - scope.length() == 0 || > > - scope.equalsIgnoreCase(OPTION_SCOPE_REQUEST) || > > - scope.equalsIgnoreCase(OPTION_SCOPE_APPLICATION) || > > - scope.equalsIgnoreCase(OPTION_SCOPE_SESSION); > > - } > > - > > - > > /** > > * Get the service object whose method actually provides the service. > > * May look up in session table. > > */ > > - public Object getServiceObject (MessageContext msgContext, > > + private Object getServiceObject (MessageContext msgContext, > > Handler service, > > String clsName, > > IntHolder scopeHolder) > > @@ -139,23 +119,14 @@ > > > > // scope can be "Request", "Session", "Application" > > // (as with Apache SOAP) > > - String scope = (String)service.getOption(OPTION_SCOPE); > > - if (scope == null) { > > - // default is Request scope > > - scope = OPTION_SCOPE_DEFAULT; > > - } > > - > > - if (scope.equalsIgnoreCase(OPTION_SCOPE_REQUEST)) { > > - // Convey the scope upwards > > - scopeHolder.value = BYTE_SCOPE_REQUEST; > > + Scope scope = > Scope.getScope((String)service.getOption(OPTION_SCOPE), Scope.DEFAULT); > > + > > + scopeHolder.value = scope.getValue(); > > > > + if (scope == Scope.REQUEST) { > > // make a one-off > > return getNewServiceObject(msgContext, clsName); > > - > > - } else if (scope.equalsIgnoreCase(OPTION_SCOPE_SESSION)) { > > - // Convey the scope upwards > > - scopeHolder.value = BYTE_SCOPE_SESSION; > > - > > + } else if (scope == Scope.SESSION) { > > // What do we do if serviceName is null at this point??? > > if (serviceName == null) > > serviceName = msgContext.getService().toString(); > > @@ -174,13 +145,10 @@ > > } > > } else { > > // was no incoming session, sigh, treat as request scope > > - scopeHolder.value = BYTE_SCOPE_REQUEST; > > + scopeHolder.value = Scope.DEFAULT.getValue(); > > return getNewServiceObject(msgContext, clsName); > > } > > - > > - } else if (scope.equalsIgnoreCase(OPTION_SCOPE_APPLICATION)) { > > - scopeHolder.value = BYTE_SCOPE_APPLICATION; > > - > > + } else if (scope == Scope.APPLICATION) { > > // MUST be AxisEngine here! > > AxisEngine engine = msgContext.getAxisEngine(); > > if (engine.getApplicationSession() != null) { > > @@ -198,15 +166,12 @@ > > } else { > > // was no application session, sigh, treat as request > scope > > // FIXME : Should we bomb in this case? > > - scopeHolder.value = BYTE_SCOPE_REQUEST; > > + scopeHolder.value = Scope.DEFAULT.getValue(); > > return getNewServiceObject(msgContext, clsName); > > } > > - > > } else { > > - > > // NOTREACHED > > return null; > > - > > } > > } > > > > @@ -324,7 +289,7 @@ > > } finally { > > // If this is a request scoped service object which > implements > > // ServiceLifecycle, let it know that it's being destroyed > now. > > - if (scope.value == BYTE_SCOPE_REQUEST && > > + if (scope.value == Scope.REQUEST.getValue() && > > obj instanceof ServiceLifecycle) { > > ((ServiceLifecycle)obj).destroy(); > > } > > @@ -411,7 +376,10 @@ > > String alias = (String)service.getOption("alias"); > > if(alias != null) emitter.setServiceElementName(alias); > > > > - emitter.setMode(service.getStyle()); > > + emitter.setMode( (service.getStyle() == Style.RPC) > > + ? Emitter.MODE_RPC > > + : Emitter.MODE_DOCUMENT); > > + > > emitter.setClsSmart(cls,url); > > emitter.setAllowedMethods(allowedMethods); > > emitter.setIntfNamespace(interfaceNamespace); > > > > > > > > 1.50 +2 -1 > xml-axis/java/src/org/apache/axis/handlers/JWSProcessor.java > > > > Index: JWSProcessor.java > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/handlers/JWSProcessor.java,v > > retrieving revision 1.49 > > retrieving revision 1.50 > > diff -u -r1.49 -r1.50 > > --- JWSProcessor.java 6 Jun 2002 14:35:58 -0000 1.49 > > +++ JWSProcessor.java 11 Jun 2002 13:26:15 -0000 1.50 > > @@ -61,6 +61,7 @@ > > import org.apache.axis.handlers.soap.SOAPService; > > import org.apache.axis.providers.java.RPCProvider; > > import org.apache.axis.transport.http.HTTPConstants; > > +import org.apache.axis.enum.Scope; > > import org.apache.axis.utils.JWSClassLoader; > > import org.apache.axis.utils.JavaUtils; > > import org.apache.axis.utils.XMLUtils; > > @@ -282,7 +283,7 @@ > > // Take the setting for the scope option from the handler > > // parameter named "scope" > > String scope = > (String)getOption(RPCProvider.OPTION_SCOPE); > > - if (scope == null) scope = > RPCProvider.OPTION_SCOPE_DEFAULT; > > + if (scope == null) scope = Scope.DEFAULT.getName(); > > rpc.setOption(RPCProvider.OPTION_SCOPE, scope); > > > > // Set up service description > > > > > > > > 1.100 +9 -9 xml-axis/java/src/org/apache/axis/MessageContext.java > > > > Index: MessageContext.java > > =================================================================== > > RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/MessageContext.java,v > > retrieving revision 1.99 > > retrieving revision 1.100 > > diff -u -r1.99 -r1.100 > > --- MessageContext.java 10 Jun 2002 22:54:49 -0000 1.99 > > +++ MessageContext.java 11 Jun 2002 13:26:15 -0000 1.100 > > @@ -60,6 +60,7 @@ > > import org.apache.axis.description.ServiceDesc; > > import org.apache.axis.encoding.TypeMapping; > > import org.apache.axis.encoding.TypeMappingRegistry; > > +import org.apache.axis.enum.Style; > > import org.apache.axis.handlers.soap.SOAPService; > > import org.apache.axis.session.Session; > > import org.apache.axis.soap.SOAP11Constants; > > @@ -180,10 +181,10 @@ > > */ > > private String username = null; > > private String password = null; > > - private int operationStyle = ServiceDesc.STYLE_RPC; > > + private Style operationStyle = Style.RPC; > > + private String encodingStyle = operationStyle.getEncoding(); > > private boolean useSOAPAction = false; > > private String SOAPActionURI = null; > > - private String encodingStyle = Constants.URI_DEFAULT_SOAP_ENC; > > > > /** Our SOAP namespaces and such - defaults to SOAP 1.1 */ > > private SOAPConstants soapConstants = new SOAP11Constants(); > > @@ -571,8 +572,7 @@ > > TypeMappingRegistry tmr = service.getTypeMappingRegistry(); > > setTypeMappingRegistry(tmr); > > setOperationStyle(service.getStyle()); > > - setEncodingStyle((service.getStyle() == ServiceDesc.STYLE_RPC) > ? > > - Constants.URI_DEFAULT_SOAP_ENC : > ""); > > + setEncodingStyle(service.getStyle().getEncoding()); > > > > // This MessageContext should now defer properties it can't > find > > // to the Service's options. > > @@ -653,7 +653,7 @@ > > * If there, then... > > * if its a Boolean, we'll return booleanValue() > > * if its an Integer, we'll return 'false' if its '0' else 'true' > > - * if its a String, we'll return 'false' if its 'false' or '0' else > 'true' > > + * if its a String, we'll return 'false' if its 'false', 'no', or > '0' - else 'true' > > * All other types return 'true' > > */ > > public boolean isPropertyTrue(String propName, boolean defaultVal) { > > @@ -710,7 +710,7 @@ > > JavaUtils.getMessage("badProp00", new String[] { > > name, "java.lang.String", > value.getClass().getName()})); > > } > > - > setOperationStyle(ServiceDesc.getStyleFromString((String)value)); > > + setOperationStyle(Style.getStyle((String)value, > Style.DEFAULT)); > > } > > else if (name.equals(Call.SOAPACTION_USE_PROPERTY)) { > > if (!(value instanceof Boolean)) { > > @@ -787,7 +787,7 @@ > > return new Boolean(getMaintainSession()); > > } > > else if (name.equals(Call.OPERATION_STYLE_PROPERTY)) { > > - return > ServiceDesc.getStringFromStyle(getOperationStyle()); > > + return (getOperationStyle() == null) ? null : > getOperationStyle().getName(); > > } > > else if (name.equals(Call.SOAPACTION_USE_PROPERTY)) { > > return new Boolean(useSOAPAction()); > > @@ -846,14 +846,14 @@ > > /** > > * Set the operation style. > > */ > > - public void setOperationStyle(int operationStyle) { > > + public void setOperationStyle(Style operationStyle) { > > this.operationStyle = operationStyle; > > } // setOperationStyle > > > > /** > > * Get the operation style. > > */ > > - public int getOperationStyle() { > > + public Style getOperationStyle() { > > return operationStyle; > > } // getOperationStyle > > > > > > > > > > 1.36 +13 -18 xml-axis/java/src/org/apache/axis/wsdl/WSDL2Java.java > > > > Index: WSDL2Java.java > > =================================================================== > > RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/WSDL2Java.java,v > > retrieving revision 1.35 > > retrieving revision 1.36 > > diff -u -r1.35 -r1.36 > > --- WSDL2Java.java 5 Jun 2002 23:24:35 -0000 1.35 > > +++ WSDL2Java.java 11 Jun 2002 13:26:16 -0000 1.36 > > @@ -60,7 +60,7 @@ > > import org.apache.axis.utils.CLUtil; > > import org.apache.axis.utils.JavaUtils; > > > > -import org.apache.axis.providers.java.JavaProvider; > > +import org.apache.axis.enum.Scope; > > > > import org.apache.axis.wsdl.gen.Parser; > > import org.apache.axis.wsdl.gen.WSDL2; > > @@ -193,11 +193,7 @@ > > break; > > > > case SKELETON_DEPLOY_OPT: > > - String skeletonDeploy = option.getArgument(0); > > - if (skeletonDeploy.equalsIgnoreCase("true")) > > - emitter.setSkeletonWanted(true); > > - else > > - emitter.setSkeletonWanted(false); > > + > emitter.setSkeletonWanted(JavaUtils.isTrueExplicitly(option.getArgument(0))); > > // --skeletonDeploy assumes --server-side, so fall thru > > > > case SERVER_OPT: > > @@ -224,20 +220,19 @@ > > break; > > > > case SCOPE_OPT: > > - String scope = option.getArgument(); > > - if (JavaProvider.OPTION_SCOPE_APPLICATION.equals(scope)) { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_APPLICATION); > > - } > > - else if (JavaProvider.OPTION_SCOPE_REQUEST.equals(scope)) > { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_REQUEST); > > - } > > - else if (JavaProvider.OPTION_SCOPE_SESSION.equals(scope)) > { > > - emitter.setScope(JavaProvider.BYTE_SCOPE_SESSION); > > - } > > - else { > > + String arg = option.getArgument(); > > + > > + // Provide 'null' default, prevents logging internal > error. > > + // we have something different to report here. > > + Scope scope = Scope.getScope(arg, null); > > + > > + if (scope != null) { > > + emitter.setScope(scope); > > + } else { > > System.err.println( > > - JavaUtils.getMessage("badScope00", scope)); > > + JavaUtils.getMessage("badScope00", arg)); > > } > > + > > break; > > > > case TEST_OPT: > > > > > > > > 1.8 +3 -0 > xml-axis/java/src/org/apache/axis/utils/axisNLS.properties > > > > Index: axisNLS.properties > > =================================================================== > > RCS file: > /home/cvs/xml-axis/java/src/org/apache/axis/utils/axisNLS.properties,v > > retrieving revision 1.7 > > retrieving revision 1.8 > > diff -u -r1.7 -r1.8 > > --- axisNLS.properties 7 Jun 2002 12:45:08 -0000 1.7 > > +++ axisNLS.properties 11 Jun 2002 13:26:16 -0000 1.8 > > @@ -861,3 +861,6 @@ > > j2wMissingClass00=The <class-of-portType> was not specified. > > w2jDuplicateWSDLURI00=The wsdl URI has already been specified as, {0}. It > cannot be specified again as {1}. > > w2jMissingWSDLURI00=The wsdl URI was not specified. > > + > > +badEnum02=Unrecognized {0}: ''{1}'' > > +badEnum03=Unrecognized {0}: ''{1}'', defaulting to ''{2}'' > > > > > > > > 1.133 +7 -7 xml-axis/java/src/org/apache/axis/client/Call.java > > > > Index: Call.java > > =================================================================== > > RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/client/Call.java,v > > retrieving revision 1.132 > > retrieving revision 1.133 > > diff -u -r1.132 -r1.133 > > --- Call.java 10 Jun 2002 22:48:33 -0000 1.132 > > +++ Call.java 11 Jun 2002 13:26:16 -0000 1.133 > > @@ -64,7 +64,7 @@ > > import org.apache.axis.MessageContext; > > import org.apache.axis.description.OperationDesc; > > import org.apache.axis.description.ParameterDesc; > > -import org.apache.axis.description.ServiceDesc; > > +import org.apache.axis.enum.Style; > > import org.apache.axis.encoding.DeserializerFactory; > > import org.apache.axis.encoding.SerializationContext; > > import org.apache.axis.encoding.SerializationContextImpl; > > @@ -157,10 +157,10 @@ > > private String username = null; > > private String password = null; > > private boolean maintainSession = false; > > - private int operationStyle = ServiceDesc.STYLE_RPC; > > + private Style operationStyle = Style.DEFAULT; > > + private String encodingStyle = > operationStyle.getEncoding(); > > private boolean useSOAPAction = false; > > private String SOAPActionURI = null; > > - private String encodingStyle = > Constants.URI_DEFAULT_SOAP_ENC; > > private Integer timeout = null; > > > > private OperationDesc operation = new OperationDesc(); > > @@ -365,7 +365,7 @@ > > return new Boolean(getMaintainSession()); > > } > > else if (name.equals(OPERATION_STYLE_PROPERTY)) { > > - return > ServiceDesc.getStringFromStyle(getOperationStyle()); > > + return getOperationStyle().getName(); > > } > > else if (name.equals(SOAPACTION_USE_PROPERTY)) { > > return new Boolean(useSOAPAction()); > > @@ -462,7 +462,7 @@ > > * @exception IllegalArgumentException if operationStyle is not "rpc" > or "document". > > */ > > public void setOperationStyle(String operationStyle) { > > - this.operationStyle = > ServiceDesc.getStyleFromString(operationStyle); > > + this.operationStyle = Style.getStyle(operationStyle, > Style.DEFAULT); > > > > /* Not being used for now... --GD > > throw new IllegalArgumentException(JavaUtils.getMessage( > > @@ -475,7 +475,7 @@ > > /** > > * Get the operation style. > > */ > > - public int getOperationStyle() { > > + public Style getOperationStyle() { > > return operationStyle; > > } // getOperationStyle > > > > @@ -516,7 +516,7 @@ > > */ > > public void setEncodingStyle(String namespaceURI) { > > encodingStyle = (namespaceURI == null) > > - ? "" > > + ? Constants.URI_LITERAL_ENC > > : namespaceURI; > > } > > > > > > > > > > >