[jBoss-Dev] CVS update: jboss/src/main/org/jboss/ejb/pluginsStatefulSessionInstanceCache.java

[jBoss CVS Development]

  User: salborini
  Date: 00/11/13 10:56:02

  Modified:    src/main/org/jboss/ejb/plugins
                        StatefulSessionInstanceCache.java
  Log:
  A bean can be passivated if the tx is not null but with status NO_TRANSACTION
  
  Revision  Changes    Path
  1.2       +15 -2     
jboss/src/main/org/jboss/ejb/plugins/StatefulSessionInstanceCache.java
  
  Index: StatefulSessionInstanceCache.java
  ===================================================================
  RCS file: 
/products/cvs/ejboss/jboss/src/main/org/jboss/ejb/plugins/StatefulSessionInstanceCache.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- StatefulSessionInstanceCache.java 2000/10/16 23:29:56     1.1
  +++ StatefulSessionInstanceCache.java 2000/11/13 18:56:02     1.2
  @@ -9,6 +9,10 @@
   
   import java.rmi.RemoteException;
   import java.rmi.NoSuchObjectException;
  +
  +import javax.transaction.Status;
  +import javax.transaction.SystemException;
  +
   import org.jboss.ejb.Container;
   import org.jboss.ejb.StatefulSessionContainer;
   import org.jboss.ejb.EnterpriseContext;
  @@ -18,7 +22,8 @@
    * Cache for stateful session beans. 
    *      
    * @author Simone Bordet ([EMAIL PROTECTED])
  - * @version $Revision: 1.1 $
  + * @author <a href="mailto:[EMAIL PROTECTED]">Sebastien Alborini</a>
  + * @version $Revision: 1.2 $
    */
   public class StatefulSessionInstanceCache 
        extends EnterpriseInstanceCache
  @@ -79,7 +84,15 @@
                {
                        if (ctx.getTransaction() != null) 
                        {
  -                             return false;
  +            try 
  +            {
  +               return (ctx.getTransaction().getStatus() == 
Status.STATUS_NO_TRANSACTION);
  +            }
  +            catch (SystemException e) 
  +            {
  +               // SA FIXME: not sure what to do here 
  +               return false;
  +            }
                        }
                }
                return true;