You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by ge...@apache.org on 2011/06/16 11:05:54 UTC
svn commit: r1136331 -
/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
Author: genspring
Date: Thu Jun 16 09:05:54 2011
New Revision: 1136331
URL: http://svn.apache.org/viewvc?rev=1136331&view=rev
Log:
OPENEJB-1361
The logic to set managed bean to local bean is simply wrong because it'll never get executed if the managed bean has interfaces.
Modified:
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
Modified: openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java?rev=1136331&r1=1136330&r2=1136331&view=diff
==============================================================================
--- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java (original)
+++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/AnnotationDeployer.java Thu Jun 16 09:05:54 2011
@@ -2399,7 +2399,7 @@ public class AnnotationDeployer implemen
&& all.remote.isEmpty()
) {
- if (interfaces.size() == 0 || beanClass.isAnnotationPresent(ManagedBean.class)) {
+ if (interfaces.size() == 0) {
// No interfaces? Then @LocalBean
sessionBean.setLocalBean(new Empty());
@@ -2416,6 +2416,12 @@ public class AnnotationDeployer implemen
return;
}
}
+
+ //alway set Local View for ManagedBean
+ if (beanClass.isAnnotationPresent(ManagedBean.class)){
+ sessionBean.setLocalBean(new Empty());
+ }
+
/**
* Track any interfaces we didn't use