axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Russell Butek" <bu...@us.ibm.com>
Subject RE: cvs commit: xml-axis/java/src/org/apache/axis/description Ser viceDesc.java
Date Thu, 18 Apr 2002 14:20:07 GMT
That's a reasonable idea.

Russell Butek
butek@us.ibm.com


Glen Daniels <gdaniels@macromedia.com> on 04/18/2002 09:02:05 AM

Please respond to axis-dev@xml.apache.org

To:    "'axis-dev@xml.apache.org'" <axis-dev@xml.apache.org>
cc:
Subject:    RE: cvs commit: xml-axis/java/src/org/apache/axis/description
       Ser        viceDesc.java




I think I'd rather see the tests have a doSkeletons property set to "yes"
or "no", which gets used on all the tests, then allow the user to set that
in their build.properties or on the command line.  Then we can just set the
gump build to run the functional-tests twice, once with "yes" and once with
"no", in fact we could even make a "full-functional-tests" target which
does that.  What do you think?

--Glen

> -----Original Message-----
> From: Russell Butek [mailto:butek@us.ibm.com]
> Sent: Thursday, April 18, 2002 9:46 AM
> To: axis-dev@xml.apache.org
> Subject: RE: cvs commit: xml-axis/java/src/org/apache/axis/description
> Ser viceDesc.java
>
>
> Thanks, Glen, that did it!  Now I can set half the tests to
> skeletonDeploy
> as we discussed.  I'll just randomly pick the first half (+
> the types test
> that caused us so much problems).  This isn't the best
> solution, not even a
> very good one, but it'll have to do until we can spend the
> time to revamp
> it.
>
> Russell Butek
> butek@us.ibm.com
>
>
> Glen Daniels <gdaniels@macromedia.com> on 04/17/2002 11:21:31 PM
>
> Please respond to axis-dev@xml.apache.org
>
> To:    "'axis-dev@xml.apache.org'" <axis-dev@xml.apache.org>
> cc:
> Subject:    RE: cvs commit:
> xml-axis/java/src/org/apache/axis/description
>        Ser  viceDesc.java
>
>
>
>
> Russell, this should fix the problem with skeletons + test.wsdl.types.
>
> --Glen
>
> > -----Original Message-----
> > From: gdaniels@apache.org [mailto:gdaniels@apache.org]
> > Sent: Thursday, April 18, 2002 12:19 AM
> > To: xml-axis-cvs@apache.org
> > Subject: cvs commit: xml-axis/java/src/org/apache/axis/description
> > ServiceDesc.java
> >
> >
> > gdaniels    02/04/17 21:19:14
> >
> >   Modified:    java/src/org/apache/axis/description ServiceDesc.java
> >   Log:
> >   If the introspected type is a Holder, use the held class
> > when determining
> >   the type QName.
> >
> >   Revision  Changes    Path
> >   1.18      +9 -1
> > 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/Servic
> > eDesc.java,v
> >   retrieving revision 1.17
> >   retrieving revision 1.18
> >   diff -u -r1.17 -r1.18
> >   --- ServiceDesc.java      15 Apr 2002 02:35:57 -0000    1.17
> >   +++ ServiceDesc.java      18 Apr 2002 04:19:14 -0000    1.18
> >   @@ -60,6 +60,7 @@
> >    import org.apache.axis.wsdl.Skeleton;
> >
> >    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;
> >   @@ -425,8 +426,15 @@
> >                        // If no type is specified, just use
> > the Java type
> >                        QName typeQName = param.getTypeQName();
> >                        if (typeQName == null) {
> >   +                        if
> (Holder.class.isAssignableFrom(type)) {
> >   +                            typeQName = tm.getTypeQName(
> >   +
> > JavaUtils.getHolderValueType(type));
> >   +                        } else {
> >   +                            typeQName = tm.getTypeQName(type);
> >   +                        }
> >   +
> >                            param.setJavaType(type);
> >   -
> param.setTypeQName(tm.getTypeQName(type));
> >   +                        param.setTypeQName(typeQName);
> >                        } else {
> >                            // A type was specified - see if
> they match
> >                            Class paramClass = tm.getClassForQName(
> >
> >
> >
> >
>
>



Mime
View raw message