You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by si...@apache.org on 2011/02/19 17:54:55 UTC

svn commit: r1072387 - /commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/CallMethodBuilderImpl.java

Author: simonetripodi
Date: Sat Feb 19 16:54:55 2011
New Revision: 1072387

URL: http://svn.apache.org/viewvc?rev=1072387&view=rev
Log:
don't reset method argument types if already set in the proper way when using element bosy as argument

Modified:
    commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/CallMethodBuilderImpl.java

Modified: commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/CallMethodBuilderImpl.java
URL: http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/CallMethodBuilderImpl.java?rev=1072387&r1=1072386&r2=1072387&view=diff
==============================================================================
--- commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/CallMethodBuilderImpl.java (original)
+++ commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/internal/rulesbinder/CallMethodBuilderImpl.java Sat Feb 19 16:54:55 2011
@@ -112,8 +112,11 @@ final class CallMethodBuilderImpl
         }
 
         this.paramCount = paramCount;
+
         if (this.paramCount == 0) {
-            this.paramTypes = new Class<?>[] { String.class };
+            if (this.paramTypes == null || this.paramTypes.length != 1) {
+                this.paramTypes = new Class<?>[] { String.class };
+            }
         } else {
             this.paramTypes = new Class<?>[this.paramCount];
             for (int i = 0; i < paramTypes.length; i++) {