[jboss-cvs] jboss-seam/src/main/org/jboss/seam/contexts ...

[Gavin King]

  User: gavin   
  Date: 06/10/10 14:12:31

  Modified:    src/main/org/jboss/seam/contexts  
                        BusinessProcessContext.java Lifecycle.java
  Log:
  minor
  
  Revision  Changes    Path
  1.31      +4 -1      jboss-seam/src/main/org/jboss/seam/contexts/BusinessProcessContext.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: BusinessProcessContext.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/contexts/BusinessProcessContext.java,v
  retrieving revision 1.30
  retrieving revision 1.31
  diff -u -b -r1.30 -r1.31
  --- BusinessProcessContext.java	3 Oct 2006 23:25:02 -0000	1.30
  +++ BusinessProcessContext.java	10 Oct 2006 18:12:31 -0000	1.31
  @@ -27,7 +27,7 @@
    * @author <a href="mailto:theute at jboss.org">Thomas Heute</a>
    * @author <a href="mailto:steve at hibernate.org">Steve Ebersole</a>
    * @author Gavin King
  - * @version $Revision: 1.30 $
  + * @version $Revision: 1.31 $
    */
   public class BusinessProcessContext implements Context {
   
  @@ -96,6 +96,9 @@
      }
   
      private Set<String> getNamesFromContext() {
  +      //TODO: note that this is called from Contexts.destroy(), 
  +      //      after the Seam-managed txn was committed, but 
  +      //      this implementation requires a hit to the database!
         HashSet<String> results = new HashSet<String>();
         org.jbpm.taskmgmt.exe.TaskInstance taskInstance = getTaskInstance();
         if (taskInstance==null)
  
  
  
  1.57      +5 -1      jboss-seam/src/main/org/jboss/seam/contexts/Lifecycle.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: Lifecycle.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/contexts/Lifecycle.java,v
  retrieving revision 1.56
  retrieving revision 1.57
  diff -u -b -r1.56 -r1.57
  --- Lifecycle.java	9 Oct 2006 00:45:00 -0000	1.56
  +++ Lifecycle.java	10 Oct 2006 18:12:31 -0000	1.57
  @@ -27,7 +27,7 @@
   /**
    * @author Gavin King
    * @author <a href="mailto:theute at jboss.org">Thomas Heute</a>
  - * @version $Revision: 1.56 $
  + * @version $Revision: 1.57 $
    */
   public class Lifecycle
   {
  @@ -333,6 +333,10 @@
                     ProcessInstance.instance().hasEnded();
               if ( destroyBusinessProcessContext )
               {
  +               //TODO: note that this occurs from Lifecycle.endRequest(), after
  +               //      the Seam-managed txn was committed, but Contexts.destroy()
  +               //      calls BusinessProcessContext.getNames(), which hits the
  +               //      database!
                  log.debug("destroying business process context");
                  Contexts.destroy( Contexts.getBusinessProcessContext() );
               }