You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openwebbeans.apache.org by co...@apache.org on 2010/07/17 17:20:52 UTC
svn commit: r965095 - in
/openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common:
component/BaseEjbBean.java proxy/EjbBeanProxyHandler.java
Author: covener
Date: Sat Jul 17 15:20:52 2010
New Revision: 965095
URL: http://svn.apache.org/viewvc?rev=965095&view=rev
Log:
rename a variable and its setter.
removeStatefulInstance = true really means the stateful instance has already been removed, but
sounds too much like it is "to be" removed by the container.
Modified:
openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/component/BaseEjbBean.java
openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/proxy/EjbBeanProxyHandler.java
Modified: openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/component/BaseEjbBean.java
URL: http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/component/BaseEjbBean.java?rev=965095&r1=965094&r2=965095&view=diff
==============================================================================
--- openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/component/BaseEjbBean.java (original)
+++ openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/component/BaseEjbBean.java Sat Jul 17 15:20:52 2010
@@ -42,8 +42,8 @@ public abstract class BaseEjbBean<T> ext
/**Injected reference local interface type*/
protected Class<?> iface = null;
- /**Remove stateful bean instance*/
- protected boolean removeStatefulInstance = false;
+ /** Has the stateful instance already been removed by a business method invocation */
+ protected boolean removedStatefulInstance = false;
/**
* Creates a new instance of the session bean.
@@ -75,9 +75,9 @@ public abstract class BaseEjbBean<T> ext
* Sets remove flag.
* @param remove flag
*/
- public void setRemoveStatefulInstance(boolean remove)
+ public void setRemovedStatefulInstance(boolean remove)
{
- this.removeStatefulInstance = remove;
+ this.removedStatefulInstance = remove;
}
@@ -140,7 +140,7 @@ public abstract class BaseEjbBean<T> ext
@Override
protected void destroyComponentInstance(T instance, CreationalContext<T> creational)
{
- if(!removeStatefulInstance && getEjbType().equals(SessionBeanType.STATEFUL))
+ if(!removedStatefulInstance && getEjbType().equals(SessionBeanType.STATEFUL))
{
//Call remove method
List<Method> methods = getRemoveMethods();
Modified: openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/proxy/EjbBeanProxyHandler.java
URL: http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/proxy/EjbBeanProxyHandler.java?rev=965095&r1=965094&r2=965095&view=diff
==============================================================================
--- openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/proxy/EjbBeanProxyHandler.java (original)
+++ openwebbeans/trunk/webbeans-ejb/src/main/java/org/apache/webbeans/ejb/common/proxy/EjbBeanProxyHandler.java Sat Jul 17 15:20:52 2010
@@ -116,7 +116,7 @@ public class EjbBeanProxyHandler impleme
{
if(checkEjbRemoveMethod(method))
{
- this.ejbBean.setRemoveStatefulInstance(true);
+ this.ejbBean.setRemovedStatefulInstance(true);
}
}