[dna-commits] DNA SVN: r137 - trunk/dna-repository/src/main/java/org/jboss/dna/repository/util.

[dna-commits at lists.jboss.org]

Author: rhauch
Date: 2008-05-09 18:59:25 -0400 (Fri, 09 May 2008)
New Revision: 137

Modified:
   trunk/dna-repository/src/main/java/org/jboss/dna/repository/util/SimpleExecutionContext.java
Log:
Changed the SimpleExecutionContext to not extend SimpleSessionFactory but to instead delegate to a SessionFactory supplied in the constructor.

Modified: trunk/dna-repository/src/main/java/org/jboss/dna/repository/util/SimpleExecutionContext.java
===================================================================
--- trunk/dna-repository/src/main/java/org/jboss/dna/repository/util/SimpleExecutionContext.java	2008-05-09 22:58:33 UTC (rev 136)
+++ trunk/dna-repository/src/main/java/org/jboss/dna/repository/util/SimpleExecutionContext.java	2008-05-09 22:59:25 UTC (rev 137)
@@ -21,22 +21,26 @@
  */
 package org.jboss.dna.repository.util;
 
+import org.jboss.dna.common.util.ArgCheck;
+
 /**
  * @author Randall Hauch
  */
-public class SimpleExecutionContext extends SimpleSessionFactory implements ExecutionContext {
+public class SimpleExecutionContext implements ExecutionContext {
 
     private final JcrTools tools = new JcrTools();
+    private final SessionFactory sessionFactory;
 
-    public SimpleExecutionContext() {
-        super();
+    public SimpleExecutionContext( SessionFactory sessionFactory ) {
+        ArgCheck.isNotNull(sessionFactory, "session factory");
+        this.sessionFactory = sessionFactory;
     }
 
     /**
      * {@inheritDoc}
      */
     public SessionFactory getSessionFactory() {
-        return this;
+        return this.sessionFactory;
     }
 
     /**