RE: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava C lassRep.java

[Glen Daniels <gd...@macromedia.com>]

This change was to fix the WSDL generation for the echo sample ("http://localhost:8080/axis/services/http://soapinterop.org/?wsdl").

We now avoid an exception, but still produce bad WSDL because we're trying to treat the HashMap argument to echoMap() as a java bean.

I think this points to a feature we want to add to the WSDL generator - the ability to specify arbitrary pieces of schema to use for particular types instead of always generating it from the Java classes.  In the Map case, we implement ser/deser to a specified schema, but the schema is not derivable from the Map class.

--Glen

> -----Original Message-----
> From: gdaniels@apache.org [mailto:gdaniels@apache.org]
> Sent: Thursday, January 10, 2002 5:29 PM
> To: xml-axis-cvs@apache.org
> Subject: cvs commit: xml-axis/java/src/org/apache/axis/wsdl/fromJava
> ClassRep.java
> 
> 
> gdaniels    02/01/10 14:28:30
> 
>   Modified:    java/src/org/apache/axis/wsdl/fromJava ClassRep.java
>   Log:
>   A little bulletproofing.
>   
>   Revision  Changes    Path
>   1.6       +3 -0      
> xml-axis/java/src/org/apache/axis/wsdl/fromJava/ClassRep.java
>   
>   Index: ClassRep.java
>   ===================================================================
>   RCS file: 
> /home/cvs/xml-axis/java/src/org/apache/axis/wsdl/fromJava/Clas
> sRep.java,v
>   retrieving revision 1.5
>   retrieving revision 1.6
>   diff -u -r1.5 -r1.6
>   --- ClassRep.java	7 Jan 2002 20:39:55 -0000	1.5
>   +++ ClassRep.java	10 Jan 2002 22:28:30 -0000	1.6
>   @@ -388,6 +388,9 @@
>         * @return true if the Property has JavaBean style accessors
>         */
>        protected boolean isJavaBeanNormal(Class cls, String 
> name, Class type) {
>   +        if ((name == null) || (name.length() == 0))
>   +            return false;
>   +        
>            try {
>                String propName = name.substring(0,1).toUpperCase()
>                    + name.substring(1);
>   
>   
>   
>