Well I updated allowed methods to the best of my knowledge.  I left the
summary section on how Java Message Provider offers methods blank since
that's where Glen D. is the guru.   I wanted to refernce the RPC Hander
documentation but I guess that hasn't made it in yet and also the WSDD
operation element too.  When I see that commited I'll create some links.

Given this documentatin requirement on me for my submitted change, I guess
were getting serious about it.    Glad to see it. It will great to look in
the Axis refrence manual to see how things works typemappings, WSDD
operation, parameter,  all those factories,  and all the possible way to
the Axis engine may be configured, WSDL2java options limitations, JWS
class,  enable logging, NLS enabling, developing custom serializers, etc.
Attachments too :-) Guess as good commiters if we see functionality not
documented we need -1 any release.  It's way too much to expect users to
sift through the code to find these gold nuggets.

Rick Rineholt
"The truth is out there...  All you need is a better search engine!"

[EMAIL PROTECTED]



Tom Jordahl <[EMAIL PROTECTED]> on 09/26/2002 05:20:05 PM

Please respond to [EMAIL PROTECTED]

To:    "'[EMAIL PROTECTED]'" <[EMAIL PROTECTED]>
cc:
Subject:    RE: SUBMIT REQUEST FOR INCLUSION TO R 1.0: cvs commit:
       xml-axis/j ava/src/org/apache/axis/description OperationDesc.java
       ServiceDesc.java



+1 on the condition that some documentation about how allowMethods works
get written up and submitted/merged along with the change.

-1 with no docs.

--
Tom Jordahl
Macromedia Server Development



-----Original Message-----
From: Rick Rineholt [mailto:[EMAIL PROTECTED]]
Sent: Thursday, September 26, 2002 5:05 PM
To: [EMAIL PROTECTED]
Subject: SUBMIT REQUEST FOR INCLUSION TO R 1.0: cvs commit:
xml-axis/java/src/org/apache/axis/description OperationDesc.java
ServiceDesc.java


If there are no more objections to this addition, I'd like to request VOTE
it to be merged in release 1.0

Thanks.

Rick Rineholt
"The truth is out there...  All you need is a better search engine!"

[EMAIL PROTECTED]

---------------------- Forwarded by Rick Rineholt/Raleigh/IBM on 09/26/2002
04:59 PM ---------------------------

[EMAIL PROTECTED] on 09/26/2002 12:35:05 AM

Please respond to [EMAIL PROTECTED]

To:    [EMAIL PROTECTED]
cc:
Subject:    cvs commit: xml-axis/java/src/org/apache/axis/description
       OperationDesc.java ServiceDesc.java



rineholt    2002/09/25 21:35:05

  Modified:    java/src/org/apache/axis/description OperationDesc.java
                        ServiceDesc.java
  Log:
  By default and if "*" is specified accept all methods found that conform.

  Revision  Changes    Path
  1.23      +2 -0
  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.22
  retrieving revision 1.23
  diff -u -r1.22 -r1.23
  --- OperationDesc.java      25 Sep 2002 03:55:16 -0000    1.22
  +++ OperationDesc.java      26 Sep 2002 04:35:05 -0000    1.23
  @@ -85,6 +85,8 @@
       // public Document method(Document)
       public static final int MSG_METHOD_DOCUMENT = 4;

  +    public static final int MSG_METHOD_NONCONFORMING = -4;
  +
       protected static Log log =
           LogFactory.getLog(OperationDesc.class.getName());




  1.62      +6 -2
  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.61
  retrieving revision 1.62
  diff -u -r1.61 -r1.62
  --- ServiceDesc.java  25 Sep 2002 16:54:24 -0000    1.61
  +++ ServiceDesc.java  26 Sep 2002 04:35:05 -0000    1.62
  @@ -102,6 +102,7 @@

       /** The name of this service */
       private String name = null;
  +    private static final String ALL_METHODS= "*";

       /** List of allowed methods */
       /** null allows everything, an empty ArrayList allows nothing */
  @@ -578,6 +579,7 @@

                   if (style == Style.MESSAGE) {
                       int messageOperType = checkMessageMethod(method);
  +                    if(messageOperType ==
                       OperationDesc.MSG_METHOD_NONCONFORMING) continue;
                       if (messageOperType == -1) {
                           throw new InternalException("Couldn't match
                       method to any of the allowable message-style
                       patterns!");
                       }
  @@ -701,9 +703,10 @@
                   return OperationDesc.MSG_METHOD_SOAPENVELOPE;
               }
           }
  -
  -        throw new InternalException (Messages.getMessage
  ("badMsgMethodParams",
  +        if( null != allowedMethods && !allowedMethods.isEmpty() )
  +          throw new InternalException (Messages.getMessage
  ("badMsgMethodParams",
                                                            method.getName
  ()));
  +        return    OperationDesc.MSG_METHOD_NONCONFORMING;
       }

       /**
  @@ -1020,6 +1023,7 @@
           // appropriately.
           if (style == Style.MESSAGE) {
               int messageOperType = checkMessageMethod(method);
  +            if(messageOperType ==
  OperationDesc.MSG_METHOD_NONCONFORMING) return;
               if (messageOperType == -1) {
                   throw new InternalException("Couldn't match method to
                   any of the allowable message-style patterns!");
               }










Reply via email to