[openejb-scm] openejb/modules/core/src/java/org/openejb EJBInvocation.java EJBInvocationImpl.java

[dblevins]

dblevins    2005/06/21 17:16:57

  Modified:    modules/core/src/java/org/openejb EJBInvocation.java
                        EJBInvocationImpl.java
  Log:

  Removing 47 references to the SimpleInvocationResult constructor and
  replacing with a factory method on EJBInvocatin interface.  The
  implementor of EJBInvocation should be able to decide the
  implementation of InvocationResult.
  
  Revision  Changes    Path
  1.3       +6 -1      
openejb/modules/core/src/java/org/openejb/EJBInvocation.java
  
  Index: EJBInvocation.java
  ===================================================================
  RCS file: 
/scm/openejb/openejb/modules/core/src/java/org/openejb/EJBInvocation.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- EJBInvocation.java        18 Jul 2004 22:32:19 -0000      1.2
  +++ EJBInvocation.java        21 Jun 2005 21:16:57 -0000      1.3
  @@ -48,6 +48,7 @@
   package org.openejb;
   
   import org.apache.geronimo.core.service.Invocation;
  +import org.apache.geronimo.core.service.InvocationResult;
   
   import org.apache.geronimo.transaction.context.TransactionContext;
   
  @@ -116,4 +117,8 @@
        * @param transactionContext the transaction context to use
        */
       void setTransactionContext(TransactionContext transactionContext);
  +
  +    InvocationResult createResult(Object object);
  +
  +    InvocationResult createExceptionResult(Exception exception);
   }
  
  
  
  1.7       +11 -1     
openejb/modules/core/src/java/org/openejb/EJBInvocationImpl.java
  
  Index: EJBInvocationImpl.java
  ===================================================================
  RCS file: 
/scm/openejb/openejb/modules/core/src/java/org/openejb/EJBInvocationImpl.java,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- EJBInvocationImpl.java    12 Feb 2005 00:48:19 -0000      1.6
  +++ EJBInvocationImpl.java    21 Jun 2005 21:16:57 -0000      1.7
  @@ -52,6 +52,8 @@
   import java.io.ObjectOutput;
   
   import org.apache.geronimo.core.service.SimpleInvocation;
  +import org.apache.geronimo.core.service.InvocationResult;
  +import org.apache.geronimo.core.service.SimpleInvocationResult;
   
   import org.apache.geronimo.transaction.context.TransactionContext;
   
  @@ -137,6 +139,14 @@
   
       public void setTransactionContext(TransactionContext transactionContext) 
{
           this.transactionContext = transactionContext;
  +    }
  +
  +    public InvocationResult createResult(Object object) {
  +        return new SimpleInvocationResult(true, object);
  +    }
  +
  +    public InvocationResult createExceptionResult(Exception exception) {
  +        return new SimpleInvocationResult(false, exception);
       }
   
       public void writeExternal(ObjectOutput out) throws IOException {