Author: rmannibucau
Date: Thu Oct 11 15:29:04 2012
New Revision: 1397108

URL: http://svn.apache.org/viewvc?rev=1397108&view=rev
Log:
OPENEJB-1919 array type comparison not working on some JVM

Modified:
    
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/MethodInfoUtil.java

Modified: 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/MethodInfoUtil.java
URL: 
http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/MethodInfoUtil.java?rev=1397108&r1=1397107&r2=1397108&view=diff
==============================================================================
--- 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/MethodInfoUtil.java
 (original)
+++ 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/assembler/classic/MethodInfoUtil.java
 Thu Oct 11 15:29:04 2012
@@ -528,7 +528,7 @@ public class MethodInfoUtil {
         for (int i = 0; i < parameterTypes.length; i++) {
             Class<?> parameterType = parameterTypes[i];
             String methodParam = methodParams.get(i);
-            if (!methodParam.equals(getName(parameterType))) {
+            if (!methodParam.equals(getName(parameterType)) && 
!methodParam.equals(parameterType.getName())) {
                 return false;
             }
         }
@@ -538,7 +538,7 @@ public class MethodInfoUtil {
 
     private static String getName(Class<?> type) {
         if (type.isArray()) {
-            return getName(type.getComponentType()) + "[]";
+            return getName(type.getComponentType()) + "[]"; // depend on JVM? 
type.getName() seems to work on Oracle one
         } else {
             return type.getName();
         }


Reply via email to