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

Gavin King gavin.king at jboss.com
Tue Oct 3 19:15:58 EDT 2006


  User: gavin   
  Date: 06/10/03 19:15:58

  Modified:    src/main/org/jboss/seam/core    BusinessProcess.java
                        ProcessInstance.java TaskInstance.java
  Log:
  JBSEAM-78, destroy method call for business process context
  
  Revision  Changes    Path
  1.4       +13 -5     jboss-seam/src/main/org/jboss/seam/core/BusinessProcess.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: BusinessProcess.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/core/BusinessProcess.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -b -r1.3 -r1.4
  --- BusinessProcess.java	3 Oct 2006 22:21:39 -0000	1.3
  +++ BusinessProcess.java	3 Oct 2006 23:15:58 -0000	1.4
  @@ -40,6 +40,16 @@
         return (BusinessProcess) Component.getInstance(BusinessProcess.class, ScopeType.CONVERSATION, true);
      }
   
  +   public boolean hasCurrentProcess()
  +   {
  +      return processId!=null;
  +   }
  +
  +   public boolean hasCurrentTask()
  +   {
  +      return taskId!=null;
  +   }
  +
      public Long getProcessId() {
         return processId;
      }
  @@ -114,7 +124,7 @@
            task.end(transitionName);
         }
         
  -      setTaskId(null);
  +      setTaskId(null); //TODO: do I really need this???!
         
         Events.instance().raiseEvent("org.jboss.seam.endTask." + task.getTask().getName());
   
  @@ -181,14 +191,12 @@
   
      public boolean checkTask()
      {
  -      TaskInstance task = org.jboss.seam.core.TaskInstance.instance();
  -      Long taskId = getTaskId();
  -      if ( task==null )
  +      if ( !hasCurrentTask() )
         {
            taskNotFound(taskId);
            return false;
         }
  -      else if ( task.hasEnded() )
  +      else if ( org.jboss.seam.core.TaskInstance.instance().hasEnded() )
         {
            taskEnded(taskId);
            return false;
  
  
  
  1.11      +2 -2      jboss-seam/src/main/org/jboss/seam/core/ProcessInstance.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: ProcessInstance.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/core/ProcessInstance.java,v
  retrieving revision 1.10
  retrieving revision 1.11
  diff -u -b -r1.10 -r1.11
  --- ProcessInstance.java	28 Jun 2006 17:32:13 -0000	1.10
  +++ ProcessInstance.java	3 Oct 2006 23:15:58 -0000	1.11
  @@ -20,7 +20,7 @@
    * jBPM ProcessInstance.
    * 
    * @author Gavin King
  - * @version $Revision: 1.10 $
  + * @version $Revision: 1.11 $
    */
   @Scope(ScopeType.APPLICATION)
   @Name("processInstance")
  @@ -49,7 +49,7 @@
      
      public static org.jbpm.graph.exe.ProcessInstance instance()
      {
  -      if ( !Contexts.isConversationContextActive() || BusinessProcess.instance().getProcessId()==null ) return null; //so we don't start a txn
  +      if ( !Contexts.isConversationContextActive() || !BusinessProcess.instance().hasCurrentProcess() ) return null; //so we don't start a txn
         
         if ( !Contexts.isApplicationContextActive() )
         {
  
  
  
  1.10      +2 -2      jboss-seam/src/main/org/jboss/seam/core/TaskInstance.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: TaskInstance.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/main/org/jboss/seam/core/TaskInstance.java,v
  retrieving revision 1.9
  retrieving revision 1.10
  diff -u -b -r1.9 -r1.10
  --- TaskInstance.java	28 Jun 2006 17:32:13 -0000	1.9
  +++ TaskInstance.java	3 Oct 2006 23:15:58 -0000	1.10
  @@ -20,7 +20,7 @@
    * jBPM TaskInstance.
    * 
    * @author Gavin King
  - * @version $Revision: 1.9 $
  + * @version $Revision: 1.10 $
    */
   @Scope(ScopeType.APPLICATION)
   @Name("taskInstance")
  @@ -49,7 +49,7 @@
      
      public static org.jbpm.taskmgmt.exe.TaskInstance instance()
      {
  -      if ( BusinessProcess.instance().getTaskId()==null ) return null; //so we don't start a txn
  +      if ( !Contexts.isConversationContextActive() || !BusinessProcess.instance().hasCurrentTask() ) return null; //so we don't start a txn
         
         if ( !Contexts.isApplicationContextActive() )
         {
  
  
  



More information about the jboss-cvs-commits mailing list