isis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danhayw...@apache.org
Subject svn commit: r1234456 [14/17] - in /incubator/isis/trunk/framework/runtimes: dflt/bytecode/dflt/src/main/java/org/apache/isis/runtimes/dflt/bytecode/dflt/classsubstitutor/ dflt/bytecode/dflt/src/main/java/org/apache/isis/runtimes/dflt/bytecode/dflt/obje...
Date Sun, 22 Jan 2012 07:36:26 GMT
Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/IsisSystemFixturesHookAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/IsisSystemFixturesHookAbstract.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/IsisSystemFixturesHookAbstract.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/IsisSystemFixturesHookAbstract.java Sun Jan 22 07:36:07 2012
@@ -44,7 +44,8 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.log4j.Logger;
 
 /**
- * An implementation of {@link IsisSystem} that has a hook for installing fixtures but does not install them itself.
+ * An implementation of {@link IsisSystem} that has a hook for installing
+ * fixtures but does not install them itself.
  */
 public abstract class IsisSystemFixturesHookAbstract implements IsisSystem {
 
@@ -70,8 +71,7 @@ public abstract class IsisSystemFixtures
         this(deploymentType, new IsisLocaleInitializer(), new IsisTimeZoneInitializer());
     }
 
-    public IsisSystemFixturesHookAbstract(final DeploymentType deploymentType,
-        final IsisLocaleInitializer localeInitializer, final IsisTimeZoneInitializer timeZoneInitializer) {
+    public IsisSystemFixturesHookAbstract(final DeploymentType deploymentType, final IsisLocaleInitializer localeInitializer, final IsisTimeZoneInitializer timeZoneInitializer) {
         this.deploymentType = deploymentType;
         this.localeInitializer = localeInitializer;
         this.timeZoneInitializer = timeZoneInitializer;
@@ -115,7 +115,8 @@ public abstract class IsisSystemFixtures
             sessionFactory = doCreateSessionFactory(deploymentType);
 
             // temporarily make a configuration available
-            // REVIEW: would rather inject this, or perhaps even the ConfigurationBuilder
+            // REVIEW: would rather inject this, or perhaps even the
+            // ConfigurationBuilder
             IsisContext.setConfiguration(getConfiguration());
 
             initContext(sessionFactory);
@@ -147,15 +148,16 @@ public abstract class IsisSystemFixtures
     // ///////////////////////////////////////////
 
     /**
-     * Hook method; the returned implementation is expected to use the same general approach as the subclass itself.
+     * Hook method; the returned implementation is expected to use the same
+     * general approach as the subclass itself.
      * 
      * <p>
-     * So, for example, <tt>IsisSystemUsingInstallers</tt> uses the {@link InstallerLookup} mechanism to find its
-     * components. The corresponding <tt>ExecutionContextFactoryUsingInstallers</tt> object returned by this method
-     * should use {@link InstallerLookup} likewise.
+     * So, for example, <tt>IsisSystemUsingInstallers</tt> uses the
+     * {@link InstallerLookup} mechanism to find its components. The
+     * corresponding <tt>ExecutionContextFactoryUsingInstallers</tt> object
+     * returned by this method should use {@link InstallerLookup} likewise.
      */
-    protected abstract IsisSessionFactory doCreateSessionFactory(final DeploymentType deploymentType)
-        throws IsisSystemException;
+    protected abstract IsisSessionFactory doCreateSessionFactory(final DeploymentType deploymentType) throws IsisSystemException;
 
     // ///////////////////////////////////////////
     // Configuration
@@ -181,7 +183,8 @@ public abstract class IsisSystemFixtures
     // ///////////////////////////////////////////
 
     /**
-     * Just returns a {@link TemplateImageLoaderAwt}; subclasses may override if required.
+     * Just returns a {@link TemplateImageLoaderAwt}; subclasses may override if
+     * required.
      */
     protected TemplateImageLoader obtainTemplateImageLoader() {
         return new TemplateImageLoaderAwt(getConfiguration());
@@ -197,15 +200,15 @@ public abstract class IsisSystemFixtures
     // PersistenceSessionFactory
     // ///////////////////////////////////////////
 
-    protected abstract PersistenceSessionFactory obtainPersistenceSessionFactory(DeploymentType deploymentType)
-        throws IsisSystemException;
+    protected abstract PersistenceSessionFactory obtainPersistenceSessionFactory(DeploymentType deploymentType) throws IsisSystemException;
 
     // ///////////////////////////////////////////
     // Fixtures (hooks)
     // ///////////////////////////////////////////
 
     /**
-     * Optional hook for appending debug information pertaining to fixtures installer if required.
+     * Optional hook for appending debug information pertaining to fixtures
+     * installer if required.
      */
     protected void appendFixturesInstallerDebug(final DebugBuilder debug) {
     }
@@ -214,9 +217,10 @@ public abstract class IsisSystemFixtures
      * The {@link LogonFixture}, if any, obtained by running fixtures.
      * 
      * <p>
-     * Intended to be used when for {@link DeploymentType#EXPLORATION exploration} (instead of an
-     * {@link ExplorationSession}) or {@link DeploymentType#PROTOTYPE prototype} deployments (saves logging in). Should
-     * be <i>ignored</i> in other {@link DeploymentType}s.
+     * Intended to be used when for {@link DeploymentType#EXPLORATION
+     * exploration} (instead of an {@link ExplorationSession}) or
+     * {@link DeploymentType#PROTOTYPE prototype} deployments (saves logging
+     * in). Should be <i>ignored</i> in other {@link DeploymentType}s.
      * 
      * <p>
      * This implementation always returns <tt>null</tt>.
@@ -239,8 +243,7 @@ public abstract class IsisSystemFixtures
     // Authentication Manager
     // ///////////////////////////////////////////
 
-    protected abstract AuthenticationManager obtainAuthenticationManager(DeploymentType deploymentType)
-        throws IsisSystemException;
+    protected abstract AuthenticationManager obtainAuthenticationManager(DeploymentType deploymentType) throws IsisSystemException;
 
     // ///////////////////////////////////////////
     // UserProfileLoader
@@ -260,8 +263,7 @@ public abstract class IsisSystemFixtures
 
     private void showSplash(final TemplateImageLoader imageLoader) {
 
-        final boolean vetoSplashFromConfig =
-            getConfiguration().getBoolean(SystemConstants.NOSPLASH_KEY, SystemConstants.NOSPLASH_DEFAULT);
+        final boolean vetoSplashFromConfig = getConfiguration().getBoolean(SystemConstants.NOSPLASH_KEY, SystemConstants.NOSPLASH_DEFAULT);
         if (!vetoSplashFromConfig && getDeploymentType().shouldShowSplash()) {
             splashWindow = new SplashWindow(imageLoader);
         }
@@ -299,10 +301,13 @@ public abstract class IsisSystemFixtures
         if (selectionName.equals("Configuration")) {
             return getConfiguration();
         } /*
-           * else if (selectionName.equals("Overview")) { debugOverview(debug); } else if
-           * (selectionName.equals("Authenticator")) { deb = IsisContext.getAuthenticationManager(); } else if
-           * (selectionName.equals("Reflector")) { deb = IsisContext.getSpecificationLoader(); } else if
-           * (selectionName.equals("Contexts")) { deb = debugListContexts(debug); } else { deb =
+           * else if (selectionName.equals("Overview")) { debugOverview(debug);
+           * } else if (selectionName.equals("Authenticator")) { deb =
+           * IsisContext.getAuthenticationManager(); } else if
+           * (selectionName.equals("Reflector")) { deb =
+           * IsisContext.getSpecificationLoader(); } else if
+           * (selectionName.equals("Contexts")) { deb =
+           * debugListContexts(debug); } else { deb =
            * debugDisplayContext(selectionName, debug); }
            */
         return null;
@@ -330,8 +335,7 @@ public abstract class IsisSystemFixtures
 
     @Override
     public String[] debugSectionNames() {
-        final String[] general =
-            new String[] { "Overview", "Authenticator", "Configuration", "Reflector", "Requests", "Contexts" };
+        final String[] general = new String[] { "Overview", "Authenticator", "Configuration", "Reflector", "Requests", "Contexts" };
         final String[] contextIds = IsisContext.getInstance().allSessionIds();
         final String[] combined = new String[general.length + contextIds.length];
         System.arraycopy(general, 0, combined, 0, general.length);
@@ -351,9 +355,7 @@ public abstract class IsisSystemFixtures
             }
 
             final String user = System.getProperty("user.name");
-            final String system =
-                System.getProperty("os.name") + " (" + System.getProperty("os.arch") + ") "
-                    + System.getProperty("os.version");
+            final String system = System.getProperty("os.name") + " (" + System.getProperty("os.arch") + ") " + System.getProperty("os.version");
             final String java = System.getProperty("java.vm.name") + " " + System.getProperty("java.vm.version");
             debug.appendln("user: " + user);
             debug.appendln("os: " + system);

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/SystemConstants.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/SystemConstants.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/SystemConstants.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/SystemConstants.java Sun Jan 22 07:36:07 2012
@@ -42,7 +42,8 @@ import org.apache.isis.runtimes.dflt.run
 public final class SystemConstants {
 
     /**
-     * Key used to lookup {@link DeploymentType} (eg via command line) in {@link IsisConfiguration}.
+     * Key used to lookup {@link DeploymentType} (eg via command line) in
+     * {@link IsisConfiguration}.
      * 
      * <p>
      * Use {@link DeploymentType#valueOf(String)} to decode.
@@ -50,7 +51,8 @@ public final class SystemConstants {
     public static final String DEPLOYMENT_TYPE_KEY = ConfigurationConstants.ROOT + "deploymentType";
 
     /**
-     * Key used to lookup {@link ObjectReflector reflector} in {@link IsisConfiguration}, and root for any
+     * Key used to lookup {@link ObjectReflector reflector} in
+     * {@link IsisConfiguration}, and root for any
      * {@link ObjectReflectorInstaller reflector}-specific configuration keys.
      */
     public final static String REFLECTOR_KEY = ConfigurationConstants.ROOT + ObjectReflectorInstaller.TYPE;
@@ -60,8 +62,9 @@ public final class SystemConstants {
     public static final String REFLECTOR_DEFAULT = "java";
 
     /**
-     * Key used to lookup {@link IsisViewer viewer} in {@link IsisConfiguration}, and root for any
-     * {@link IsisViewerInstaller viewer}-specific configuration keys.
+     * Key used to lookup {@link IsisViewer viewer} in {@link IsisConfiguration}
+     * , and root for any {@link IsisViewerInstaller viewer}-specific
+     * configuration keys.
      */
     public final static String VIEWER_KEY = ConfigurationConstants.ROOT + IsisViewerInstaller.TYPE;
     /**
@@ -70,11 +73,12 @@ public final class SystemConstants {
     public static final String VIEWER_DEFAULT = "dnd";
 
     /**
-     * Key used to lookup {@link PersistenceSessionFactory persistor} in {@link IsisConfiguration}, and root for any
-     * {@link PersistenceMechanismInstaller persistor}-specific configuration keys.
+     * Key used to lookup {@link PersistenceSessionFactory persistor} in
+     * {@link IsisConfiguration}, and root for any
+     * {@link PersistenceMechanismInstaller persistor}-specific configuration
+     * keys.
      */
-    public final static String OBJECT_PERSISTOR_INSTALLER_KEY = ConfigurationConstants.ROOT
-        + PersistenceMechanismInstaller.TYPE;
+    public final static String OBJECT_PERSISTOR_INSTALLER_KEY = ConfigurationConstants.ROOT + PersistenceMechanismInstaller.TYPE;
 
     // TODO: inline
     public static final String OBJECT_PERSISTOR_KEY = OBJECT_PERSISTOR_INSTALLER_KEY;
@@ -84,11 +88,12 @@ public final class SystemConstants {
     public static final String OBJECT_PERSISTOR_PRODUCTION_DEFAULT = "xml";
 
     /**
-     * Key used to lookup {@link UserProfileStore user profile store} (via command line) in {@link IsisConfiguration},
-     * and root for any {@link UserProfileStoreInstaller user profile store} -specific configuration keys.
+     * Key used to lookup {@link UserProfileStore user profile store} (via
+     * command line) in {@link IsisConfiguration}, and root for any
+     * {@link UserProfileStoreInstaller user profile store} -specific
+     * configuration keys.
      */
-    public final static String PROFILE_PERSISTOR_INSTALLER_KEY = ConfigurationConstants.ROOT
-        + UserProfileStoreInstaller.TYPE;
+    public final static String PROFILE_PERSISTOR_INSTALLER_KEY = ConfigurationConstants.ROOT + UserProfileStoreInstaller.TYPE;
 
     // TODO: inline
     public static final String USER_PROFILE_STORE_KEY = PROFILE_PERSISTOR_INSTALLER_KEY;
@@ -98,11 +103,12 @@ public final class SystemConstants {
     public static final String USER_PROFILE_STORE_PRODUCTION_DEFAULT = "xml";
 
     /**
-     * Key used to lookup {@link AuthenticationManager authentication manager} in {@link IsisConfiguration}, and root
-     * for any {@link AuthenticationManagerInstaller authentication manager}-specific configuration keys.
+     * Key used to lookup {@link AuthenticationManager authentication manager}
+     * in {@link IsisConfiguration}, and root for any
+     * {@link AuthenticationManagerInstaller authentication manager}-specific
+     * configuration keys.
      */
-    public static final String AUTHENTICATION_INSTALLER_KEY = ConfigurationConstants.ROOT
-        + AuthenticationManagerInstaller.TYPE;
+    public static final String AUTHENTICATION_INSTALLER_KEY = ConfigurationConstants.ROOT + AuthenticationManagerInstaller.TYPE;
     /**
      * Default for {@link #AUTHENTICATION_INSTALLER_KEY} if not exploring.
      */
@@ -113,11 +119,12 @@ public final class SystemConstants {
     public static final String AUTHENTICATION_EXPLORATION_DEFAULT = "noop";
 
     /**
-     * Key used to lookup {@link AuthorizationManager authorization manager} in {@link IsisConfiguration}, and root for
-     * any {@link AuthorizationManagerInstaller authorization manager}-specific configuration keys.
+     * Key used to lookup {@link AuthorizationManager authorization manager} in
+     * {@link IsisConfiguration}, and root for any
+     * {@link AuthorizationManagerInstaller authorization manager}-specific
+     * configuration keys.
      */
-    public static final String AUTHORIZATION_INSTALLER_KEY = ConfigurationConstants.ROOT
-        + AuthorizationManagerInstaller.TYPE;
+    public static final String AUTHORIZATION_INSTALLER_KEY = ConfigurationConstants.ROOT + AuthorizationManagerInstaller.TYPE;
     /**
      * Default for {@link #AUTHORIZATION_DEFAULT} if production.
      */
@@ -129,8 +136,9 @@ public final class SystemConstants {
     public static final String AUTHORIZATION_NON_PRODUCTION_DEFAULT = "noop";
 
     /**
-     * Key used to lookup {@link ServicesInstaller services installer} in {@link IsisConfiguration}, and root for any
-     * {@link ServicesInstaller services installer}-specific configuration keys.
+     * Key used to lookup {@link ServicesInstaller services installer} in
+     * {@link IsisConfiguration}, and root for any {@link ServicesInstaller
+     * services installer}-specific configuration keys.
      */
     public static final String SERVICES_INSTALLER_KEY = ConfigurationConstants.ROOT + ServicesInstaller.TYPE;
     /**
@@ -139,8 +147,9 @@ public final class SystemConstants {
     public static final String SERVICES_INSTALLER_DEFAULT = "configuration";
 
     /**
-     * Key used to lookup {@link FixturesInstaller fixtures installer} in {@link IsisConfiguration}, and root for any
-     * {@link FixturesInstaller fixtures installer}-specific configuration keys.
+     * Key used to lookup {@link FixturesInstaller fixtures installer} in
+     * {@link IsisConfiguration}, and root for any {@link FixturesInstaller
+     * fixtures installer}-specific configuration keys.
      */
     public static final String FIXTURES_INSTALLER_KEY = ConfigurationConstants.ROOT + FixturesInstaller.TYPE;
     /**
@@ -149,8 +158,10 @@ public final class SystemConstants {
     public static final String FIXTURES_INSTALLER_DEFAULT = "configuration";
 
     /**
-     * Key used to lookup {@link TemplateImageLoaderInstaller template image loader} in {@link IsisConfiguration}, and
-     * root for any {@link TemplateImageLoaderInstaller template image loader}-specific configuration keys.
+     * Key used to lookup {@link TemplateImageLoaderInstaller template image
+     * loader} in {@link IsisConfiguration}, and root for any
+     * {@link TemplateImageLoaderInstaller template image loader}-specific
+     * configuration keys.
      */
     public static final String IMAGE_LOADER_KEY = ConfigurationConstants.ROOT + TemplateImageLoaderInstaller.TYPE;
     /**
@@ -159,8 +170,10 @@ public final class SystemConstants {
     public static final String IMAGE_LOADER_DEFAULT = "awt";
 
     /**
-     * Key used to lookup {@link ClientConnectionInstaller client connection installer} in {@link IsisConfiguration},
-     * and root for any {@link ClientConnectionInstaller client connection installer}-specific configuration keys.
+     * Key used to lookup {@link ClientConnectionInstaller client connection
+     * installer} in {@link IsisConfiguration}, and root for any
+     * {@link ClientConnectionInstaller client connection installer}-specific
+     * configuration keys.
      */
     public static final String CLIENT_CONNECTION_KEY = ConfigurationConstants.ROOT + ClientConnectionInstaller.TYPE;
     /**
@@ -169,8 +182,10 @@ public final class SystemConstants {
     public static final String CLIENT_CONNECTION_DEFAULT = "encoding_socket";
 
     /**
-     * Key used to lookup {@link EmbeddedWebServerInstaller embedded web installer} in {@link IsisConfiguration}, and
-     * root for any {@link EmbeddedWebServerInstaller embedded web server installer}-specific configuration keys.
+     * Key used to lookup {@link EmbeddedWebServerInstaller embedded web
+     * installer} in {@link IsisConfiguration}, and root for any
+     * {@link EmbeddedWebServerInstaller embedded web server installer}-specific
+     * configuration keys.
      */
     public static final String WEBSERVER_KEY = ConfigurationConstants.ROOT + EmbeddedWebServerInstaller.TYPE;
     /**
@@ -179,22 +194,26 @@ public final class SystemConstants {
     public static final String WEBSERVER_DEFAULT = "jetty";
 
     /**
-     * Key by which requested fixture (eg via command line) is made available in {@link IsisConfiguration}.
+     * Key by which requested fixture (eg via command line) is made available in
+     * {@link IsisConfiguration}.
      */
     public final static String FIXTURE_KEY = ConfigurationConstants.ROOT + "fixture";
 
     /**
-     * Key by which requested user (eg via command line) is made available in {@link IsisConfiguration} .
+     * Key by which requested user (eg via command line) is made available in
+     * {@link IsisConfiguration} .
      */
     public final static String USER_KEY = ConfigurationConstants.ROOT + "user";
 
     /**
-     * Key by which requested password (eg via command line) is made available in {@link IsisConfiguration}.
+     * Key by which requested password (eg via command line) is made available
+     * in {@link IsisConfiguration}.
      */
     public final static String PASSWORD_KEY = ConfigurationConstants.ROOT + "password";
 
     /**
-     * Key as to whether to show splash (eg via command line) is made available in {@link IsisConfiguration}.
+     * Key as to whether to show splash (eg via command line) is made available
+     * in {@link IsisConfiguration}.
      * 
      * <p>
      * Use {@link Splash#valueOf(String)} to decode.

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContext.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContext.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContext.java Sun Jan 22 07:36:07 2012
@@ -47,9 +47,11 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.log4j.Logger;
 
 /**
- * Provides singleton <i>access to</i> the current (session scoped) {@link IsisSession}, along with convenience methods
- * to obtain application-scoped components and also any transaction-scoped components {@link TransactionScopedComponent}
- * s if a {@link IsisTransaction} {@link IsisSession#getCurrentTransaction() is in progress}.
+ * Provides singleton <i>access to</i> the current (session scoped)
+ * {@link IsisSession}, along with convenience methods to obtain
+ * application-scoped components and also any transaction-scoped components
+ * {@link TransactionScopedComponent} s if a {@link IsisTransaction}
+ * {@link IsisSession#getCurrentTransaction() is in progress}.
  * 
  * <p>
  * Somewhat analogous to (the static methods in) <tt>HibernateUtil</tt>.
@@ -105,7 +107,8 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Whether the {@link IsisContext#getInstance() singleton} itself may be replaced.
+     * Whether the {@link IsisContext#getInstance() singleton} itself may be
+     * replaced.
      */
     protected static enum ContextReplacePolicy {
         NOT_REPLACEABLE, REPLACEABLE
@@ -115,11 +118,10 @@ public abstract class IsisContext implem
      * Creates a new instance of the {@link IsisSession} holder.
      * 
      * <p>
-     * Will throw an exception if an instance has already been created and is not
-     * {@link ContextReplacePolicy#REPLACEABLE}.
+     * Will throw an exception if an instance has already been created and is
+     * not {@link ContextReplacePolicy#REPLACEABLE}.
      */
-    protected IsisContext(final ContextReplacePolicy replacePolicy, final SessionClosePolicy sessionClosePolicy,
-        final IsisSessionFactory sessionFactory) {
+    protected IsisContext(final ContextReplacePolicy replacePolicy, final SessionClosePolicy sessionClosePolicy, final IsisSessionFactory sessionFactory) {
         if (singleton != null && !singleton.isContextReplaceable()) {
             throw new IsisException("Isis Context already set up and cannot be replaced");
         }
@@ -152,7 +154,8 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Whether any open session can be automatically {@link #closeSessionInstance() close}d on
+     * Whether any open session can be automatically
+     * {@link #closeSessionInstance() close}d on
      * {@link #openSessionInstance(AuthenticationSession) open}.
      */
     public final boolean isSessionAutocloseable() {
@@ -160,7 +163,8 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Helper method for subclasses' implementation of {@link #openSessionInstance(AuthenticationSession)}.
+     * Helper method for subclasses' implementation of
+     * {@link #openSessionInstance(AuthenticationSession)}.
      */
     protected void applySessionClosePolicy() {
         if (getSessionInstance() == null) {
@@ -191,10 +195,12 @@ public abstract class IsisContext implem
      * Ignored if already closed.
      * 
      * <p>
-     * This method is <i>not</i> marked <tt>final</tt> so it can be overridden if necessarily. Generally speaking this
-     * shouldn't be necessary; one case where it might though is if an implementation has multiple concurrent uses of a
-     * session, in which case "closing" the session really means just deregistering the usage of it by a particular
-     * thread; only when all threads have finished with a session can it really be closed.
+     * This method is <i>not</i> marked <tt>final</tt> so it can be overridden
+     * if necessarily. Generally speaking this shouldn't be necessary; one case
+     * where it might though is if an implementation has multiple concurrent
+     * uses of a session, in which case "closing" the session really means just
+     * deregistering the usage of it by a particular thread; only when all
+     * threads have finished with a session can it really be closed.
      */
     public void closeSessionInstance() {
         if (getSessionInstance() != null) {
@@ -204,12 +210,12 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Overridable hook method called from {@link #closeSessionInstance()}, allowing subclasses to clean up (for example
-     * datastructures).
+     * Overridable hook method called from {@link #closeSessionInstance()},
+     * allowing subclasses to clean up (for example datastructures).
      * 
      * <p>
-     * The {@link #getSessionInstance() current} {@link IsisSession} will already have been {@link IsisSession#close()
-     * closed}.
+     * The {@link #getSessionInstance() current} {@link IsisSession} will
+     * already have been {@link IsisSession#close() closed}.
      */
     protected void doClose() {
     }
@@ -227,8 +233,8 @@ public abstract class IsisContext implem
      * Locates the current {@link IsisSession}.
      * 
      * <p>
-     * This might just be a singleton (eg {@link IsisContextStatic}), or could be retrieved from the thread (eg
-     * {@link IsisContextThreadLocal}).
+     * This might just be a singleton (eg {@link IsisContextStatic}), or could
+     * be retrieved from the thread (eg {@link IsisContextThreadLocal}).
      */
     public abstract IsisSession getSessionInstance();
 
@@ -268,7 +274,8 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Convenience method to return {@link IsisSession} for specified {@link IsisSession#getId()}.
+     * Convenience method to return {@link IsisSession} for specified
+     * {@link IsisSession#getId()}.
      * 
      * <p>
      * Provided primarily for debugging.
@@ -295,7 +302,8 @@ public abstract class IsisContext implem
     // ///////////////////////////////////////////////////////////
 
     /**
-     * Convenience method returning the {@link IsisSessionFactory} of the current {@link #getSession() session}.
+     * Convenience method returning the {@link IsisSessionFactory} of the
+     * current {@link #getSession() session}.
      */
     public static IsisSessionFactory getSessionFactory() {
         return getInstance().getSessionFactoryInstance();
@@ -393,8 +401,8 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Convenience method to return the {@link #getSession() current} {@link IsisSession}'s {@link IsisSession#getId()
-     * id}.
+     * Convenience method to return the {@link #getSession() current}
+     * {@link IsisSession}'s {@link IsisSession#getId() id}.
      * 
      * @see IsisSession#getId()
      */
@@ -456,11 +464,12 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Convenience method, returning the current {@link IsisTransaction transaction} (if any).
+     * Convenience method, returning the current {@link IsisTransaction
+     * transaction} (if any).
      * 
      * <p>
-     * Transactions are managed using the {@link IsisTransactionManager} obtainable from the {@link IsisSession's}
-     * {@link PersistenceSession}.
+     * Transactions are managed using the {@link IsisTransactionManager}
+     * obtainable from the {@link IsisSession's} {@link PersistenceSession}.
      * 
      * @see IsisSession#getCurrentTransaction()
      * @see PersistenceSession#getTransactionManager()
@@ -470,16 +479,16 @@ public abstract class IsisContext implem
     }
 
     /**
-     * Convenience method, returning the {@link MessageBroker} of the {@link #getCurrentTransaction() current
-     * transaction}.
+     * Convenience method, returning the {@link MessageBroker} of the
+     * {@link #getCurrentTransaction() current transaction}.
      */
     public static MessageBroker getMessageBroker() {
         return getCurrentTransaction().getMessageBroker();
     }
 
     /**
-     * Convenience method, returning the {@link UpdateNotifier} of the {@link #getCurrentTransaction() current
-     * transaction}.
+     * Convenience method, returning the {@link UpdateNotifier} of the
+     * {@link #getCurrentTransaction() current transaction}.
      */
     public static UpdateNotifier getUpdateNotifier() {
         return getCurrentTransaction().getUpdateNotifier();

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextMultiUser.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextMultiUser.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextMultiUser.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextMultiUser.java Sun Jan 22 07:36:07 2012
@@ -23,7 +23,8 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.isis.runtimes.dflt.runtime.system.session.IsisSessionFactory;
 
 /**
- * Provides <i>access to</i> the current {@link IsisSession} in a multi-user deployment.
+ * Provides <i>access to</i> the current {@link IsisSession} in a multi-user
+ * deployment.
  */
 public abstract class IsisContextMultiUser extends IsisContext {
 
@@ -35,8 +36,7 @@ public abstract class IsisContextMultiUs
         this(ContextReplacePolicy.NOT_REPLACEABLE, SessionClosePolicy.EXPLICIT_CLOSE, sessionFactory);
     }
 
-    protected IsisContextMultiUser(final ContextReplacePolicy contextReplacePolicy,
-        final SessionClosePolicy sessionClosePolicy, final IsisSessionFactory sessionFactory) {
+    protected IsisContextMultiUser(final ContextReplacePolicy contextReplacePolicy, final SessionClosePolicy sessionClosePolicy, final IsisSessionFactory sessionFactory) {
         super(contextReplacePolicy, sessionClosePolicy, sessionFactory);
     }
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextPipe.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextPipe.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextPipe.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextPipe.java Sun Jan 22 07:36:07 2012
@@ -25,9 +25,10 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.isis.runtimes.dflt.runtime.system.session.IsisSessionFactory;
 
 /**
- * A specialised IsisContext implementation that provides two sets of components: one for the server; and one for the
- * client. This simply determines the current thread and if that thread is the server thread then it provides server
- * data. For any other thread the client data is used.
+ * A specialised IsisContext implementation that provides two sets of
+ * components: one for the server; and one for the client. This simply
+ * determines the current thread and if that thread is the server thread then it
+ * provides server data. For any other thread the client data is used.
  */
 public class IsisContextPipe extends IsisContextMultiUser {
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextStatic.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextStatic.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextStatic.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextStatic.java Sun Jan 22 07:36:07 2012
@@ -27,8 +27,9 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.isis.runtimes.dflt.runtime.system.session.IsisSessionFactory;
 
 /**
- * Provides <i>access to</i> the current {@link IsisSession} in a single-user {@link DeploymentType deployment} (and
- * thus implemented as a <tt>static</tt> singleton).
+ * Provides <i>access to</i> the current {@link IsisSession} in a single-user
+ * {@link DeploymentType deployment} (and thus implemented as a <tt>static</tt>
+ * singleton).
  */
 public class IsisContextStatic extends IsisContext {
 
@@ -39,19 +40,18 @@ public class IsisContextStatic extends I
     // //////////////////////////////////////////////
 
     public static IsisContext createInstance(final IsisSessionFactory sessionFactory) {
-        return new IsisContextStatic(ContextReplacePolicy.NOT_REPLACEABLE, SessionClosePolicy.EXPLICIT_CLOSE,
-            sessionFactory);
+        return new IsisContextStatic(ContextReplacePolicy.NOT_REPLACEABLE, SessionClosePolicy.EXPLICIT_CLOSE, sessionFactory);
     }
 
     /**
-     * Intended for testing; the singleton can be replaced and sessions are autoclosed.
+     * Intended for testing; the singleton can be replaced and sessions are
+     * autoclosed.
      */
     public static IsisContext createRelaxedInstance(final IsisSessionFactory sessionFactory) {
         return new IsisContextStatic(ContextReplacePolicy.REPLACEABLE, SessionClosePolicy.AUTO_CLOSE, sessionFactory);
     }
 
-    protected IsisContextStatic(final ContextReplacePolicy replacePolicy, final SessionClosePolicy sessionClosePolicy,
-        final IsisSessionFactory sessionFactory) {
+    protected IsisContextStatic(final ContextReplacePolicy replacePolicy, final SessionClosePolicy sessionClosePolicy, final IsisSessionFactory sessionFactory) {
         super(replacePolicy, sessionClosePolicy, sessionFactory);
     }
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextThreadLocal.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextThreadLocal.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextThreadLocal.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/context/IsisContextThreadLocal.java Sun Jan 22 07:36:07 2012
@@ -29,7 +29,8 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.log4j.Logger;
 
 /**
- * Basic multi-user implementation of Isis that stores a set of components for each thread in use.
+ * Basic multi-user implementation of Isis that stores a set of components for
+ * each thread in use.
  */
 public class IsisContextThreadLocal extends IsisContextMultiUser {
 
@@ -121,10 +122,12 @@ public class IsisContextThreadLocal exte
     // /////////////////////////////////////////////////////////
 
     /**
-     * Is only intended to be called through {@link IsisContext#openSession(AuthenticationSession)}.
+     * Is only intended to be called through
+     * {@link IsisContext#openSession(AuthenticationSession)}.
      * 
      * <p>
-     * Implementation note: an alternative design would have just been to bind onto a thread local.
+     * Implementation note: an alternative design would have just been to bind
+     * onto a thread local.
      */
     @Override
     public IsisSession openSessionInstance(final AuthenticationSession authenticationSession) {
@@ -132,8 +135,7 @@ public class IsisContextThreadLocal exte
         synchronized (sessionsByThread) {
             applySessionClosePolicy();
             final IsisSession session = getSessionFactoryInstance().openSession(authenticationSession);
-            LOG.debug("  opening session " + session + " (count " + sessionsByThread.size() + ") for "
-                + authenticationSession.getUserName());
+            LOG.debug("  opening session " + session + " (count " + sessionsByThread.size() + ") for " + authenticationSession.getUserName());
             saveSession(thread, session);
             session.open();
             return session;
@@ -143,8 +145,7 @@ public class IsisContextThreadLocal exte
     protected IsisSession createAndOpenSession(final Thread thread, final AuthenticationSession authenticationSession) {
         final IsisSession session = getSessionFactoryInstance().openSession(authenticationSession);
         session.open();
-        LOG.info("  opening session " + session + " (count " + sessionsByThread.size() + ") for "
-            + authenticationSession.getUserName());
+        LOG.info("  opening session " + session + " (count " + sessionsByThread.size() + ") for " + authenticationSession.getUserName());
         return session;
     }
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/internal/SplashWindow.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/internal/SplashWindow.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/internal/SplashWindow.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/internal/SplashWindow.java Sun Jan 22 07:36:07 2012
@@ -152,7 +152,8 @@ public class SplashWindow extends Window
 
         if (logo != null) {
             g.drawImage(logo, PADDING, PADDING, this);
-            // g.drawRect(PADDING, PADDING, logo.getWidth(this) - 1, logo.getHeight(this) - 1);
+            // g.drawRect(PADDING, PADDING, logo.getWidth(this) - 1,
+            // logo.getHeight(this) - 1);
         } else {
             g.setFont(logoFont);
             final FontMetrics fm = g.getFontMetrics();
@@ -195,7 +196,8 @@ public class SplashWindow extends Window
     }
 
     /**
-     * leaves the screen up for the specified period (in seconds) and then removes it.
+     * leaves the screen up for the specified period (in seconds) and then
+     * removes it.
      */
     public void removeAfterDelay(final int seconds) {
         this.delay = seconds * 1000;

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/AdapterManager.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/AdapterManager.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/AdapterManager.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/AdapterManager.java Sun Jan 22 07:36:07 2012
@@ -26,16 +26,18 @@ import org.apache.isis.core.metamodel.ad
 import org.apache.isis.core.metamodel.adapter.oid.Oid;
 
 /**
- * Responsible for managing the {@link ObjectAdapter adapter}s and {@link Oid identities} for each and every POJO that
- * is being used by the framework.
+ * Responsible for managing the {@link ObjectAdapter adapter}s and {@link Oid
+ * identities} for each and every POJO that is being used by the framework.
  * 
  * <p>
- * It provides a consistent set of adapters in memory, providing an {@link ObjectAdapter adapter} for the POJOs that are
- * in use ensuring that the same object is not loaded twice into memory.
+ * It provides a consistent set of adapters in memory, providing an
+ * {@link ObjectAdapter adapter} for the POJOs that are in use ensuring that the
+ * same object is not loaded twice into memory.
  * 
  * <p>
- * Each POJO is given an {@link ObjectAdapter adapter} so that the framework can work with the POJOs even though it does
- * not understand their types. Each POJO maps to an {@link ObjectAdapter adapter} and these are reused.
+ * Each POJO is given an {@link ObjectAdapter adapter} so that the framework can
+ * work with the POJOs even though it does not understand their types. Each POJO
+ * maps to an {@link ObjectAdapter adapter} and these are reused.
  */
 public interface AdapterManager extends ObjectAdapterLookup, Injectable, AdapterMap {
 
@@ -44,15 +46,18 @@ public interface AdapterManager extends 
     // /////////////////////////////////////////////////////////
 
     /**
-     * Removes the specified {@link ObjectAdapter adapter} from the identity maps.
+     * Removes the specified {@link ObjectAdapter adapter} from the identity
+     * maps.
      */
     void removeAdapter(ObjectAdapter adapter);
 
     /**
-     * Removes the {@link ObjectAdapter adapter} identified by the specified {@link Oid}.
+     * Removes the {@link ObjectAdapter adapter} identified by the specified
+     * {@link Oid}.
      * 
      * <p>
-     * Should be same as {@link #getAdapterFor(Oid)} followed by {@link #removeAdapter(ObjectAdapter)}.
+     * Should be same as {@link #getAdapterFor(Oid)} followed by
+     * {@link #removeAdapter(ObjectAdapter)}.
      */
     void removeAdapter(Oid oid);
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/ObjectFactory.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/ObjectFactory.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/ObjectFactory.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/ObjectFactory.java Sun Jan 22 07:36:07 2012
@@ -25,8 +25,8 @@ import org.apache.isis.core.metamodel.sp
 public interface ObjectFactory extends SessionScopedComponent {
 
     /**
-     * Should instantiate the object, and in addition initialize the domain object (for example, inject any services and
-     * repositories into it).
+     * Should instantiate the object, and in addition initialize the domain
+     * object (for example, inject any services and repositories into it).
      * 
      */
     <T> T instantiate(Class<T> cls) throws ObjectInstantiationException;

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/OidGenerator.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/OidGenerator.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/OidGenerator.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/OidGenerator.java Sun Jan 22 07:36:07 2012
@@ -34,39 +34,45 @@ import org.apache.isis.runtimes.dflt.run
 public interface OidGenerator extends DebuggableWithTitle, SessionScopedComponent, Injectable {
 
     /**
-     * Create a new {@link Oid#isTransient() transient} {@link Oid} for the supplied pojo, uniquely distinguishable from
-     * any other {@link Oid}.
+     * Create a new {@link Oid#isTransient() transient} {@link Oid} for the
+     * supplied pojo, uniquely distinguishable from any other {@link Oid}.
      */
     Oid createTransientOid(Object pojo);
 
     /**
-     * Creates a new sub-ID for the specified object for use in an {@link AggregatedOid}.
+     * Creates a new sub-ID for the specified object for use in an
+     * {@link AggregatedOid}.
      */
     String createAggregateId(Object pojo);
 
     /**
-     * Convert the {@link Oid} from {@link Oid#isTransient() transient} to persistent, storing the previous {@link Oid}
-     * as {@link Oid#getPrevious()}
+     * Convert the {@link Oid} from {@link Oid#isTransient() transient} to
+     * persistent, storing the previous {@link Oid} as {@link Oid#getPrevious()}
      * 
      * <p>
      * Implementation notes:
      * <ul>
-     * <li>Note that the {@link AdapterManager} will need to be updated so that the {@link ObjectAdapter adapter} is
-     * re-mapped by the new {@link Oid}. However, the <i>not</i> the responsibility of the {@link OidGenerator} to do
-     * this; it merely ensures that the {@link Oid} is in the correct new state.
-     * <li>Some implementations may require that state be set up on the {@link Oid} prior to call. Check for details.
+     * <li>Note that the {@link AdapterManager} will need to be updated so that
+     * the {@link ObjectAdapter adapter} is re-mapped by the new {@link Oid}.
+     * However, the <i>not</i> the responsibility of the {@link OidGenerator} to
+     * do this; it merely ensures that the {@link Oid} is in the correct new
+     * state.
+     * <li>Some implementations may require that state be set up on the
+     * {@link Oid} prior to call. Check for details.
      * </ul>
      */
     void convertTransientToPersistentOid(Oid oid);
 
     /**
-     * An {@link OidStringifier} to use for stringifying instances of the concrete {@link Oid} generated by this
-     * generator.
+     * An {@link OidStringifier} to use for stringifying instances of the
+     * concrete {@link Oid} generated by this generator.
      * 
      * <p>
-     * All {@link Oid}s (ultimately being {@link Encodable}) can be stringified using {@link OidStringifierHex}.
-     * However, some implementations (for example {@link SerialOid}) implement {@link DirectlyStringableOid} which means
-     * that we can get a more user-friendly string representation of the {@link Oid}.
+     * All {@link Oid}s (ultimately being {@link Encodable}) can be stringified
+     * using {@link OidStringifierHex}. However, some implementations (for
+     * example {@link SerialOid}) implement {@link DirectlyStringableOid} which
+     * means that we can get a more user-friendly string representation of the
+     * {@link Oid}.
      */
     OidStringifier getOidStringifier();
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceQuery.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceQuery.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceQuery.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceQuery.java Sun Jan 22 07:36:07 2012
@@ -23,7 +23,8 @@ import org.apache.isis.applib.query.Quer
 import org.apache.isis.core.metamodel.spec.ObjectSpecification;
 
 /**
- * Defines a criteria for including instances in set, corresponds to {@link Query} in the applib.
+ * Defines a criteria for including instances in set, corresponds to
+ * {@link Query} in the applib.
  */
 public interface PersistenceQuery {
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSession.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSession.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSession.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSession.java Sun Jan 22 07:36:07 2012
@@ -30,13 +30,12 @@ import org.apache.isis.core.metamodel.sp
 import org.apache.isis.runtimes.dflt.runtime.system.transaction.IsisTransactionManager;
 import org.apache.isis.runtimes.dflt.runtime.system.transaction.IsisTransactionManagerAware;
 
-public interface PersistenceSession extends PersistenceSessionContainer, PersistenceSessionForceReloader,
-    PersistenceSessionAdaptedServiceManager, PersistenceSessionTransactionManagement, PersistenceSessionHydrator,
-    PersistenceSessionTestSupport, SpecificationLoaderAware, IsisTransactionManagerAware, SessionScopedComponent,
-    Injectable, DebuggableWithTitle {
+public interface PersistenceSession extends PersistenceSessionContainer, PersistenceSessionForceReloader, PersistenceSessionAdaptedServiceManager, PersistenceSessionTransactionManagement, PersistenceSessionHydrator, PersistenceSessionTestSupport, SpecificationLoaderAware,
+        IsisTransactionManagerAware, SessionScopedComponent, Injectable, DebuggableWithTitle {
 
     /**
-     * The {@link PersistenceSessionFactory} that created this {@link PersistenceSession}.
+     * The {@link PersistenceSessionFactory} that created this
+     * {@link PersistenceSession}.
      */
     public PersistenceSessionFactory getPersistenceSessionFactory();
 
@@ -51,11 +50,14 @@ public interface PersistenceSession exte
     public void close();
 
     /**
-     * Determine if the object store has been initialized with its set of start up objects.
+     * Determine if the object store has been initialized with its set of start
+     * up objects.
      * 
      * <p>
-     * This method is called only once after the {@link ApplicationScopedComponent#init()} has been called. If this flag
-     * returns <code>false</code> the framework will run the fixtures to initialise the persistor.
+     * This method is called only once after the
+     * {@link ApplicationScopedComponent#init()} has been called. If this flag
+     * returns <code>false</code> the framework will run the fixtures to
+     * initialise the persistor.
      */
     boolean isFixturesInstalled();
 
@@ -94,8 +96,9 @@ public interface PersistenceSession exte
      * Inject the {@link IsisTransactionManager}.
      * 
      * <p>
-     * This must be injected using setter-based injection rather than through the constructor because there is a
-     * bidirectional relationship between the {@link PersistenceSessionHydrator} and the {@link IsisTransactionManager}.
+     * This must be injected using setter-based injection rather than through
+     * the constructor because there is a bidirectional relationship between the
+     * {@link PersistenceSessionHydrator} and the {@link IsisTransactionManager}.
      * 
      * @see #getTransactionManager()
      */
@@ -113,8 +116,10 @@ public interface PersistenceSession exte
      * Inject the {@link SpecificationLoader}.
      * 
      * <p>
-     * The need to inject the reflector was introduced to support the HibernateObjectStore, which installs its own
-     * <tt>HibernateClassStrategy</tt> to cope with the proxy classes that Hibernate wraps around lists, sets and maps.
+     * The need to inject the reflector was introduced to support the
+     * HibernateObjectStore, which installs its own
+     * <tt>HibernateClassStrategy</tt> to cope with the proxy classes that
+     * Hibernate wraps around lists, sets and maps.
      */
     @Override
     void setSpecificationLoader(SpecificationLoader specificationLoader);

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionContainer.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionContainer.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionContainer.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionContainer.java Sun Jan 22 07:36:07 2012
@@ -39,37 +39,42 @@ public interface PersistenceSessionConta
     // /////////////////////////////////////////////////////////
 
     /**
-     * Creates a new instance of the specified type and returns it in an adapter whose resolved state set to
-     * {@link ResolveState#TRANSIENT} (except if the type is marked as
-     * {@link ObjectSpecification#isValueOrIsAggregated() aggregated} in which case it will be set to
-     * {@link ResolveState#VALUE}).
+     * Creates a new instance of the specified type and returns it in an adapter
+     * whose resolved state set to {@link ResolveState#TRANSIENT} (except if the
+     * type is marked as {@link ObjectSpecification#isValueOrIsAggregated()
+     * aggregated} in which case it will be set to {@link ResolveState#VALUE}).
      * 
      * <p>
-     * <b><i> REVIEW: not sure about {@link ResolveState#VALUE} - see comments in
-     * {@link #adapterFor(Object, Oid, Version)}.</i></b>
+     * <b><i> REVIEW: not sure about {@link ResolveState#VALUE} - see comments
+     * in {@link #adapterFor(Object, Oid, Version)}.</i></b>
      * 
      * <p>
-     * While creating the object it will be initialised with default values and its created lifecycle method (its
-     * logical constructor) will be invoked. Contrast this with
+     * While creating the object it will be initialised with default values and
+     * its created lifecycle method (its logical constructor) will be invoked.
+     * Contrast this with
      * {@link #recreateTransientInstance(Oid, ObjectSpecification)}.
      * 
      * <p>
-     * This method is ultimately delegated to by the {@link DomainObjectContainer}.
+     * This method is ultimately delegated to by the
+     * {@link DomainObjectContainer}.
      */
     ObjectAdapter createInstance(ObjectSpecification specification);
 
     /**
-     * Creates a new instance of the specified type and returns an adapter with an aggregated OID that show that this
-     * new object belongs to the specified parent. The new object's resolved state is set to
+     * Creates a new instance of the specified type and returns an adapter with
+     * an aggregated OID that show that this new object belongs to the specified
+     * parent. The new object's resolved state is set to
      * {@link ResolveState#RESOLVED} as it state is part of it parent.
      * 
      * <p>
-     * While creating the object it will be initialised with default values and its created lifecycle method (its
-     * logical constructor) will be invoked. Contrast this with
+     * While creating the object it will be initialised with default values and
+     * its created lifecycle method (its logical constructor) will be invoked.
+     * Contrast this with
      * {@link #recreateTransientInstance(Oid, ObjectSpecification)}.
      * 
      * <p>
-     * This method is ultimately delegated to by the {@link DomainObjectContainer}.
+     * This method is ultimately delegated to by the
+     * {@link DomainObjectContainer}.
      */
     ObjectAdapter createAggregatedInstance(ObjectSpecification specification, ObjectAdapter parent);
 
@@ -78,7 +83,8 @@ public interface PersistenceSessionConta
     // /////////////////////////////////////////////////////////
 
     /**
-     * Loads the object identified by the specified {@link Oid} from the persisted set of objects.
+     * Loads the object identified by the specified {@link Oid} from the
+     * persisted set of objects.
      * 
      * <p>
      * As used primarily by <tt>ServerDistribution</tt>.
@@ -89,7 +95,8 @@ public interface PersistenceSessionConta
      * Finds and returns instances that match the specified query.
      * 
      * <p>
-     * The {@link QueryCardinality} determines whether all instances or just the first matching instance is returned.
+     * The {@link QueryCardinality} determines whether all instances or just the
+     * first matching instance is returned.
      * 
      * @throws UnsupportedFindException
      *             if the criteria is not support by this persistor
@@ -97,12 +104,14 @@ public interface PersistenceSessionConta
     <T> ObjectAdapter findInstances(Query<T> query, QueryCardinality cardinality);
 
     /**
-     * Finds and returns instances that match the specified {@link PersistenceQuery}.
+     * Finds and returns instances that match the specified
+     * {@link PersistenceQuery}.
      * 
      * <p>
-     * Compared to {@link #findInstances(Query, QueryCardinality)}, not that there is no {@link QueryCardinality}
-     * parameter. That's because {@link PersistenceQuery} intrinsically carry the knowledge as to how many rows they
-     * return.
+     * Compared to {@link #findInstances(Query, QueryCardinality)}, not that
+     * there is no {@link QueryCardinality} parameter. That's because
+     * {@link PersistenceQuery} intrinsically carry the knowledge as to how many
+     * rows they return.
      * 
      * @throws UnsupportedFindException
      *             if the criteria is not support by this persistor
@@ -110,7 +119,8 @@ public interface PersistenceSessionConta
     ObjectAdapter findInstances(PersistenceQuery criteria);
 
     /**
-     * Whether there are any instances of the specified {@link ObjectSpecification type}.
+     * Whether there are any instances of the specified
+     * {@link ObjectSpecification type}.
      * 
      * <p>
      * Used (ostensibly) by client-side code.
@@ -122,18 +132,21 @@ public interface PersistenceSessionConta
     // /////////////////////////////////////////////////////////
 
     /**
-     * Re-initialises the fields of an object. If the object is unresolved then the object's missing data should be
-     * retrieved from the persistence mechanism and be used to set up the value objects and associations.
+     * Re-initialises the fields of an object. If the object is unresolved then
+     * the object's missing data should be retrieved from the persistence
+     * mechanism and be used to set up the value objects and associations.
      */
     void resolveImmediately(ObjectAdapter object);
 
     /**
-     * Hint that specified field within the specified object is likely to be needed soon. This allows the object's data
-     * to be loaded, ready for use.
+     * Hint that specified field within the specified object is likely to be
+     * needed soon. This allows the object's data to be loaded, ready for use.
      * 
      * <p>
-     * This method need not do anything, but offers the object store the opportunity to load in objects before their
-     * use. Contrast this with resolveImmediately, which requires an object to be loaded before continuing.
+     * This method need not do anything, but offers the object store the
+     * opportunity to load in objects before their use. Contrast this with
+     * resolveImmediately, which requires an object to be loaded before
+     * continuing.
      * 
      * @see #resolveImmediately(ObjectAdapter)
      */
@@ -144,24 +157,29 @@ public interface PersistenceSessionConta
     // /////////////////////////////////////////////////////////
 
     /**
-     * Makes an {@link ObjectAdapter} persistent. The specified object should be stored away via this object store's
-     * persistence mechanism, and have an new and unique OID assigned to it (by calling the object's <code>setOid</code>
-     * method). The object, should also be added to the cache as the object is implicitly 'in use'.
+     * Makes an {@link ObjectAdapter} persistent. The specified object should be
+     * stored away via this object store's persistence mechanism, and have an
+     * new and unique OID assigned to it (by calling the object's
+     * <code>setOid</code> method). The object, should also be added to the
+     * cache as the object is implicitly 'in use'.
      * 
      * <p>
-     * If the object has any associations then each of these, where they aren't already persistent, should also be made
-     * persistent by recursively calling this method.
+     * If the object has any associations then each of these, where they aren't
+     * already persistent, should also be made persistent by recursively calling
+     * this method.
      * </p>
      * 
      * <p>
-     * If the object to be persisted is a collection, then each element of that collection, that is not already
-     * persistent, should be made persistent by recursively calling this method.
+     * If the object to be persisted is a collection, then each element of that
+     * collection, that is not already persistent, should be made persistent by
+     * recursively calling this method.
      * </p>
      */
     void makePersistent(ObjectAdapter object);
 
     /**
-     * Mark the {@link ObjectAdapter} as changed, and therefore requiring flushing to the persistence mechanism.
+     * Mark the {@link ObjectAdapter} as changed, and therefore requiring
+     * flushing to the persistence mechanism.
      */
     void objectChanged(ObjectAdapter object);
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionFactory.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionFactory.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionFactory.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionFactory.java Sun Jan 22 07:36:07 2012
@@ -35,7 +35,8 @@ public interface PersistenceSessionFacto
     DeploymentType getDeploymentType();
 
     /**
-     * Creates a {@link PersistenceSession} with the implementing object as the {@link PersistenceSession}'s
+     * Creates a {@link PersistenceSession} with the implementing object as the
+     * {@link PersistenceSession}'s
      * {@link PersistenceSession#getPersistenceSessionFactory() owning factory}.
      */
     PersistenceSession createPersistenceSession();

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionHydrator.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionHydrator.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionHydrator.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionHydrator.java Sun Jan 22 07:36:07 2012
@@ -28,18 +28,22 @@ import org.apache.isis.core.metamodel.sp
 public interface PersistenceSessionHydrator extends Injectable {
 
     /**
-     * Returns an {@link ObjectAdapter adapter} of the {@link ObjectSpecification type} specified.
+     * Returns an {@link ObjectAdapter adapter} of the
+     * {@link ObjectSpecification type} specified.
      * 
      * <p>
-     * If an adapter exists in the {@link AdapterManager map} then that adapter is returned immediately. Otherwise a new
-     * domain object of the type specified is {@link ObjectFactory created} and then an adapter is recreated as per
-     * {@link #recreateAdapter(Oid, Object)}.
+     * If an adapter exists in the {@link AdapterManager map} then that adapter
+     * is returned immediately. Otherwise a new domain object of the type
+     * specified is {@link ObjectFactory created} and then an adapter is
+     * recreated as per {@link #recreateAdapter(Oid, Object)}.
      * 
      * <p>
-     * Note: the similar looking method {@link PersistenceSessionContainer#loadObject(Oid, ObjectSpecification)}
-     * retrieves the existing object from the persistent store (if not available in the {@link AdapterManager maps} .
-     * Once the object has been retrieved, the object store calls back to {@link #recreateAdapter(Oid, Object)} to map
-     * it.
+     * Note: the similar looking method
+     * {@link PersistenceSessionContainer#loadObject(Oid, ObjectSpecification)}
+     * retrieves the existing object from the persistent store (if not available
+     * in the {@link AdapterManager maps} . Once the object has been retrieved,
+     * the object store calls back to {@link #recreateAdapter(Oid, Object)} to
+     * map it.
      * 
      * @see #recreateAdapter(Oid, Object)
      * @see PersistenceSessionContainer#loadObject(Oid, ObjectSpecification)
@@ -47,13 +51,16 @@ public interface PersistenceSessionHydra
     ObjectAdapter recreateAdapter(Oid oid, ObjectSpecification specification);
 
     /**
-     * Returns an adapter for the provided {@link Oid}, wrapping the provided domain object.
+     * Returns an adapter for the provided {@link Oid}, wrapping the provided
+     * domain object.
      * 
      * <p>
-     * If an adapter exists in the {@link AdapterManager map} for either the {@link Oid} or the domain object then that
-     * adapter is returned immediately. Otherwise a new adapter is created using the specified {@link Oid} and its
-     * resolved state set to either {@link ResolveState#TRANSIENT} or {@link ResolveState#GHOST} based on whether the
-     * {@link Oid} is {@link Oid#isTransient() transient} or not.
+     * If an adapter exists in the {@link AdapterManager map} for either the
+     * {@link Oid} or the domain object then that adapter is returned
+     * immediately. Otherwise a new adapter is created using the specified
+     * {@link Oid} and its resolved state set to either
+     * {@link ResolveState#TRANSIENT} or {@link ResolveState#GHOST} based on
+     * whether the {@link Oid} is {@link Oid#isTransient() transient} or not.
      */
     ObjectAdapter recreateAdapter(Oid oid, Object pojo);
 }

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionTransactionManagement.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionTransactionManagement.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionTransactionManagement.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/persistence/PersistenceSessionTransactionManagement.java Sun Jan 22 07:36:07 2012
@@ -33,7 +33,8 @@ import org.apache.isis.runtimes.dflt.run
 public interface PersistenceSessionTransactionManagement {
 
     /**
-     * Mark as {@link #objectChanged(ObjectAdapter) changed } all {@link Dirtiable} objects that have been
+     * Mark as {@link #objectChanged(ObjectAdapter) changed } all
+     * {@link Dirtiable} objects that have been
      * {@link Dirtiable#markDirty(ObjectAdapter) manually marked} as dirty.
      * 
      * <p>

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSession.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSession.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSession.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSession.java Sun Jan 22 07:36:07 2012
@@ -30,11 +30,12 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.isis.runtimes.dflt.runtime.system.transaction.IsisTransaction;
 
 /**
- * Analogous to a Hibernate <tt>Session</tt>, holds the current set of components for a specific execution context (such
- * as on a thread).
+ * Analogous to a Hibernate <tt>Session</tt>, holds the current set of
+ * components for a specific execution context (such as on a thread).
  * 
  * <p>
- * The <tt>IsisContext</tt> class (in <tt>nof-core</tt>) is responsible for locating the current execution context.
+ * The <tt>IsisContext</tt> class (in <tt>nof-core</tt>) is responsible for
+ * locating the current execution context.
  * 
  * @see IsisSessionFactory
  */
@@ -48,8 +49,9 @@ public interface IsisSession extends Ses
      * The creating {@link IsisSessionFactory factory}.
      * 
      * <p>
-     * Note that from the factory we can {@link IsisSessionFactory#getIsisSystem() get to} the {@link IsisSystem}, and
-     * thus other {@link ApplicationScopedComponent}s.
+     * Note that from the factory we can
+     * {@link IsisSessionFactory#getIsisSystem() get to} the {@link IsisSystem},
+     * and thus other {@link ApplicationScopedComponent}s.
      */
     public IsisSessionFactory getSessionFactory();
 
@@ -58,8 +60,10 @@ public interface IsisSession extends Ses
     // //////////////////////////////////////////////////////
 
     /**
-     * Normal lifecycle is managed using callbacks in {@link SessionScopedComponent}. This method is to allow the outer
-     * {@link ApplicationScopedComponent}s to shutdown, closing any and all running {@link IsisSession}s.
+     * Normal lifecycle is managed using callbacks in
+     * {@link SessionScopedComponent}. This method is to allow the outer
+     * {@link ApplicationScopedComponent}s to shutdown, closing any and all
+     * running {@link IsisSession}s.
      */
     public void closeAll();
 
@@ -77,7 +81,8 @@ public interface IsisSession extends Ses
     // //////////////////////////////////////////////////////
 
     /**
-     * Returns the {@link AuthenticationSession} representing this user for this {@link IsisSession}.
+     * Returns the {@link AuthenticationSession} representing this user for this
+     * {@link IsisSession}.
      */
     public AuthenticationSession getAuthenticationSession();
 
@@ -89,7 +94,8 @@ public interface IsisSession extends Ses
      * The {@link PersistenceSession} within this {@link IsisSession}.
      * 
      * <p>
-     * Would have been created by the {@link #getSessionFactory() owning factory}'s
+     * Would have been created by the {@link #getSessionFactory() owning
+     * factory}'s
      * 
      */
     public PersistenceSession getPersistenceSession();
@@ -99,8 +105,8 @@ public interface IsisSession extends Ses
     // //////////////////////////////////////////////////////
 
     /**
-     * Returns the {@link ObjectAdapter adapted} <tt>Perspective</tt> for the user who is using this {@link IsisSession}
-     * .
+     * Returns the {@link ObjectAdapter adapted} <tt>Perspective</tt> for the
+     * user who is using this {@link IsisSession} .
      */
 
     public UserProfile getUserProfile();

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionDefault.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionDefault.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionDefault.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionDefault.java Sun Jan 22 07:36:07 2012
@@ -44,11 +44,12 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.log4j.Logger;
 
 /**
- * Analogous to a Hibernate <tt>Session</tt>, holds the current set of components for a specific execution context (such
- * as on a thread).
+ * Analogous to a Hibernate <tt>Session</tt>, holds the current set of
+ * components for a specific execution context (such as on a thread).
  * 
  * <p>
- * The <tt>IsisContext</tt> class (in <tt>nof-core</tt>) is responsible for locating the current execution context.
+ * The <tt>IsisContext</tt> class (in <tt>nof-core</tt>) is responsible for
+ * locating the current execution context.
  * 
  * @see IsisSessionFactory
  */
@@ -62,16 +63,15 @@ public class IsisSessionDefault implemen
     private final IsisSessionFactory executionContextFactory;
 
     private final AuthenticationSession authenticationSession;
-    private PersistenceSession persistenceSession; // only non-final so can be replaced in tests.
+    private PersistenceSession persistenceSession; // only non-final so can be
+                                                   // replaced in tests.
     private final UserProfile userProfile;
 
     private final int id;
     private long accessTime;
     private String debugSnapshot;
 
-    public IsisSessionDefault(final IsisSessionFactory sessionFactory,
-        final AuthenticationSession authenticationSession, final PersistenceSession persistenceSession,
-        final UserProfile userProfile) {
+    public IsisSessionDefault(final IsisSessionFactory sessionFactory, final AuthenticationSession authenticationSession, final PersistenceSession persistenceSession, final UserProfile userProfile) {
 
         // global context
         ensureThatArg(sessionFactory, is(not(nullValue())), "execution context factory is required");
@@ -173,7 +173,8 @@ public class IsisSessionDefault implemen
     // //////////////////////////////////////////////////////
 
     /**
-     * Returns the security session representing this user for this execution context.
+     * Returns the security session representing this user for this execution
+     * context.
      */
     @Override
     public AuthenticationSession getAuthenticationSession() {
@@ -231,7 +232,8 @@ public class IsisSessionDefault implemen
     // //////////////////////////////////////////////////////
 
     /**
-     * Convenience method that returns the {@link IsisTransaction} of the session, if any.
+     * Convenience method that returns the {@link IsisTransaction} of the
+     * session, if any.
      */
     @Override
     public IsisTransaction getCurrentTransaction() {

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactory.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactory.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactory.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactory.java Sun Jan 22 07:36:07 2012
@@ -46,39 +46,47 @@ public interface IsisSessionFactory exte
     IsisSession openSession(final AuthenticationSession session);
 
     /**
-     * The {@link ApplicationScopedComponent application-scoped} {@link DeploymentType}.
+     * The {@link ApplicationScopedComponent application-scoped}
+     * {@link DeploymentType}.
      */
     public DeploymentType getDeploymentType();
 
     /**
-     * The {@link ApplicationScopedComponent application-scoped} {@link IsisConfiguration}.
+     * The {@link ApplicationScopedComponent application-scoped}
+     * {@link IsisConfiguration}.
      */
     public IsisConfiguration getConfiguration();
 
     /**
-     * The {@link ApplicationScopedComponent application-scoped} {@link SpecificationLoader}.
+     * The {@link ApplicationScopedComponent application-scoped}
+     * {@link SpecificationLoader}.
      */
     public SpecificationLoader getSpecificationLoader();
 
     /**
-     * The {@link ApplicationScopedComponent application-scoped} {@link TemplateImageLoader}.
+     * The {@link ApplicationScopedComponent application-scoped}
+     * {@link TemplateImageLoader}.
      */
     public TemplateImageLoader getTemplateImageLoader();
 
     /**
-     * The {@link AuthenticationManager} that will be used to authenticate and create {@link AuthenticationSession}s
-     * {@link IsisSession#getAuthenticationSession() within} the {@link IsisSession}.
+     * The {@link AuthenticationManager} that will be used to authenticate and
+     * create {@link AuthenticationSession}s
+     * {@link IsisSession#getAuthenticationSession() within} the
+     * {@link IsisSession}.
      */
     public AuthenticationManager getAuthenticationManager();
 
     /**
-     * The {@link AuthorizationManager} that will be used to authorize access to domain objects.
+     * The {@link AuthorizationManager} that will be used to authorize access to
+     * domain objects.
      */
     public AuthorizationManager getAuthorizationManager();
 
     /**
-     * The {@link PersistenceSessionFactory} that will be used to create {@link PersistenceSession}
-     * {@link IsisSession#getPersistenceSession() within} the {@link IsisSession}.
+     * The {@link PersistenceSessionFactory} that will be used to create
+     * {@link PersistenceSession} {@link IsisSession#getPersistenceSession()
+     * within} the {@link IsisSession}.
      */
     public PersistenceSessionFactory getPersistenceSessionFactory();
 

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryAbstract.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryAbstract.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryAbstract.java Sun Jan 22 07:36:07 2012
@@ -42,14 +42,17 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.isis.runtimes.dflt.runtime.system.persistence.PersistenceSessionFactory;
 
 /**
- * Creates an implementation of {@link IsisSessionFactory#openSession(AuthenticationSession)} to create an
- * {@link IsisSession}, but delegates to subclasses to actually obtain the components that make up that
- * {@link IsisSession}.
+ * Creates an implementation of
+ * {@link IsisSessionFactory#openSession(AuthenticationSession)} to create an
+ * {@link IsisSession}, but delegates to subclasses to actually obtain the
+ * components that make up that {@link IsisSession}.
  * 
  * <p>
- * The idea is that one subclass can use the {@link InstallerLookup} design to lookup installers for components (and
- * hence create the components themselves), whereas another subclass might simply use Spring (or another DI container)
- * to inject in the components according to some Spring-configured application context.
+ * The idea is that one subclass can use the {@link InstallerLookup} design to
+ * lookup installers for components (and hence create the components
+ * themselves), whereas another subclass might simply use Spring (or another DI
+ * container) to inject in the components according to some Spring-configured
+ * application context.
  */
 public abstract class IsisSessionFactoryAbstract implements IsisSessionFactory {
 
@@ -63,11 +66,8 @@ public abstract class IsisSessionFactory
     private final UserProfileLoader userProfileLoader;
     private final List<Object> serviceList;
 
-    public IsisSessionFactoryAbstract(final DeploymentType deploymentType, final IsisConfiguration configuration,
-        final SpecificationLoader specificationLoader, final TemplateImageLoader templateImageLoader,
-        final AuthenticationManager authenticationManager, final AuthorizationManager authorizationManager,
-        final UserProfileLoader userProfileLoader, final PersistenceSessionFactory persistenceSessionFactory,
-        final List<Object> serviceList) {
+    public IsisSessionFactoryAbstract(final DeploymentType deploymentType, final IsisConfiguration configuration, final SpecificationLoader specificationLoader, final TemplateImageLoader templateImageLoader, final AuthenticationManager authenticationManager,
+            final AuthorizationManager authorizationManager, final UserProfileLoader userProfileLoader, final PersistenceSessionFactory persistenceSessionFactory, final List<Object> serviceList) {
 
         ensureThatArg(deploymentType, is(not(nullValue())));
         ensureThatArg(configuration, is(not(nullValue())));
@@ -95,7 +95,8 @@ public abstract class IsisSessionFactory
     // ///////////////////////////////////////////
 
     /**
-     * Wires components as necessary, and then {@link ApplicationScopedComponent#init() init}ializes all.
+     * Wires components as necessary, and then
+     * {@link ApplicationScopedComponent#init() init}ializes all.
      */
     @Override
     public void init() {
@@ -132,11 +133,11 @@ public abstract class IsisSessionFactory
         final UserProfile userProfile = userProfileLoader.getProfile(authenticationSession);
         ensureThatArg(userProfile, is(not(nullValue())));
 
-        // inject into persistenceSession any/all application-scoped components that it requires
+        // inject into persistenceSession any/all application-scoped components
+        // that it requires
         getSpecificationLoader().injectInto(persistenceSession);
 
-        final IsisSessionDefault isisSessionDefault =
-            new IsisSessionDefault(this, authenticationSession, persistenceSession, userProfile);
+        final IsisSessionDefault isisSessionDefault = new IsisSessionDefault(this, authenticationSession, persistenceSession, userProfile);
 
         return isisSessionDefault;
     }

Modified: incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryDefault.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryDefault.java?rev=1234456&r1=1234455&r2=1234456&view=diff
==============================================================================
--- incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryDefault.java (original)
+++ incubator/isis/trunk/framework/runtimes/dflt/runtime/src/main/java/org/apache/isis/runtimes/dflt/runtime/system/session/IsisSessionFactoryDefault.java Sun Jan 22 07:36:07 2012
@@ -31,18 +31,14 @@ import org.apache.isis.runtimes.dflt.run
 import org.apache.isis.runtimes.dflt.runtime.system.persistence.PersistenceSessionFactory;
 
 /**
- * As its superclass, but provides a default for some of more basic components (that is, where the core framework offers
- * only a single implementation).
+ * As its superclass, but provides a default for some of more basic components
+ * (that is, where the core framework offers only a single implementation).
  */
 public class IsisSessionFactoryDefault extends IsisSessionFactoryAbstract {
 
-    public IsisSessionFactoryDefault(final DeploymentType deploymentType, final IsisConfiguration configuration,
-        final TemplateImageLoader templateImageLoader, final SpecificationLoader specificationLoader,
-        final AuthenticationManager authenticationManager, final AuthorizationManager authorizationManager,
-        final UserProfileLoader userProfileLoader, final PersistenceSessionFactory persistenceSessionFactory,
-        final List<Object> servicesList) {
-        super(deploymentType, configuration, specificationLoader, templateImageLoader, authenticationManager,
-            authorizationManager, userProfileLoader, persistenceSessionFactory, servicesList);
+    public IsisSessionFactoryDefault(final DeploymentType deploymentType, final IsisConfiguration configuration, final TemplateImageLoader templateImageLoader, final SpecificationLoader specificationLoader, final AuthenticationManager authenticationManager,
+            final AuthorizationManager authorizationManager, final UserProfileLoader userProfileLoader, final PersistenceSessionFactory persistenceSessionFactory, final List<Object> servicesList) {
+        super(deploymentType, configuration, specificationLoader, templateImageLoader, authenticationManager, authorizationManager, userProfileLoader, persistenceSessionFactory, servicesList);
     }
 
 }



Mime
View raw message