incubator-isis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danhayw...@apache.org
Subject svn commit: r1234388 [3/4] - in /incubator/isis/trunk/framework/applib/src: main/java/org/apache/isis/applib/ main/java/org/apache/isis/applib/adapters/ main/java/org/apache/isis/applib/annotation/ main/java/org/apache/isis/applib/clock/ main/java/org/...
Date Sat, 21 Jan 2012 18:30:35 GMT
Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/FixtureType.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/FixtureType.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/FixtureType.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/FixtureType.java Sat Jan 21 18:30:32 2012
@@ -24,36 +24,43 @@ import org.apache.isis.applib.profiles.P
 import org.apache.isis.applib.profiles.Profile;
 
 /**
- * Enumerates the different types of {@link InstallableFixture fixture}s supported.
+ * Enumerates the different types of {@link InstallableFixture fixture}s
+ * supported.
  * 
  * @see InstallableFixture#getType()
  */
 public enum FixtureType {
     /**
-     * A fixture that installs data (either reference data or operational data) into an object store.
+     * A fixture that installs data (either reference data or operational data)
+     * into an object store.
      * 
      * <p>
-     * Some object stores are in-memory only, in which case these will always want fixtures of this type to be
-     * installed. However, for object stores that persist the data (such as XML or to an RDBMS), these typically do
-     * <i>not</i> want data fixtures run (except possibly for the very first time booted to initially seed them).
+     * Some object stores are in-memory only, in which case these will always
+     * want fixtures of this type to be installed. However, for object stores
+     * that persist the data (such as XML or to an RDBMS), these typically do
+     * <i>not</i> want data fixtures run (except possibly for the very first
+     * time booted to initially seed them).
      */
     DOMAIN_OBJECTS,
     /**
-     * Analogous to {@link FixtureType#DATA}, but for fixtures (in particular, the {@link PerspectiveFixture}) that are
-     * used to setup {@link Profile}s and their {@link Perspective}s.
+     * Analogous to {@link FixtureType#DATA}, but for fixtures (in particular,
+     * the {@link PerspectiveFixture}) that are used to setup {@link Profile}s
+     * and their {@link Perspective}s.
      * 
      * @see UserProfileService
      */
     USER_PROFILES,
     /**
-     * A fixture that neither installs data into the object store nor perspectives into the {@link UserProfileService}.
+     * A fixture that neither installs data into the object store nor
+     * perspectives into the {@link UserProfileService}.
      * 
      * <p>
      * Fixtures of this type are always installed. Typical examples are:
      * <ul>
      * <li>composite fixtures that just aggregate other fixtures
      * <li>fixtures that set up the date/time (see {@link DateFixture})
-     * <li>fixtures that specify the user to logon as (see {@link LogonFixture}).
+     * <li>fixtures that specify the user to logon as (see {@link LogonFixture}
+     * ).
      * </ul>
      */
     OTHER;

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/InstallableFixture.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/InstallableFixture.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/InstallableFixture.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/InstallableFixture.java Sat Jan 21 18:30:32 2012
@@ -22,8 +22,9 @@ package org.apache.isis.applib.fixtures;
 public interface InstallableFixture {
 
     /**
-     * Determines whether the fixture will be {@link #install() install}ed, dependent on the state of the object (data)
-     * store and the user profile store.
+     * Determines whether the fixture will be {@link #install() install}ed,
+     * dependent on the state of the object (data) store and the user profile
+     * store.
      */
     FixtureType getType();
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/LogonFixture.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/LogonFixture.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/LogonFixture.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/LogonFixture.java Sat Jan 21 18:30:32 2012
@@ -25,15 +25,18 @@ import com.google.common.collect.Immutab
 import com.google.common.collect.Lists;
 
 /**
- * Indicates that the demo or test should be run as the specified user, with the specified roles.
+ * Indicates that the demo or test should be run as the specified user, with the
+ * specified roles.
  * 
  * <p>
- * Note: this fixture does not in itself do anything (its {@link #install()} is a no-op). However, if present in the
- * fixture list then is &quot;noticed&quot; by the framework, and is used to automatically logon when the framework is
+ * Note: this fixture does not in itself do anything (its {@link #install()} is
+ * a no-op). However, if present in the fixture list then is &quot;noticed&quot;
+ * by the framework, and is used to automatically logon when the framework is
  * booted (providing running in prototype or exploration, not in production).
  * 
  * <p>
- * To change the user during the installation of fixtures, either use {@link SwitchUserFixture}.
+ * To change the user during the installation of fixtures, either use
+ * {@link SwitchUserFixture}.
  * 
  * @see SwitchUserFixture
  */

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/SwitchUserFixture.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/SwitchUserFixture.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/SwitchUserFixture.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/SwitchUserFixture.java Sat Jan 21 18:30:32 2012
@@ -21,21 +21,24 @@ package org.apache.isis.applib.fixtures;
 
 import java.util.List;
 
-import com.google.common.collect.ImmutableList;
-import com.google.common.collect.Lists;
-
 import org.apache.isis.applib.fixtures.switchuser.SwitchUserService;
 import org.apache.isis.applib.fixtures.switchuser.SwitchUserServiceAware;
 
+import com.google.common.collect.ImmutableList;
+import com.google.common.collect.Lists;
+
 /**
- * Sole purpose is to switch the current user while object fixtures are being installed.
+ * Sole purpose is to switch the current user while object fixtures are being
+ * installed.
  * 
  * <p>
- * An alternative is to switch user using the {@link AbstractFixture#switchUser(String, String...) switchUser} method.
+ * An alternative is to switch user using the
+ * {@link AbstractFixture#switchUser(String, String...) switchUser} method.
  * 
  * <p>
- * Note that (unlike the otherwise similar {@link DateFixture}) the last user switched to is <i>not</i> used as the
- * logon fixture. If you want to automatically logon as some user, use the {@link LogonFixture}.
+ * Note that (unlike the otherwise similar {@link DateFixture}) the last user
+ * switched to is <i>not</i> used as the logon fixture. If you want to
+ * automatically logon as some user, use the {@link LogonFixture}.
  * 
  * @see DateFixture
  * @see LogonFixture

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserService.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserService.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserService.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserService.java Sat Jan 21 18:30:32 2012
@@ -25,11 +25,13 @@ import org.apache.isis.applib.fixtures.A
 import org.apache.isis.applib.fixtures.userprofile.UserProfileService;
 
 /**
- * Not intended to be used directly; decouples the {@link AbstractFixture}, which needs to be able to switch users
- * dynamically, from the rest of the framework's runtime.
+ * Not intended to be used directly; decouples the {@link AbstractFixture},
+ * which needs to be able to switch users dynamically, from the rest of the
+ * framework's runtime.
  * 
  * <p>
- * A suitable implementation is injected into {@link AbstractFixture} when installed.
+ * A suitable implementation is injected into {@link AbstractFixture} when
+ * installed.
  * 
  * @see UserProfileService
  */

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserServiceAware.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserServiceAware.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserServiceAware.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/switchuser/SwitchUserServiceAware.java Sat Jan 21 18:30:32 2012
@@ -25,7 +25,8 @@ import org.apache.isis.applib.fixtures.A
  * Implement if require {@link SwitchUserService} to be injected into fixture.
  * 
  * <p>
- * Most fixtures will subclass from {@link AbstractFixture} which does indeed implement this interface.
+ * Most fixtures will subclass from {@link AbstractFixture} which does indeed
+ * implement this interface.
  */
 public interface SwitchUserServiceAware {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileService.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileService.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileService.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileService.java Sat Jan 21 18:30:32 2012
@@ -25,11 +25,13 @@ import org.apache.isis.applib.profiles.P
 import org.apache.isis.applib.profiles.Profile;
 
 /**
- * Not intended to be used directly; decouples the {@link UserProfileFixture}, which needs to persist
- * {@link Perspective}s, from the rest of the framework's runtime.
+ * Not intended to be used directly; decouples the {@link UserProfileFixture},
+ * which needs to persist {@link Perspective}s, from the rest of the framework's
+ * runtime.
  * 
  * <p>
- * A suitable implementation is injected into {@link UserProfileFixture} when installed.
+ * A suitable implementation is injected into {@link UserProfileFixture} when
+ * installed.
  * 
  * @see SwitchUserService
  */

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileServiceAware.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileServiceAware.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileServiceAware.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/fixtures/userprofile/UserProfileServiceAware.java Sat Jan 21 18:30:32 2012
@@ -22,7 +22,8 @@ package org.apache.isis.applib.fixtures.
 import org.apache.isis.applib.fixtures.UserProfileFixture;
 
 /**
- * Not intended to be implemented directly; is implemented by {@link UserProfileFixture}.
+ * Not intended to be implemented directly; is implemented by
+ * {@link UserProfileFixture}.
  * 
  * <p>
  * If using perspectives then subclass from {@link UserProfileFixture}.

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/Bounded.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/Bounded.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/Bounded.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/Bounded.java Sat Jan 21 18:30:32 2012
@@ -20,8 +20,8 @@
 package org.apache.isis.applib.marker;
 
 /**
- * Marker interface to indicate that a class has a static set of instances; they are loaded on first use and are not
- * refreshed.
+ * Marker interface to indicate that a class has a static set of instances; they
+ * are loaded on first use and are not refreshed.
  * 
  * Use {@link Bounded} annotation in preference to this marker interface.
  */

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/ImmutableUntilPersisted.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/ImmutableUntilPersisted.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/ImmutableUntilPersisted.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/ImmutableUntilPersisted.java Sat Jan 21 18:30:32 2012
@@ -22,7 +22,8 @@ package org.apache.isis.applib.marker;
 import org.apache.isis.applib.annotation.Immutable;
 
 /**
- * Marker interface to show that an object cannot be changed before it is persisted.
+ * Marker interface to show that an object cannot be changed before it is
+ * persisted.
  * 
  * Use {@link Immutable} annotation in preference to this marker interface.
  */

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/NeverImmutable.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/NeverImmutable.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/NeverImmutable.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/marker/NeverImmutable.java Sat Jan 21 18:30:32 2012
@@ -22,7 +22,8 @@ package org.apache.isis.applib.marker;
 import org.apache.isis.applib.annotation.Immutable;
 
 /**
- * Marker interface to show that an object can always be changed, even after persisted.
+ * Marker interface to show that an object can always be changed, even after
+ * persisted.
  * 
  * Use {@link Immutable} annotation in preference to this marker interface.
  */

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/maybe/Maybe.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/maybe/Maybe.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/maybe/Maybe.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/maybe/Maybe.java Sat Jan 21 18:30:32 2012
@@ -17,86 +17,84 @@
  *  under the License.
  */
 
+package org.apache.isis.applib.maybe;
 
-package org.apache.isis.applib.maybe;
-
 /**
  * As an alternative to returning null; equivalent to Scala's <tt>Option</tt>
  * type (and Haskell's <tt>Maybe</tt>).
  * 
  * <p>
- * Not used by the default programming model, but may be in the
- * future.  Alternatively, you might wish to use for your own custom
- * programming model.  For example, a <tt>validateXxx()</tt> could return
- * a <tt>Maybe&lt;String&gt;</tt> rather than a simple nullable <tt>String</tt>.
+ * Not used by the default programming model, but may be in the future.
+ * Alternatively, you might wish to use for your own custom programming model.
+ * For example, a <tt>validateXxx()</tt> could return a
+ * <tt>Maybe&lt;String&gt;</tt> rather than a simple nullable <tt>String</tt>.
  * 
  * <p>
- * May also be used by domain objects (and is used more generally by the
- * Isis framework itself).
- */
-public final class Maybe<T> {
-	
-	public final static <T> Maybe<T> setTo(T t) {
-		return new Maybe<T>(t);
-	}
-
-	/**
-	 * A {@link Maybe} that is not set, with the type specified using a class object. 
-     *
-     * <p>
+ * May also be used by domain objects (and is used more generally by the Isis
+ * framework itself).
+ */
+public final class Maybe<T> {
+
+    public final static <T> Maybe<T> setTo(final T t) {
+        return new Maybe<T>(t);
+    }
+
+    /**
+     * A {@link Maybe} that is not set, with the type specified using a class
+     * object.
+     * 
+     * <p>
      * <code>
      * Maybe.notSet(String.class)
-     * </code>
-	 */
-	public final static <T> Maybe<T> notSet(Class<T> cls) {
-		return new Maybe<T>(null);
-	}
-
-    /**
-     * A {@link Maybe} that is not set, with the type specified using a type parameter.
-     *
-     * <p>
+     * </code>
+     */
+    public final static <T> Maybe<T> notSet(final Class<T> cls) {
+        return new Maybe<T>(null);
+    }
+
+    /**
+     * A {@link Maybe} that is not set, with the type specified using a type
+     * parameter.
+     * 
+     * <p>
      * <code>
      * Maybe.&lt;String&gt;notSet()
-     * </code>
-     */
-    public final static <T> Maybe<T> notSet() {
-        return new Maybe<T>(null);
-    }
-
-    public final static <T> Maybe<T> setIf(boolean condition, T t) {
-        if (condition) {
-            return Maybe.setTo(t);
-        } else {
-            return Maybe.notSet();
-        }
-    }
-
-    /**
-     * Reciprocal of {@link #setIf(boolean, Object)}.
-     */
-    public final static <T> Maybe<T> notSetIf(boolean b, T t) {
-        return setIf(!b, t);
-    }
-
-
-
-	private final T t;
-
-	private Maybe(T t) {
-		this.t = t;
-	}
-	
-	public boolean isSet() {
-		return t != null;
-	}
-	
-	public T get() {
-		if (!isSet()) {
-			throw new IllegalStateException("No object");
-		}
-		return t;
-	}
-
-
-}
+     * </code>
+     */
+    public final static <T> Maybe<T> notSet() {
+        return new Maybe<T>(null);
+    }
+
+    public final static <T> Maybe<T> setIf(final boolean condition, final T t) {
+        if (condition) {
+            return Maybe.setTo(t);
+        } else {
+            return Maybe.notSet();
+        }
+    }
+
+    /**
+     * Reciprocal of {@link #setIf(boolean, Object)}.
+     */
+    public final static <T> Maybe<T> notSetIf(final boolean b, final T t) {
+        return setIf(!b, t);
+    }
+
+    private final T t;
+
+    private Maybe(final T t) {
+        this.t = t;
+    }
+
+    public boolean isSet() {
+        return t != null;
+    }
+
+    public T get() {
+        if (!isSet()) {
+            throw new IllegalStateException("No object");
+        }
+        return t;
+    }
+
+}

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Localization.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Localization.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Localization.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Localization.java Sat Jan 21 18:30:32 2012
@@ -22,10 +22,9 @@ import java.util.Locale;
 import java.util.TimeZone;
 
 public interface Localization {
-    
+
     Locale getLocale();
-    
+
     TimeZone getTimeZone();
-    
-}
 
+}

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Perspective.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Perspective.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Perspective.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Perspective.java Sat Jan 21 18:30:32 2012
@@ -22,20 +22,25 @@ package org.apache.isis.applib.profiles;
 import org.apache.isis.applib.fixtures.UserProfileFixture;
 
 /**
- * Domain object representing a particular perspective on the services or objects available to a logged-in user.
+ * Domain object representing a particular perspective on the services or
+ * objects available to a logged-in user.
  * 
  * <p>
- * The word "perspective" is used here in a similar sense to the Eclipse IDE, meaning a configuration of objects. You
- * might also think of it as the user's desktop (though that metaphor only makes sense with the DnD viewer).
+ * The word "perspective" is used here in a similar sense to the Eclipse IDE,
+ * meaning a configuration of objects. You might also think of it as the user's
+ * desktop (though that metaphor only makes sense with the DnD viewer).
  * 
  * <p>
- * Note that this type is an interface, not a class. The actual implementation is provided by the framework itself.
+ * Note that this type is an interface, not a class. The actual implementation
+ * is provided by the framework itself.
  * 
  * <p>
- * {@link Perspective}s go together with {@link Profile}s: a {@link Profile} is a container of multiple
- * {@link Perspective}s. As such, {@link Perspective}s can be created from {@link Profile}s; {@link Profile}s themselves
- * are created using the {@link UserProfileFixture} can be used. Thereafter the @{link Profile} and its
- * {@link Perspective}s are stored in a <tt>profilestore</tt> (analogous to an object store).
+ * {@link Perspective}s go together with {@link Profile}s: a {@link Profile} is
+ * a container of multiple {@link Perspective}s. As such, {@link Perspective}s
+ * can be created from {@link Profile}s; {@link Profile}s themselves are created
+ * using the {@link UserProfileFixture} can be used. Thereafter the @{link
+ * Profile} and its {@link Perspective}s are stored in a <tt>profilestore</tt>
+ * (analogous to an object store).
  */
 public interface Perspective {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Profile.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Profile.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Profile.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/profiles/Profile.java Sat Jan 21 18:30:32 2012
@@ -22,17 +22,20 @@ package org.apache.isis.applib.profiles;
 import org.apache.isis.applib.fixtures.UserProfileFixture;
 
 /**
- * Domain object representing a collection of options or preferences for a user, along with a set of {@link Perspective}
- * s for that user.
+ * Domain object representing a collection of options or preferences for a user,
+ * along with a set of {@link Perspective} s for that user.
  * 
  * <p>
- * Note that this type is an interface, not a class. The actual implementation is provided by the framework itself.
+ * Note that this type is an interface, not a class. The actual implementation
+ * is provided by the framework itself.
  * 
  * <p>
- * {@link Perspective}s go together with {@link Profile}s: a {@link Profile} is a container of multiple
- * {@link Perspective}s. As such, {@link Perspective}s can be created from {@link Profile}s; {@link Profile}s themselves
- * are created using the {@link UserProfileFixture} can be used. Thereafter the @{link Profile} and its
- * {@link Perspective}s are stored in a <tt>profilestore</tt> (analogous to an object store).
+ * {@link Perspective}s go together with {@link Profile}s: a {@link Profile} is
+ * a container of multiple {@link Perspective}s. As such, {@link Perspective}s
+ * can be created from {@link Profile}s; {@link Profile}s themselves are created
+ * using the {@link UserProfileFixture} can be used. Thereafter the @{link
+ * Profile} and its {@link Perspective}s are stored in a <tt>profilestore</tt>
+ * (analogous to an object store).
  */
 public interface Profile {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/Query.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/Query.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/Query.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/Query.java Sat Jan 21 18:30:32 2012
@@ -28,14 +28,16 @@ import org.apache.isis.applib.filter.Fil
  * For use by repository implementations, representing the values of a query.
  * 
  * <p>
- * The implementations of these objects are be provided by the underlying persistor/object store; consult its
- * documentation.
+ * The implementations of these objects are be provided by the underlying
+ * persistor/object store; consult its documentation.
  * 
  * <p>
- * <b>Note:</b> that not every object store will necessarily support this interface. In particular, the in-memory object
- * store does not. For this, you can use the {@link Filter} interface to similar effect, for example in
- * {@link DomainObjectContainer#allMatches(Class, Filter)}). Note that the filtering is done within the
- * {@link DomainObjectContainer} rather than being pushed back to the object store.
+ * <b>Note:</b> that not every object store will necessarily support this
+ * interface. In particular, the in-memory object store does not. For this, you
+ * can use the {@link Filter} interface to similar effect, for example in
+ * {@link DomainObjectContainer#allMatches(Class, Filter)}). Note that the
+ * filtering is done within the {@link DomainObjectContainer} rather than being
+ * pushed back to the object store.
  */
 public interface Query<T> extends Serializable {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryAbstract.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryAbstract.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryAbstract.java Sat Jan 21 18:30:32 2012
@@ -31,8 +31,8 @@ public abstract class QueryAbstract<T> i
 
     private final String resultTypeName;
     /**
-     * Derived from {@link #getResultTypeName()}, with respect to the {@link Thread#getContextClassLoader() current
-     * thread's class loader}.
+     * Derived from {@link #getResultTypeName()}, with respect to the
+     * {@link Thread#getContextClassLoader() current thread's class loader}.
      */
     private transient Class<T> resultType;
 
@@ -46,7 +46,8 @@ public abstract class QueryAbstract<T> i
 
     /**
      * @throws IllegalStateException
-     *             (wrapping a {@link ClassNotFoundException}) if the class could not be determined.
+     *             (wrapping a {@link ClassNotFoundException}) if the class
+     *             could not be determined.
      */
     @Override
     @SuppressWarnings("unchecked")

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryBuiltInAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryBuiltInAbstract.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryBuiltInAbstract.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryBuiltInAbstract.java Sat Jan 21 18:30:32 2012
@@ -24,18 +24,21 @@ import java.io.Serializable;
 import org.apache.isis.applib.DomainObjectContainer;
 
 /**
- * Although (through this class) the subclasses implements {@link Query} and thus are meant to be {@link Serializable},
- * this isn't actually required of the built-in queries because they are all converted into corresponding
+ * Although (through this class) the subclasses implements {@link Query} and
+ * thus are meant to be {@link Serializable}, this isn't actually required of
+ * the built-in queries because they are all converted into corresponding
  * <tt>PersistenceQuery</tt> in the runtime for remoting purposes.
  * 
  * <p>
- * The principle reason for this is to reduce the size of the API from the {@link DomainObjectContainer} to
- * <tt>RuntimeContext</tt>, as well as possibly to the embedded viewer's <tt>EmbeddedContext</tt>. It also means that
- * the requirements for writing an object store are more easily expressed: support the three built-in queries, plus any
- * others.
+ * The principle reason for this is to reduce the size of the API from the
+ * {@link DomainObjectContainer} to <tt>RuntimeContext</tt>, as well as possibly
+ * to the embedded viewer's <tt>EmbeddedContext</tt>. It also means that the
+ * requirements for writing an object store are more easily expressed: support
+ * the three built-in queries, plus any others.
  * 
  * <p>
- * Note also that the {@link QueryFindByPattern} isn't actually serializable (because it references an arbitrary pojo).
+ * Note also that the {@link QueryFindByPattern} isn't actually serializable
+ * (because it references an arbitrary pojo).
  */
 public abstract class QueryBuiltInAbstract<T> extends QueryAbstract<T> {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryDefault.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryDefault.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryDefault.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryDefault.java Sat Jan 21 18:30:32 2012
@@ -24,27 +24,28 @@ import java.util.HashMap;
 import java.util.Map;
 
 /**
- * Default implementation of {@link Query} that supports parameter/argument values, along with a query name.
+ * Default implementation of {@link Query} that supports parameter/argument
+ * values, along with a query name.
  */
 public class QueryDefault<T> extends QueryAbstract<T> {
 
     private static final long serialVersionUID = 1L;
 
     /**
-     * Convenience factory method, preferable to {@link #QueryDefault(Class, String, Object...) constructor} because
-     * will automatically genericize.
+     * Convenience factory method, preferable to
+     * {@link #QueryDefault(Class, String, Object...) constructor} because will
+     * automatically genericize.
      */
-    public static <Q> QueryDefault<Q> create(final Class<Q> resultType, final String queryName,
-        final Object... paramArgs) {
+    public static <Q> QueryDefault<Q> create(final Class<Q> resultType, final String queryName, final Object... paramArgs) {
         return new QueryDefault<Q>(resultType, queryName, paramArgs);
     }
 
     /**
-     * Convenience factory method, preferable to {@link #QueryDefault(Class, String, Map) constructor} because will
+     * Convenience factory method, preferable to
+     * {@link #QueryDefault(Class, String, Map) constructor} because will
      * automatically genericize.
      */
-    public static <Q> QueryDefault<Q> create(final Class<Q> resultType, final String queryName,
-        final Map<String, Object> argumentsByParameterName) {
+    public static <Q> QueryDefault<Q> create(final Class<Q> resultType, final String queryName, final Map<String, Object> argumentsByParameterName) {
         return new QueryDefault<Q>(resultType, queryName, argumentsByParameterName);
     }
 
@@ -82,8 +83,7 @@ public class QueryDefault<T> extends Que
         this(resultType, queryName, asMap(paramArgs));
     }
 
-    public QueryDefault(final Class<T> resultType, final String queryName,
-        final Map<String, Object> argumentsByParameterName) {
+    public QueryDefault(final Class<T> resultType, final String queryName, final Map<String, Object> argumentsByParameterName) {
         super(resultType);
         this.queryName = queryName;
         this.argumentsByParameterName = argumentsByParameterName;

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryFindAllInstances.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryFindAllInstances.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryFindAllInstances.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/query/QueryFindAllInstances.java Sat Jan 21 18:30:32 2012
@@ -22,8 +22,9 @@ package org.apache.isis.applib.query;
 import java.io.Serializable;
 
 /**
- * Although implements {@link Query} and thus is intended to be (and indeed is) {@link Serializable}, it will be
- * converted into a <tt>PersistenceQuery</tt> in the runtime for remoting purposes.
+ * Although implements {@link Query} and thus is intended to be (and indeed is)
+ * {@link Serializable}, it will be converted into a <tt>PersistenceQuery</tt>
+ * in the runtime for remoting purposes.
  * 
  * <p>
  * See discussion in {@link QueryBuiltInAbstract} for further details.

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/security/UserMemento.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/security/UserMemento.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/security/UserMemento.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/security/UserMemento.java Sat Jan 21 18:30:32 2012
@@ -27,7 +27,8 @@ import org.apache.isis.applib.annotation
 import org.apache.isis.applib.annotation.NotPersistable;
 
 /**
- * Details, obtained from the container, about the user and his roles. Read-only.
+ * Details, obtained from the container, about the user and his roles.
+ * Read-only.
  */
 @NotPersistable
 public final class UserMemento {
@@ -110,7 +111,8 @@ public final class UserMemento {
     }
 
     /**
-     * Determines if the user fulfills the specified role. Roles are compared lexically by role name.
+     * Determines if the user fulfills the specified role. Roles are compared
+     * lexically by role name.
      */
     public boolean hasRole(final String roleName) {
         for (final RoleMemento role : roles) {

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/Snapshottable.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/Snapshottable.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/Snapshottable.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/Snapshottable.java Sat Jan 21 18:30:32 2012
@@ -20,7 +20,8 @@
 package org.apache.isis.applib.snapshot;
 
 /**
- * Marker interface for domain objects that can be snapshot using <xx>XmlSnapshot</tt>.
+ * Marker interface for domain objects that can be snapshot using
+ * <xx>XmlSnapshot</tt>.
  */
 public interface Snapshottable {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/SnapshottableWithInclusions.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/SnapshottableWithInclusions.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/SnapshottableWithInclusions.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/snapshot/SnapshottableWithInclusions.java Sat Jan 21 18:30:32 2012
@@ -22,8 +22,8 @@ package org.apache.isis.applib.snapshot;
 import java.util.List;
 
 /**
- * Optional subinterface of {@link Snapshottable}s, used by <tt>XmlSnapshot</tt> to automatically include additional
- * paths within the snapshot.
+ * Optional subinterface of {@link Snapshottable}s, used by <tt>XmlSnapshot</tt>
+ * to automatically include additional paths within the snapshot.
  */
 public interface SnapshottableWithInclusions extends Snapshottable {
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/AbstractSpecification.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/AbstractSpecification.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/AbstractSpecification.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/AbstractSpecification.java Sat Jan 21 18:30:32 2012
@@ -27,12 +27,15 @@ import java.lang.reflect.Method;
  * <p>
  * Provides two main features:
  * <ul>
- * <li>first, is type-safe (with invalid type being either ignored or constituting a failure), and
- * <li>second, checks for nulls (with a null either being ignore or again constituting a failure)
+ * <li>first, is type-safe (with invalid type being either ignored or
+ * constituting a failure), and
+ * <li>second, checks for nulls (with a null either being ignore or again
+ * constituting a failure)
  * </ul>
  * 
  * <p>
- * Implementation note: inspired by (borrowed code from) Hamcrest's <tt>TypeSafeMatcher</tt>.
+ * Implementation note: inspired by (borrowed code from) Hamcrest's
+ * <tt>TypeSafeMatcher</tt>.
  */
 public abstract class AbstractSpecification<T> implements Specification {
 
@@ -57,8 +60,7 @@ public abstract class AbstractSpecificat
     }
 
     private static boolean isSatisfiesSafelyMethod(final Method method) {
-        return method.getName().equals("satisfiesSafely") && method.getParameterTypes().length == 1
-            && !method.isSynthetic();
+        return method.getName().equals("satisfiesSafely") && method.getParameterTypes().length == 1 && !method.isSynthetic();
     }
 
     private final Class<?> expectedType;
@@ -76,7 +78,8 @@ public abstract class AbstractSpecificat
     }
 
     /**
-     * Checks not null and is correct type, and delegates to {@link #satisfiesSafely(Object)}.
+     * Checks not null and is correct type, and delegates to
+     * {@link #satisfiesSafely(Object)}.
      */
     @Override
     @SuppressWarnings({ "unchecked" })
@@ -92,11 +95,12 @@ public abstract class AbstractSpecificat
     }
 
     /**
-     * If <tt>null</tt> then satisfied, otherwise is reason why the specification is not satisfied.
+     * If <tt>null</tt> then satisfied, otherwise is reason why the
+     * specification is not satisfied.
      * 
      * <p>
-     * Subclasses should implement this. The item will already have been checked for the specific type and will never be
-     * null.
+     * Subclasses should implement this. The item will already have been checked
+     * for the specific type and will never be null.
      */
     public abstract String satisfiesSafely(T obj);
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/Specification.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/Specification.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/Specification.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/Specification.java Sat Jan 21 18:30:32 2012
@@ -20,12 +20,14 @@
 package org.apache.isis.applib.spec;
 
 /**
- * An implementation of the <i>Specification</i> pattern, as described in Eric Evans' <i>Domain Driven Design</i>, p224.
+ * An implementation of the <i>Specification</i> pattern, as described in Eric
+ * Evans' <i>Domain Driven Design</i>, p224.
  */
 public interface Specification {
 
     /**
-     * If <tt>null</tt> then satisfied, otherwise is reason why the specification is not satisfied.
+     * If <tt>null</tt> then satisfied, otherwise is reason why the
+     * specification is not satisfied.
      */
     public String satisfies(Object obj);
 }

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationAnd.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationAnd.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationAnd.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationAnd.java Sat Jan 21 18:30:32 2012
@@ -25,8 +25,9 @@ import org.apache.isis.applib.util.Reaso
  * Adapter to make it easy to perform boolean algebra on {@link Specification}s.
  * 
  * <p>
- * Subclasses represent the intersection of multiple {@link Specification}s. An implementation should instantiate the
- * {@link Specification}s to be satisfied in its constructor.
+ * Subclasses represent the intersection of multiple {@link Specification}s. An
+ * implementation should instantiate the {@link Specification}s to be satisfied
+ * in its constructor.
  * 
  * <p>
  * For example:

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationNot.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationNot.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationNot.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationNot.java Sat Jan 21 18:30:32 2012
@@ -24,8 +24,9 @@ package org.apache.isis.applib.spec;
  * 
  * <p>
  * <p>
- * Subclasses represent the logical inverse of a {@link Specification}s. An implementation should instantiate the
- * {@link Specification}s to be satisfied in its constructor.
+ * Subclasses represent the logical inverse of a {@link Specification}s. An
+ * implementation should instantiate the {@link Specification}s to be satisfied
+ * in its constructor.
  * 
  * <p>
  * For example:

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationOr.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationOr.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationOr.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/spec/SpecificationOr.java Sat Jan 21 18:30:32 2012
@@ -25,8 +25,9 @@ import org.apache.isis.applib.util.Reaso
  * Adapter to make it easy to perform boolean algebra on {@link Specification}s.
  * 
  * <p>
- * Subclasses represent the conjunction of multiple {@link Specification}s. An implementation should instantiate the
- * {@link Specification}s to be satisfied in its constructor.
+ * Subclasses represent the conjunction of multiple {@link Specification}s. An
+ * implementation should instantiate the {@link Specification}s to be satisfied
+ * in its constructor.
  * 
  * <p>
  * For example:

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/ReasonBuffer.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/ReasonBuffer.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/ReasonBuffer.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/ReasonBuffer.java Sat Jan 21 18:30:32 2012
@@ -20,12 +20,13 @@
 package org.apache.isis.applib.util;
 
 /**
- * Helper class to create properly concatenated reason string for use in method that return {@link String}s with
- * reasons.
+ * Helper class to create properly concatenated reason string for use in method
+ * that return {@link String}s with reasons.
  * 
  * <p>
- * If no reasons are specified {@link #getReason()} will return <code>null</code> , otherwise it will return a
- * {@link String} with all the valid reasons concatenated with a semi-colon separating each one.
+ * If no reasons are specified {@link #getReason()} will return
+ * <code>null</code> , otherwise it will return a {@link String} with all the
+ * valid reasons concatenated with a semi-colon separating each one.
  * 
  * <p>
  * An alternative is to use the {@link Reasons} class.
@@ -46,7 +47,8 @@ public class ReasonBuffer {
     }
 
     /**
-     * Append a reason to the list of existing reasons if the condition flag is true.
+     * Append a reason to the list of existing reasons if the condition flag is
+     * true.
      */
     public void appendOnCondition(final boolean condition, final String reason) {
         if (condition) {
@@ -55,7 +57,8 @@ public class ReasonBuffer {
     }
 
     /**
-     * Return the combined set of reasons, or <code>null</code> if there are none.
+     * Return the combined set of reasons, or <code>null</code> if there are
+     * none.
      */
     public String getReason() {
         return reasonBuffer.length() == 0 ? null : reasonBuffer.toString();

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/TitleBuffer.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/TitleBuffer.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/TitleBuffer.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/util/TitleBuffer.java Sat Jan 21 18:30:32 2012
@@ -23,18 +23,21 @@ import java.lang.reflect.InvocationTarge
 import java.lang.reflect.Method;
 
 /**
- * Title buffer is a utility class to help produce titles for objects without having to add lots of guard code. It
- * provides two basic method: one to concatenate a title to the buffer; another to append a title with a joiner string,
- * taking care adding in necessary spaces. The benefits of using this class is that null references are safely ignored
- * (rather than appearing as 'null'), and joiners (a space by default) are only added when needed.
+ * Title buffer is a utility class to help produce titles for objects without
+ * having to add lots of guard code. It provides two basic method: one to
+ * concatenate a title to the buffer; another to append a title with a joiner
+ * string, taking care adding in necessary spaces. The benefits of using this
+ * class is that null references are safely ignored (rather than appearing as
+ * 'null'), and joiners (a space by default) are only added when needed.
  */
 public class TitleBuffer {
     private static final String SPACE = " ";
 
     /**
-     * Determines if the specified object's title (from its <code>toString</code> method) is empty. Will return true if
-     * either: the specified reference is null; the object's <code>toString</code> method returns null; or if the
-     * <code>toString</code> returns an empty string.
+     * Determines if the specified object's title (from its
+     * <code>toString</code> method) is empty. Will return true if either: the
+     * specified reference is null; the object's <code>toString</code> method
+     * returns null; or if the <code>toString</code> returns an empty string.
      */
     public static boolean isEmpty(final Object object) {
         final String title = titleFor(object);
@@ -42,8 +45,8 @@ public class TitleBuffer {
     }
 
     /**
-     * Reflectively run the <tt>String title()</tt> method if it exists, else fall back to the <tt>toString()</tt>
-     * method.
+     * Reflectively run the <tt>String title()</tt> method if it exists, else
+     * fall back to the <tt>toString()</tt> method.
      */
     private static String titleFor(final Object object) {
         if (object == null) {
@@ -68,8 +71,8 @@ public class TitleBuffer {
     }
 
     /**
-     * Determines if the specified text is empty. Will return true if either: the specified reference is null; or if the
-     * reference is an empty string.
+     * Determines if the specified text is empty. Will return true if either:
+     * the specified reference is null; or if the reference is an empty string.
      */
     public static boolean isEmpty(final String text) {
         return text == null || text.equals("");
@@ -131,13 +134,15 @@ public class TitleBuffer {
     }
 
     /**
-     * Appends the title of the specified object, or the specified text if the objects title is null or empty. Prepends
-     * a space if there is already some text in this title object.
+     * Appends the title of the specified object, or the specified text if the
+     * objects title is null or empty. Prepends a space if there is already some
+     * text in this title object.
      * 
      * @param object
      *            the object whose title is to be appended to this title.
      * @param defaultValue
-     *            a textual value to be used if the object's title is null or empty.
+     *            a textual value to be used if the object's title is null or
+     *            empty.
      * @return a reference to the called object (itself).
      */
     public TitleBuffer append(final Object object, final String defaultValue) {
@@ -150,7 +155,8 @@ public class TitleBuffer {
     }
 
     /**
-     * Appends a space (if there is already some text in this title object) and then the specified text.
+     * Appends a space (if there is already some text in this title object) and
+     * then the specified text.
      * 
      * @return a reference to the called object (itself).
      */
@@ -162,8 +168,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Appends the joining string and the title of the specified object (from its <code>toString</code> method). If the
-     * object is empty then nothing will be appended.
+     * Appends the joining string and the title of the specified object (from
+     * its <code>toString</code> method). If the object is empty then nothing
+     * will be appended.
      * 
      * @see #isEmpty(Object)
      */
@@ -176,10 +183,13 @@ public class TitleBuffer {
     }
 
     /**
-     * Append the <code>joiner</code> text, a space, and the title of the specified ObjectAdapter (<code>object</code>)
-     * (got by calling the objects title() method) to the text of this TitleString object. If the title of the specified
-     * object is null then use the <code>defaultValue</code> text. If both the objects title and the default value are
-     * null or equate to a zero-length string then no text will be appended ; not even the joiner text.
+     * Append the <code>joiner</code> text, a space, and the title of the
+     * specified ObjectAdapter (<code>object</code>) (got by calling the objects
+     * title() method) to the text of this TitleString object. If the title of
+     * the specified object is null then use the <code>defaultValue</code> text.
+     * If both the objects title and the default value are null or equate to a
+     * zero-length string then no text will be appended ; not even the joiner
+     * text.
      * 
      * @param joiner
      *            text to append before the title
@@ -200,8 +210,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Appends the joiner text, a space, and the text to the text of this TitleString object. If no text yet exists in
-     * the object then the joiner text and space are omitted.
+     * Appends the joiner text, a space, and the text to the text of this
+     * TitleString object. If no text yet exists in the object then the joiner
+     * text and space are omitted.
      * 
      * @return a reference to the called object (itself).
      */
@@ -220,8 +231,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Append a space to the text of this TitleString object if, and only if, there is some existing text i.e., a space
-     * is only added to existing text and will not create a text entry consisting of only one space.
+     * Append a space to the text of this TitleString object if, and only if,
+     * there is some existing text i.e., a space is only added to existing text
+     * and will not create a text entry consisting of only one space.
      * 
      * @return a reference to the called object (itself).
      */
@@ -238,8 +250,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Concatenate the the title value (the result of calling an objects label() method) to this TitleString object. If
-     * the value is null the no text is added.
+     * Concatenate the the title value (the result of calling an objects label()
+     * method) to this TitleString object. If the value is null the no text is
+     * added.
      * 
      * @param object
      *            the ObjectAdapter to get a title from
@@ -251,8 +264,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Concatenate the the title value (the result of calling an objects label() method), or the specified default value
-     * if the title is equal to null or is empty, to this TitleString object.
+     * Concatenate the the title value (the result of calling an objects label()
+     * method), or the specified default value if the title is equal to null or
+     * is empty, to this TitleString object.
      * 
      * @param object
      *            the ObjectAdapter to get a title from
@@ -271,7 +285,8 @@ public class TitleBuffer {
     }
 
     /**
-     * Concatenate the specified text on to the end of the text of this TitleString.
+     * Concatenate the specified text on to the end of the text of this
+     * TitleString.
      * 
      * @param text
      *            text to append
@@ -283,8 +298,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Concatenate the joiner text and the text to the text of this TitleString object. If no text yet exists in
-     * the object then the joiner text is omitted.
+     * Concatenate the joiner text and the text to the text of this TitleString
+     * object. If no text yet exists in the object then the joiner text is
+     * omitted.
      * 
      * @return a reference to the called object (itself).
      */
@@ -297,8 +313,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Concatenate the joiner text and the title of the object to the text of this TitleString object. If no object yet exists in
-     * the object then the joiner text is omitted.
+     * Concatenate the joiner text and the title of the object to the text of
+     * this TitleString object. If no object yet exists in the object then the
+     * joiner text is omitted.
      * 
      * @return a reference to the called object (itself).
      */
@@ -311,8 +328,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Concatenate the joiner text and the title of the object to the text of this TitleString object. If no object yet exists in
-     * the object then defaultValue is used instead.
+     * Concatenate the joiner text and the title of the object to the text of
+     * this TitleString object. If no object yet exists in the object then
+     * defaultValue is used instead.
      * 
      * @return a reference to the called object (itself).
      */
@@ -336,8 +354,9 @@ public class TitleBuffer {
     }
 
     /**
-     * Truncates this title so it has a maximum number of words. Spaces are used to determine words, thus two spaces in
-     * a title will cause two words to be mistakenly identified.
+     * Truncates this title so it has a maximum number of words. Spaces are used
+     * to determine words, thus two spaces in a title will cause two words to be
+     * mistakenly identified.
      * 
      * @param noWords
      *            the number of words to show
@@ -357,7 +376,8 @@ public class TitleBuffer {
             pos++;
         }
         if (pos < title.length()) {
-            title.setLength(pos - 1); // string.delete(pos - 1, string.length());
+            title.setLength(pos - 1); // string.delete(pos - 1,
+                                      // string.length());
             title.append("...");
         }
         return this;

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Color.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Color.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Color.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Color.java Sat Jan 21 18:30:32 2012
@@ -22,11 +22,12 @@ package org.apache.isis.applib.value;
 import org.apache.isis.applib.annotation.Value;
 
 /**
- * Color is simple numerical representation of a color using the brightness of red, green and blue (RGB) components.
+ * Color is simple numerical representation of a color using the brightness of
+ * red, green and blue (RGB) components.
  * 
  * <p>
- * Where there is no basic colors (RGB all equal 0) then you get black; where each color is at maximum (RGB all equal
- * 255) you get white.
+ * Where there is no basic colors (RGB all equal 0) then you get black; where
+ * each color is at maximum (RGB all equal 255) you get white.
  */
 @Value(semanticsProviderName = "org.apache.isis.core.progmodel.facets.value.color.ColorValueSemanticsProvider")
 public class Color extends Magnitude<Color> {
@@ -47,7 +48,8 @@ public class Color extends Magnitude<Col
     }
 
     /**
-     * returns true if the number of this object has the same value as the specified number
+     * returns true if the number of this object has the same value as the
+     * specified number
      */
     @Override
     public boolean isEqualTo(final Color number) {

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Date.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Date.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Date.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Date.java Sat Jan 21 18:30:32 2012
@@ -19,15 +19,14 @@
 
 package org.apache.isis.applib.value;
 
+import org.apache.isis.applib.Defaults;
+import org.apache.isis.applib.annotation.Value;
+import org.apache.isis.applib.clock.Clock;
 import org.joda.time.DateTime;
 import org.joda.time.DateTimeFieldType;
 import org.joda.time.Period;
 import org.joda.time.format.DateTimeFormat;
 
-import org.apache.isis.applib.Defaults;
-import org.apache.isis.applib.annotation.Value;
-import org.apache.isis.applib.clock.Clock;
-
 /**
  * Value object representing a date (not time) value.
  * 
@@ -49,7 +48,7 @@ public class Date extends Magnitude<Date
      * Create a Date object for today's date.
      */
     public Date() {
-        DateTime time = Clock.getTimeAsDateTime().withTime(0, 0, 0, 0);
+        final DateTime time = Clock.getTimeAsDateTime().withTime(0, 0, 0, 0);
         date = new DateTime(time, Defaults.getTimeZone());
     }
 
@@ -62,7 +61,8 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Create a Date object based on the specified Java date object. The time portion of the Java date is disposed of.
+     * Create a Date object based on the specified Java date object. The time
+     * portion of the Java date is disposed of.
      */
     public Date(final java.util.Date date) {
         this.date = new DateTime(date.getTime(), Defaults.getTimeZone());
@@ -76,21 +76,22 @@ public class Date extends Magnitude<Date
         this.date = new DateTime(date);
     }
 
-    private DateTime newDateTime(int year, int month, int day) {
+    private DateTime newDateTime(final int year, final int month, final int day) {
         return new DateTime(year, month, day, 0, 0, 0, 0, Defaults.getTimeZone());
     }
 
     protected Date createDate(final DateTime date) {
-        Date newDate = new Date(date);
+        final Date newDate = new Date(date);
         return newDate;
     }
 
     /**
-     * Add the specified days, years and months to this date value and return a new date object containing the result.
+     * Add the specified days, years and months to this date value and return a
+     * new date object containing the result.
      */
     public Date add(final int years, final int months, final int days) {
-        Period add = new Period(years, months, 0, days, 0, 0, 0, 0);
-        DateTime newDate = date.plus(add);
+        final Period add = new Period(years, months, 0, days, 0, 0, 0, 0);
+        final DateTime newDate = date.plus(add);
         return new Date(newDate);
     }
 
@@ -112,7 +113,7 @@ public class Date extends Magnitude<Date
      * @see java.util.Date
      */
     public java.util.Date dateValue() {
-        java.util.Date javaDate = date.toDate();
+        final java.util.Date javaDate = date.toDate();
         return javaDate;
     }
 
@@ -125,7 +126,8 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Calculates, and returns, a date representing the last day of the month relative to the current date.
+     * Calculates, and returns, a date representing the last day of the month
+     * relative to the current date.
      * 
      * @author Joshua Cassidy
      */
@@ -161,8 +163,8 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Calculates, and returns, an int representing the day of the week relative to the current date. With Mon = 0
-     * through to Sun = 6
+     * Calculates, and returns, an int representing the day of the week relative
+     * to the current date. With Mon = 0 through to Sun = 6
      * 
      * @author Joshua Cassidy
      */
@@ -185,7 +187,8 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Returns true if the date of this object has the same value as the specified date
+     * Returns true if the date of this object has the same value as the
+     * specified date
      */
     @Override
     public boolean isEqualTo(final Date date) {
@@ -193,7 +196,8 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Returns true if the time of this object is earlier than the specified time
+     * Returns true if the time of this object is earlier than the specified
+     * time
      */
     @Override
     public boolean isLessThan(final Date date) {
@@ -206,39 +210,40 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Determines if this date and the specified date represent the same day of the month, eg both dates are for the
-     * 3rd.
+     * Determines if this date and the specified date represent the same day of
+     * the month, eg both dates are for the 3rd.
      */
     public boolean sameDayOfMonthAs(final Date as) {
         return sameAs(as, DateTimeFieldType.dayOfMonth());
     }
 
     /**
-     * Determines if this date and the specified date represent the same day of the week, eg both dates are on a
-     * Tuesday.
+     * Determines if this date and the specified date represent the same day of
+     * the week, eg both dates are on a Tuesday.
      */
     public boolean sameDayOfWeekAs(final Date as) {
         return sameAs(as, DateTimeFieldType.dayOfWeek());
     }
 
     /**
-     * Determines if this date and the specified date represent the same day of the year, eg both dates are for the
-     * 108th day of the year.
+     * Determines if this date and the specified date represent the same day of
+     * the year, eg both dates are for the 108th day of the year.
      */
     public boolean sameDayOfYearAs(final Date as) {
         return sameAs(as, DateTimeFieldType.dayOfYear());
     }
 
     /**
-     * Determines if this date and the specified date represent the same month, eg both dates are for the March.
+     * Determines if this date and the specified date represent the same month,
+     * eg both dates are for the March.
      */
     public boolean sameMonthAs(final Date as) {
         return sameAs(as, DateTimeFieldType.monthOfYear());
     }
 
     /**
-     * Determines if this date and the specified date represent the same week in the year, eg both dates are the for the
-     * 18th week of the year.
+     * Determines if this date and the specified date represent the same week in
+     * the year, eg both dates are the for the 18th week of the year.
      */
     public boolean sameWeekAs(final Date as) {
         return sameAs(as, DateTimeFieldType.weekOfWeekyear());
@@ -252,21 +257,24 @@ public class Date extends Magnitude<Date
     }
 
     /**
-     * Calculates, and returns, a date representing the first day of the month relative to the current date.
+     * Calculates, and returns, a date representing the first day of the month
+     * relative to the current date.
      */
     public Date startOfMonth() {
         return new Date(date.dayOfMonth().withMinimumValue());
     }
 
     /**
-     * Calculates, and returns, a date representing the first day of the week relative to the current date.
+     * Calculates, and returns, a date representing the first day of the week
+     * relative to the current date.
      */
     public Date startOfWeek() {
         return new Date(date.dayOfWeek().withMinimumValue());
     }
 
     /**
-     * Calculates, and returns, a date representing the first day of the year relative to the current date.
+     * Calculates, and returns, a date representing the first day of the year
+     * relative to the current date.
      */
     public Date startOfYear() {
         return new Date(date.dayOfYear().withMinimumValue());

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/DateTime.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/DateTime.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/DateTime.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/DateTime.java Sat Jan 21 18:30:32 2012
@@ -23,16 +23,15 @@ import java.util.Calendar;
 import java.util.Date;
 import java.util.TimeZone;
 
-import org.joda.time.DateTimeZone;
-import org.joda.time.Period;
-
 import org.apache.isis.applib.Defaults;
 import org.apache.isis.applib.annotation.Value;
 import org.apache.isis.applib.clock.Clock;
+import org.joda.time.DateTimeZone;
+import org.joda.time.Period;
 
 /**
- * Value object representing a date and time value. By default, the time is initialised to the current time, unless
- * otherwise specified.
+ * Value object representing a date and time value. By default, the time is
+ * initialised to the current time, unless otherwise specified.
  */
 @Value(semanticsProviderName = "org.apache.isis.core.progmodel.facets.value.datetime.DateTimeValueSemanticsProvider")
 public class DateTime extends Magnitude<DateTime> {
@@ -44,14 +43,14 @@ public class DateTime extends Magnitude<
      * Create a Time object for storing a timeStamp set to the current time.
      */
     public DateTime() {
-        org.joda.time.DateTime d = new org.joda.time.DateTime(Clock.getTime(), Defaults.getTimeZone());
+        final org.joda.time.DateTime d = new org.joda.time.DateTime(Clock.getTime(), Defaults.getTimeZone());
         // dateTime = d.secondOfMinute().setCopy(0);
         dateTime = d;
     }
 
     /**
-     * Create a DateTime from the provided java.util.Date, assuming that the date is in UTC. If not, see {@link
-     * DateTime#}
+     * Create a DateTime from the provided java.util.Date, assuming that the
+     * date is in UTC. If not, see {@link DateTime#}
      * 
      * @param date
      */
@@ -63,8 +62,8 @@ public class DateTime extends Magnitude<
         this.dateTime = new org.joda.time.DateTime(millisSinceEpoch, Defaults.getTimeZone());
     }
 
-    public DateTime(final Date date, TimeZone timeZone) {
-        DateTimeZone tz = DateTimeZone.forTimeZone(timeZone);
+    public DateTime(final Date date, final TimeZone timeZone) {
+        final DateTimeZone tz = DateTimeZone.forTimeZone(timeZone);
         this.dateTime = new org.joda.time.DateTime(date, tz);
     }
 
@@ -76,14 +75,12 @@ public class DateTime extends Magnitude<
      * Creates a DateTime on the specified day/month/year, with the current time
      */
     public DateTime(final int year, final int month, final int day) {
-        Time time = new Time();
+        final Time time = new Time();
         // this(year, month, day, time.getHour(), time.getMinute(), 0);
 
         // replace below with something more like the above..
         checkTime(year, month, day, 0, 0);
-        dateTime =
-            new org.joda.time.DateTime(year, month, day, time.getHour(), time.getMinute(), time.getSecond(), 0,
-                Defaults.getTimeZone());
+        dateTime = new org.joda.time.DateTime(year, month, day, time.getHour(), time.getMinute(), time.getSecond(), 0, Defaults.getTimeZone());
     }
 
     /**
@@ -94,7 +91,8 @@ public class DateTime extends Magnitude<
     }
 
     /**
-     * Create a Date object set to the specified day, month, year, hour, minute, and second.
+     * Create a Date object set to the specified day, month, year, hour, minute,
+     * and second.
      */
     public DateTime(final int year, final int month, final int day, final int hour, final int minute, final int second) {
         checkTime(year, month, day, hour, minute);
@@ -106,7 +104,7 @@ public class DateTime extends Magnitude<
             throw new IllegalArgumentException("Month must be in the range 1 - 12 inclusive " + month);
         }
 
-        org.joda.time.DateTime dateTime = new org.joda.time.DateTime(year, month, 1, 0, 0, 0, 0);
+        final org.joda.time.DateTime dateTime = new org.joda.time.DateTime(year, month, 1, 0, 0, 0, 0);
         final int lastDayOfMonth = dateTime.dayOfMonth().getMaximumValue();
 
         if ((day < 1) || (day > lastDayOfMonth)) {
@@ -126,15 +124,15 @@ public class DateTime extends Magnitude<
      * Add the specified time period to this date value.
      */
     public DateTime add(final int years, final int months, final int days, final int hours, final int minutes) {
-        Period period = new Period(years, months, 0, days, hours, minutes, 0, 0);
-        org.joda.time.DateTime dateTime = this.dateTime.plus(period);
+        final Period period = new Period(years, months, 0, days, hours, minutes, 0, 0);
+        final org.joda.time.DateTime dateTime = this.dateTime.plus(period);
         return new DateTime(dateTime);
     }
 
     /**
      * Add the specified days, years and months to this date value.
      */
-    public DateTime add(int years, int months, int days) {
+    public DateTime add(final int years, final int months, final int days) {
         return add(years, months, days, 0, 0);
     }
 
@@ -147,7 +145,8 @@ public class DateTime extends Magnitude<
     }
 
     /**
-     * Be careful - the TimeZone of the java.util.Date is based on the system default.
+     * Be careful - the TimeZone of the java.util.Date is based on the system
+     * default.
      */
     public java.util.Date dateValue() {
         return new Date(dateTime.getMillis());
@@ -194,7 +193,8 @@ public class DateTime extends Magnitude<
     }
 
     /**
-     * A DateTime that represents the start of the current month. Time is preserved.
+     * A DateTime that represents the start of the current month. Time is
+     * preserved.
      */
     public DateTime startOfMonth() {
         return new DateTime(dateTime.withDayOfMonth(1));
@@ -208,7 +208,8 @@ public class DateTime extends Magnitude<
     }
 
     /**
-     * returns true if the time stamp of this object has the same value as the specified time
+     * returns true if the time stamp of this object has the same value as the
+     * specified time
      */
     @Override
     public boolean isEqualTo(final DateTime timeStamp) {
@@ -216,38 +217,39 @@ public class DateTime extends Magnitude<
     }
 
     /**
-     * returns true if the timeStamp of this object is earlier than the specified timeStamp
+     * returns true if the timeStamp of this object is earlier than the
+     * specified timeStamp
      */
     @Override
     public boolean isLessThan(final DateTime timeStamp) {
         return dateTime.isBefore((timeStamp).dateTime);
     }
 
-    public boolean isSameDayAs(DateTime dateTime2) {
+    public boolean isSameDayAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : getDayOfYear() == dateTime2.getDayOfYear();
     }
 
-    public boolean sameDayOfWeekAs(DateTime dateTime2) {
+    public boolean sameDayOfWeekAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : dateTime.getDayOfWeek() == dateTime2.dateTime.getDayOfWeek();
     }
 
-    public boolean sameDayOfMonthAs(DateTime dateTime2) {
+    public boolean sameDayOfMonthAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : dateTime.getDayOfMonth() == dateTime2.dateTime.getDayOfMonth();
     }
 
-    public boolean sameDayOfYearAs(DateTime dateTime2) {
+    public boolean sameDayOfYearAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : dateTime.getDayOfYear() == dateTime2.dateTime.getDayOfYear();
     }
 
-    public boolean sameWeekAs(DateTime dateTime2) {
+    public boolean sameWeekAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : dateTime.getWeekOfWeekyear() == dateTime2.dateTime.getWeekOfWeekyear();
     }
 
-    public boolean sameMonthAs(DateTime dateTime2) {
+    public boolean sameMonthAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : getMonth() == dateTime2.getMonth();
     }
 
-    public boolean sameYearAs(DateTime dateTime2) {
+    public boolean sameYearAs(final DateTime dateTime2) {
         return dateTime2 == null ? false : getYear() == dateTime2.getYear();
     }
 
@@ -298,7 +300,8 @@ public class DateTime extends Magnitude<
 
     @Override
     public String toString() {
-        // return getYear() + "-" + getMonth() + "-" + getDay() + " " + getHour() + ":" + getMinute();
+        // return getYear() + "-" + getMonth() + "-" + getDay() + " " +
+        // getHour() + ":" + getMinute();
         return String.format("%04d-%02d-%02d %02d:%02d", getYear(), getMonth(), getDay(), getHour(), getMinute());
     }
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Percentage.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Percentage.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Percentage.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Percentage.java Sat Jan 21 18:30:32 2012
@@ -115,7 +115,8 @@ public class Percentage extends Magnitud
 
     @Override
     public int hashCode() {
-        // multiply by 100 just in case the percentage is being stored as 0.0 to 1.0
+        // multiply by 100 just in case the percentage is being stored as 0.0 to
+        // 1.0
         return (int) (floatValue() * 100);
     }
 

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Time.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Time.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Time.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/Time.java Sat Jan 21 18:30:32 2012
@@ -19,15 +19,14 @@
 
 package org.apache.isis.applib.value;
 
+import org.apache.isis.applib.Defaults;
+import org.apache.isis.applib.annotation.Value;
+import org.apache.isis.applib.clock.Clock;
 import org.joda.time.DateTime;
 import org.joda.time.DateTimeZone;
 import org.joda.time.Period;
 import org.joda.time.format.DateTimeFormat;
 
-import org.apache.isis.applib.Defaults;
-import org.apache.isis.applib.annotation.Value;
-import org.apache.isis.applib.clock.Clock;
-
 /**
  * Value object representing a time value.
  * 
@@ -35,11 +34,12 @@ import org.apache.isis.applib.clock.Cloc
  * TODO: other methods to implement:
  * <ul>
  * <li>comparison methods</li>
- * <li>sameHourAs() hour ==hour sameMinuteAs() minutes = minutes sameTimeAs(hour, min) hour == hour & minutes == minutes
- * </li>
- * <li>withinNextTimePeriod(int hours, int minutes); withinTimePeriod(Date d, int hours, int minutes)</li>
- * <li>withinPreviousTimePeriod(int hours, int minutes); d.hour >= this.hour >= d.hour + hours & d.minutes >=
- * this.minutes >= d.minutes + minutes</li>
+ * <li>sameHourAs() hour ==hour sameMinuteAs() minutes = minutes
+ * sameTimeAs(hour, min) hour == hour & minutes == minutes</li>
+ * <li>withinNextTimePeriod(int hours, int minutes); withinTimePeriod(Date d,
+ * int hours, int minutes)</li>
+ * <li>withinPreviousTimePeriod(int hours, int minutes); d.hour >= this.hour >=
+ * d.hour + hours & d.minutes >= this.minutes >= d.minutes + minutes</li>
  * </ul>
  */
 @Value(semanticsProviderName = "org.apache.isis.core.progmodel.facets.value.time.TimeValueSemanticsProvider")
@@ -56,7 +56,7 @@ public class Time extends Magnitude<Time
      * Create a Time object set to the current time.
      */
     public Time() {
-        DateTime dateTime = Clock.getTimeAsDateTime();
+        final DateTime dateTime = Clock.getTimeAsDateTime();
         time = dateTime.withDate(1970, 1, 1); // Epoch is 1970-01-01
     }
 
@@ -65,13 +65,14 @@ public class Time extends Magnitude<Time
     }
 
     /**
-     * Create a Time object for storing a time with the time set to the specified hours and minutes.
+     * Create a Time object for storing a time with the time set to the
+     * specified hours and minutes.
      */
     public Time(final int hour, final int minute) {
         this(hour, minute, 0);
     }
 
-    public Time(int hour, int minute, int second) {
+    public Time(final int hour, final int minute, final int second) {
         time = time(hour, minute, second);
     }
 
@@ -81,7 +82,8 @@ public class Time extends Magnitude<Time
     }
 
     /**
-     * Create a Time object for storing a time with the time set to the specified time of the Java Date object.
+     * Create a Time object for storing a time with the time set to the
+     * specified time of the Java Date object.
      */
     public Time(final java.sql.Date date) {
 
@@ -96,12 +98,13 @@ public class Time extends Magnitude<Time
      */
 
     public Time(final java.util.Date date, final DateTimeZone dateTimeZone) {
-        DateTime DateTime = new DateTime(date.getTime(), dateTimeZone);
+        final DateTime DateTime = new DateTime(date.getTime(), dateTimeZone);
         this.time = DateTime.secondOfMinute().setCopy(0);
     }
 
     /**
-     * Create a Time object for storing a time with the time set to the specified time of the Joda Time DateTime object.
+     * Create a Time object for storing a time with the time set to the
+     * specified time of the Joda Time DateTime object.
      */
     public Time(final DateTime dateTime) {
         this.time = newDateTime(dateTime.getHourOfDay(), dateTime.getMinuteOfHour(), dateTime.getSecondOfMinute());
@@ -110,15 +113,16 @@ public class Time extends Magnitude<Time
     /**
      * Create a new Time object from the millisSinceEpoch, using UTC.
      */
-    public Time(long millisSinceEpoch) {
+    public Time(final long millisSinceEpoch) {
         this.time = new DateTime(millisSinceEpoch, Defaults.getTimeZone());
     }
 
     /**
-     * Add the specified hours and minutes to this time value, returned as a new Time object.
+     * Add the specified hours and minutes to this time value, returned as a new
+     * Time object.
      */
     public Time add(final int hours, final int minutes) {
-        Period period = new Period(hours, minutes, 0, 0);
+        final Period period = new Period(hours, minutes, 0, 0);
         return new Time(time.plus(period));
     }
 
@@ -137,7 +141,8 @@ public class Time extends Magnitude<Time
     }
 
     /*
-     * public java.util.Date dateValue() { return (date == null) ? null : date; }
+     * public java.util.Date dateValue() { return (date == null) ? null : date;
+     * }
      */
 
     public int getHour() {
@@ -153,7 +158,8 @@ public class Time extends Magnitude<Time
     }
 
     /**
-     * returns true if the time of this object has the same value as the specified time
+     * returns true if the time of this object has the same value as the
+     * specified time
      */
     @Override
     public boolean isEqualTo(final Time time) {
@@ -161,7 +167,8 @@ public class Time extends Magnitude<Time
     }
 
     /**
-     * returns true if the time of this object is earlier than the specified time
+     * returns true if the time of this object is earlier than the specified
+     * time
      */
     @Override
     public boolean isLessThan(final Time time) {
@@ -238,7 +245,7 @@ public class Time extends Magnitude<Time
     }
 
     public java.sql.Time asJavaTime() {
-        java.sql.Time time1 = java.sql.Time.valueOf(toString());
+        final java.sql.Time time1 = java.sql.Time.valueOf(toString());
         // TODO: confirm that this is in UTC
         return time1;
     }

Modified: incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/TimeStamp.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/TimeStamp.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/TimeStamp.java (original)
+++ incubator/isis/trunk/framework/applib/src/main/java/org/apache/isis/applib/value/TimeStamp.java Sat Jan 21 18:30:32 2012
@@ -23,8 +23,9 @@ import org.apache.isis.applib.annotation
 import org.apache.isis.applib.clock.Clock;
 
 /**
- * Value object representing a date/time value marking a point in time This is a user facing date/time value, more a
- * marker used to indicate the temporal relationship between two objects.
+ * Value object representing a date/time value marking a point in time This is a
+ * user facing date/time value, more a marker used to indicate the temporal
+ * relationship between two objects.
  * 
  * @see DateTime
  */
@@ -35,7 +36,8 @@ public class TimeStamp extends Magnitude
     private final long time;
 
     /**
-     * Create a TimeStamp object for storing a timeStamp set to the current time.
+     * Create a TimeStamp object for storing a timeStamp set to the current
+     * time.
      */
     public TimeStamp() {
         time = Clock.getTime();
@@ -46,7 +48,8 @@ public class TimeStamp extends Magnitude
     }
 
     /**
-     * returns true if the time stamp of this object has the same value as the specified timeStamp
+     * returns true if the time stamp of this object has the same value as the
+     * specified timeStamp
      */
     @Override
     public boolean isEqualTo(final TimeStamp timeStamp) {
@@ -54,7 +57,8 @@ public class TimeStamp extends Magnitude
     }
 
     /**
-     * returns true if the timeStamp of this object is earlier than the specified timeStamp
+     * returns true if the timeStamp of this object is earlier than the
+     * specified timeStamp
      */
     @Override
     public boolean isLessThan(final TimeStamp timeStamp) {

Modified: incubator/isis/trunk/framework/applib/src/test/java/org/apache/isis/applib/ContainedObjectTest.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/applib/src/test/java/org/apache/isis/applib/ContainedObjectTest.java?rev=1234388&r1=1234387&r2=1234388&view=diff
==============================================================================
--- incubator/isis/trunk/framework/applib/src/test/java/org/apache/isis/applib/ContainedObjectTest.java (original)
+++ incubator/isis/trunk/framework/applib/src/test/java/org/apache/isis/applib/ContainedObjectTest.java Sat Jan 21 18:30:32 2012
@@ -21,13 +21,12 @@ package org.apache.isis.applib;
 
 import static org.junit.Assert.assertEquals;
 
+import org.apache.isis.applib.security.UserMemento;
 import org.jmock.Expectations;
 import org.jmock.Mockery;
 import org.junit.Before;
 import org.junit.Test;
 
-import org.apache.isis.applib.security.UserMemento;
-
 public class ContainedObjectTest {
 
     private DomainObjectContainer container;



Mime
View raw message