craigmcc    01/01/22 19:35:53

  Modified:    src/share/org/apache/struts/digester CallMethodRule.java
                        Digester.java SetNextRule.java SetTopRule.java
  Log:
  Document an already existing restriction on "paramType" and "paramTypes"
  arguments used within the Digester package.  If the method you ultimately
  call has a parameter of a Java primitive type, you must use the
  corresponding Java wrapper class (such as "java.lang.Boolean" for a
  "boolean" parameter) instead.
  
  PR: BugRat Bug Report #692
  Submitted by: Elod Horvath <[EMAIL PROTECTED]>
  
  Revision  Changes    Path
  1.6       +7 -4      
jakarta-struts/src/share/org/apache/struts/digester/CallMethodRule.java
  
  Index: CallMethodRule.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/CallMethodRule.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- CallMethodRule.java       2001/01/07 22:39:06     1.5
  +++ CallMethodRule.java       2001/01/23 03:35:52     1.6
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/CallMethodRule.java,v 
1.5 2001/01/07 22:39:06 craigmcc Exp $
  - * $Revision: 1.5 $
  - * $Date: 2001/01/07 22:39:06 $
  + * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/CallMethodRule.java,v 
1.6 2001/01/23 03:35:52 craigmcc Exp $
  + * $Revision: 1.6 $
  + * $Date: 2001/01/23 03:35:52 $
    *
    * ====================================================================
    * 
  @@ -75,7 +75,7 @@
    * element.
    *
    * @author Craig McClanahan
  - * @version $Revision: 1.5 $ $Date: 2001/01/07 22:39:06 $
  + * @version $Revision: 1.6 $ $Date: 2001/01/23 03:35:52 $
    */
   
   public final class CallMethodRule extends Rule {
  @@ -109,6 +109,9 @@
        * @param paramCount The number of parameters to collect, or
        *  zero for a single argument from the body of ths element
        * @param paramTypes The Java class names of the arguments
  +     *  (if you wish to use a primitive type, specify the corresonding
  +     *  Java wrapper class instead, such as <code>java.lang.Boolean</code>
  +     *  for a <code>boolean</code> parameter)
        */
       public CallMethodRule(Digester digester, String methodName,
                             int paramCount, String paramTypes[]) {
  
  
  
  1.14      +13 -4     
jakarta-struts/src/share/org/apache/struts/digester/Digester.java
  
  Index: Digester.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/Digester.java,v
  retrieving revision 1.13
  retrieving revision 1.14
  diff -u -r1.13 -r1.14
  --- Digester.java     2001/01/02 23:40:18     1.13
  +++ Digester.java     2001/01/23 03:35:52     1.14
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/Digester.java,v 1.13 
2001/01/02 23:40:18 craigmcc Exp $
  - * $Revision: 1.13 $
  - * $Date: 2001/01/02 23:40:18 $
  + * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/Digester.java,v 1.14 
2001/01/23 03:35:52 craigmcc Exp $
  + * $Revision: 1.14 $
  + * $Date: 2001/01/23 03:35:52 $
    *
    * ====================================================================
    * 
  @@ -102,7 +102,7 @@
    * even from the same thread.</p>
    *
    * @author Craig McClanahan
  - * @version $Revision: 1.13 $ $Date: 2001/01/02 23:40:18 $
  + * @version $Revision: 1.14 $ $Date: 2001/01/23 03:35:52 $
    */
   
   public final class Digester extends HandlerBase {
  @@ -800,6 +800,9 @@
        *  for a single parameter from the body of this element)
        * @param paramTypes Set of Java class names for the types
        *  of the expected parameters
  +     *  (if you wish to use a primitive type, specify the corresonding
  +     *  Java wrapper class instead, such as <code>java.lang.Boolean</code>
  +     *  for a <code>boolean</code> parameter)
        */
       public void addCallMethod(String pattern, String methodName,
                              int paramCount, String paramTypes[]) {
  @@ -895,6 +898,9 @@
        * @param pattern Element matching pattern
        * @param methodName Method name to call on the parent element
        * @param paramType Java class name of the expected parameter type
  +     *  (if you wish to use a primitive type, specify the corresonding
  +     *  Java wrapper class instead, such as <code>java.lang.Boolean</code>
  +     *  for a <code>boolean</code> parameter)
        */
       public void addSetNext(String pattern, String methodName,
                           String paramType) {
  @@ -953,6 +959,9 @@
        * @param pattern Element matching pattern
        * @param methodName Method name to call on the parent element
        * @param paramType Java class name of the expected parameter type
  +     *  (if you wish to use a primitive type, specify the corresonding
  +     *  Java wrapper class instead, such as <code>java.lang.Boolean</code>
  +     *  for a <code>boolean</code> parameter)
        */
       public void addSetTop(String pattern, String methodName,
                          String paramType) {
  
  
  
  1.3       +7 -4      
jakarta-struts/src/share/org/apache/struts/digester/SetNextRule.java
  
  Index: SetNextRule.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/SetNextRule.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SetNextRule.java  2000/08/01 20:03:29     1.2
  +++ SetNextRule.java  2001/01/23 03:35:52     1.3
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/SetNextRule.java,v 1.2 
2000/08/01 20:03:29 craigmcc Exp $
  - * $Revision: 1.2 $
  - * $Date: 2000/08/01 20:03:29 $
  + * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/SetNextRule.java,v 1.3 
2001/01/23 03:35:52 craigmcc Exp $
  + * $Revision: 1.3 $
  + * $Date: 2001/01/23 03:35:52 $
    *
    * ====================================================================
    * 
  @@ -73,7 +73,7 @@
    * commonly used to establish parent-child relationships.
    *
    * @author Craig McClanahan
  - * @version $Revision: 1.2 $ $Date: 2000/08/01 20:03:29 $
  + * @version $Revision: 1.3 $ $Date: 2001/01/23 03:35:52 $
    */
   
   public final class SetNextRule extends Rule {
  @@ -103,6 +103,9 @@
        * @param digester The associated Digester
        * @param methodName Method name of the parent method to call
        * @param paramType Java class of the parent method's argument
  +     *  (if you wish to use a primitive type, specify the corresonding
  +     *  Java wrapper class instead, such as <code>java.lang.Boolean</code>
  +     *  for a <code>boolean</code> parameter)
        */
       public SetNextRule(Digester digester, String methodName,
                          String paramType) {
  
  
  
  1.3       +7 -4      
jakarta-struts/src/share/org/apache/struts/digester/SetTopRule.java
  
  Index: SetTopRule.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/SetTopRule.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SetTopRule.java   2000/08/01 20:03:29     1.2
  +++ SetTopRule.java   2001/01/23 03:35:52     1.3
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/SetTopRule.java,v 1.2 
2000/08/01 20:03:29 craigmcc Exp $
  - * $Revision: 1.2 $
  - * $Date: 2000/08/01 20:03:29 $
  + * $Header: 
/home/cvs/jakarta-struts/src/share/org/apache/struts/digester/SetTopRule.java,v 1.3 
2001/01/23 03:35:52 craigmcc Exp $
  + * $Revision: 1.3 $
  + * $Date: 2001/01/23 03:35:52 $
    *
    * ====================================================================
    * 
  @@ -72,7 +72,7 @@
    * object, passing the (top-1) (child) object as an argument.
    *
    * @author Craig McClanahan
  - * @version $Revision: 1.2 $ $Date: 2000/08/01 20:03:29 $
  + * @version $Revision: 1.3 $ $Date: 2001/01/23 03:35:52 $
    */
   
   public final class SetTopRule extends Rule {
  @@ -102,6 +102,9 @@
        * @param digester The associated Digester
        * @param methodName Method name of the parent method to call
        * @param paramType Java class of the parent method's argument
  +     *  (if you wish to use a primitive type, specify the corresonding
  +     *  Java wrapper class instead, such as <code>java.lang.Boolean</code>
  +     *  for a <code>boolean</code> parameter)
        */
       public SetTopRule(Digester digester, String methodName,
                            String argument) {
  
  
  

Reply via email to