You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by rm...@apache.org on 2013/07/01 13:34:13 UTC

svn commit: r1498388 - /geronimo/specs/trunk/geronimo-el_2.2_spec/src/main/java/javax/el/BeanELResolver.java

Author: rmannibucau
Date: Mon Jul  1 11:34:13 2013
New Revision: 1498388

URL: http://svn.apache.org/r1498388
Log:
GERONIMO-6476 using param values instead of indexes when calling an EL method with varags

Modified:
    geronimo/specs/trunk/geronimo-el_2.2_spec/src/main/java/javax/el/BeanELResolver.java

Modified: geronimo/specs/trunk/geronimo-el_2.2_spec/src/main/java/javax/el/BeanELResolver.java
URL: http://svn.apache.org/viewvc/geronimo/specs/trunk/geronimo-el_2.2_spec/src/main/java/javax/el/BeanELResolver.java?rev=1498388&r1=1498387&r2=1498388&view=diff
==============================================================================
--- geronimo/specs/trunk/geronimo-el_2.2_spec/src/main/java/javax/el/BeanELResolver.java (original)
+++ geronimo/specs/trunk/geronimo-el_2.2_spec/src/main/java/javax/el/BeanELResolver.java Mon Jul  1 11:34:13 2013
@@ -444,7 +444,7 @@ public class BeanELResolver extends ELRe
                         Object targetArray = Array.newInstance(varArgsClassType.getComponentType(), params.length - iCurrentIndex);
                         Class<?> componentClassType = varArgsClassType.getComponentType();
                         for (int i = 0, iLoopSize = params.length - iCurrentIndex; i < iLoopSize; i++) {
-                            Array.set(targetArray, i, expressionFactory.coerceToType(iCurrentIndex + i, componentClassType));
+                            Array.set(targetArray, i, expressionFactory.coerceToType(params[iCurrentIndex + i], componentClassType));
                         }
                         finalParamValues[iCurrentIndex] = targetArray;
                     }