cvs commit: xml-axis-wsif/java/src/org/apache/wsif/providers/soap/apacheaxis WSIFOperation_ApacheAxis.java

[antelder]

antelder    2003/01/13 12:02:25

  Modified:    java/src/org/apache/wsif/providers/soap/apacheaxis
                        WSIFOperation_ApacheAxis.java
  Log:
  Fix soap parts in mime binding defect
  
  Revision  Changes    Path
  1.56      +5 -5      
xml-axis-wsif/java/src/org/apache/wsif/providers/soap/apacheaxis/WSIFOperation_ApacheAxis.java
  
  Index: WSIFOperation_ApacheAxis.java
  ===================================================================
  RCS file: 
/home/cvs/xml-axis-wsif/java/src/org/apache/wsif/providers/soap/apacheaxis/WSIFOperation_ApacheAxis.java,v
  retrieving revision 1.55
  retrieving revision 1.56
  diff -u -r1.55 -r1.56
  --- WSIFOperation_ApacheAxis.java     7 Jan 2003 17:37:38 -0000       1.55
  +++ WSIFOperation_ApacheAxis.java     13 Jan 2003 20:02:25 -0000      1.56
  @@ -589,9 +589,9 @@
   
       /**
        * Gets the parts from a WSDL message.
  -     * If the partNames list is not null only the parts named
  -     * in the partName list are return, otherwise all parts
  -     * are returned.
  +     * If the partNames list (from WSDL soap:body parts=) is 
  +     * not null only the parts named in the partName list are
  +     * returned, otherwise all parts are returned.
        */
       private ArrayList getParts(javax.wsdl.Message m, List partNames ) throws 
WSIFException{
           ArrayList al = new ArrayList();
  @@ -607,7 +607,7 @@
                        "' not defined in message " + m);
           }
           
  -        if (partNames == null || partNames.size() < 1) {
  +        if (partNames == null) {
               if (parts != null) {
                    al.addAll(parts);
               }
  @@ -621,7 +621,7 @@
                                "' in soap:body parts not in message" + m);
                        } 
                        // as there can be multiple mime:content elements which
  -                     // specify a coice of types (which wsif ignores for now)
  +                     // specify a coice of types (TODO: which wsif ignores for now)
                        // we only want each mime part once
                        if (!al.contains(p)) { 
                            al.add(p);