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

Posted to commits@tomee.apache.org by rm...@apache.org on 2012/09/20 09:45:08 UTC

svn commit: r1387896 - /openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/proxy/LocalBeanProxyFactory.java

Author: rmannibucau
Date: Thu Sep 20 07:45:08 2012
New Revision: 1387896

URL: http://svn.apache.org/viewvc?rev=1387896&view=rev
Log:
cdi beans protected methods can be called but not ejb ones (localbeans) - see LocalBeanProxyFactoryTest.testNonPublicMethods

Modified:
    openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/proxy/LocalBeanProxyFactory.java

Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/proxy/LocalBeanProxyFactory.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/proxy/LocalBeanProxyFactory.java?rev=1387896&r1=1387895&r2=1387896&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/proxy/LocalBeanProxyFactory.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/proxy/LocalBeanProxyFactory.java Thu Sep 20 07:45:08 2012
@@ -179,7 +179,7 @@ public class LocalBeanProxyFactory imple
             for (final Method method : entry.getValue()) {
                 final String name = method.getName();
 
-                if (!Modifier.isPrivate(method.getModifiers())
+                if (Modifier.isPublic(method.getModifiers())
                         || (method.getParameterTypes().length == 0 && ("finalize".equals(name)
                         || "clone".equals(name)))) {
                     // forward invocations of any public methods or