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) {