isis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danhayw...@apache.org
Subject svn commit: r1234453 [7/33] - in /incubator/isis/trunk/framework/core: commons/src/main/java/org/apache/isis/core/commons/authentication/ commons/src/main/java/org/apache/isis/core/commons/components/ commons/src/main/java/org/apache/isis/core/commons/...
Date Sun, 22 Jan 2012 06:59:23 GMT
Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/InteractionUtils.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/InteractionUtils.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/InteractionUtils.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/InteractionUtils.java Sun Jan 22 06:58:43 2012
@@ -42,8 +42,7 @@ public final class InteractionUtils {
         return result;
     }
 
-    public static InteractionResultSet isVisibleResultSet(final FacetHolder facetHolder,
-        final VisibilityContext<?> context, final InteractionResultSet resultSet) {
+    public static InteractionResultSet isVisibleResultSet(final FacetHolder facetHolder, final VisibilityContext<?> context, final InteractionResultSet resultSet) {
         return resultSet.add(isVisibleResult(facetHolder, context));
     }
 
@@ -58,8 +57,7 @@ public final class InteractionUtils {
         return result;
     }
 
-    public static InteractionResultSet isUsableResultSet(final FacetHolder facetHolder,
-        final UsabilityContext<?> context, final InteractionResultSet resultSet) {
+    public static InteractionResultSet isUsableResultSet(final FacetHolder facetHolder, final UsabilityContext<?> context, final InteractionResultSet resultSet) {
         return resultSet.add(isUsableResult(facetHolder, context));
     }
 
@@ -73,8 +71,7 @@ public final class InteractionUtils {
         return result;
     }
 
-    public static InteractionResultSet isValidResultSet(final FacetHolder facetHolder,
-        final ValidityContext<?> context, final InteractionResultSet resultSet) {
+    public static InteractionResultSet isValidResultSet(final FacetHolder facetHolder, final ValidityContext<?> context, final InteractionResultSet resultSet) {
         return resultSet.add(isValidResult(facetHolder, context));
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectTitleContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectTitleContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectTitleContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectTitleContext.java Sun Jan 22 06:58:43 2012
@@ -29,14 +29,14 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link ObjectTitleEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link ObjectTitleEvent}.
  */
 public class ObjectTitleContext extends AccessContext<ObjectTitleEvent> {
 
     private final String title;
 
-    public ObjectTitleContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod,
-        final ObjectAdapter target, final Identifier identifier, final String title) {
+    public ObjectTitleContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier, final String title) {
         super(InteractionContextType.OBJECT_TITLE, session, invocationMethod, identifier, target);
         this.title = title;
     }

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectValidityContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectValidityContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectValidityContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ObjectValidityContext.java Sun Jan 22 06:58:43 2012
@@ -29,12 +29,12 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link ObjectValidityEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link ObjectValidityEvent}.
  */
 public class ObjectValidityContext extends ValidityContext<ObjectValidityEvent> implements ProposedHolder {
 
-    public ObjectValidityContext(final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier) {
+    public ObjectValidityContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier) {
         super(InteractionContextType.OBJECT_VALIDATE, session, invocationMethod, identifier, target);
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ParseValueContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ParseValueContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ParseValueContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ParseValueContext.java Sun Jan 22 06:58:43 2012
@@ -29,14 +29,14 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link ParseValueEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link ParseValueEvent}.
  */
 public class ParseValueContext extends ValidityContext<ParseValueEvent> implements ProposedHolder {
 
     private final ObjectAdapter proposed;
 
-    public ParseValueContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod,
-        final ObjectAdapter target, final Identifier identifier, final ObjectAdapter proposed) {
+    public ParseValueContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier, final ObjectAdapter proposed) {
         super(InteractionContextType.PARSE_VALUE, session, invocationMethod, identifier, target);
         this.proposed = proposed;
     }

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyAccessContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyAccessContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyAccessContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyAccessContext.java Sun Jan 22 06:58:43 2012
@@ -29,15 +29,14 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link PropertyAccessEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link PropertyAccessEvent}.
  */
 public class PropertyAccessContext extends AccessContext<PropertyAccessEvent> {
 
     private final ObjectAdapter value;
 
-    public PropertyAccessContext(final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier id,
-        final ObjectAdapter value) {
+    public PropertyAccessContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier id, final ObjectAdapter value) {
         super(InteractionContextType.PROPERTY_READ, session, invocationMethod, id, target);
 
         this.value = value;

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyModifyContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyModifyContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyModifyContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyModifyContext.java Sun Jan 22 06:58:43 2012
@@ -29,15 +29,14 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link PropertyModifyEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link PropertyModifyEvent}.
  */
 public class PropertyModifyContext extends ValidityContext<PropertyModifyEvent> implements ProposedHolder {
 
     private final ObjectAdapter proposed;
 
-    public PropertyModifyContext(final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier id,
-        final ObjectAdapter proposed) {
+    public PropertyModifyContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier id, final ObjectAdapter proposed) {
         super(InteractionContextType.PROPERTY_MODIFY, session, invocationMethod, id, target);
 
         this.proposed = proposed;

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyUsabilityContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyUsabilityContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyUsabilityContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyUsabilityContext.java Sun Jan 22 06:58:43 2012
@@ -29,12 +29,12 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link PropertyUsabilityEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link PropertyUsabilityEvent}.
  */
 public class PropertyUsabilityContext extends UsabilityContext<PropertyUsabilityEvent> {
 
-    public PropertyUsabilityContext(final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier) {
+    public PropertyUsabilityContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier) {
         super(InteractionContextType.PROPERTY_USABLE, session, invocationMethod, identifier, target);
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyVisibilityContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyVisibilityContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyVisibilityContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/PropertyVisibilityContext.java Sun Jan 22 06:58:43 2012
@@ -29,12 +29,12 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link PropertyVisibilityEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link PropertyVisibilityEvent}.
  */
 public class PropertyVisibilityContext extends VisibilityContext<PropertyVisibilityEvent> {
 
-    public PropertyVisibilityContext(final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier) {
+    public PropertyVisibilityContext(final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final ObjectAdapter target, final Identifier identifier) {
         super(InteractionContextType.PROPERTY_VISIBLE, session, invocationMethod, identifier, target);
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ProposedHolder.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ProposedHolder.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ProposedHolder.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ProposedHolder.java Sun Jan 22 06:58:43 2012
@@ -23,13 +23,14 @@ import org.apache.isis.core.metamodel.ad
 import org.apache.isis.core.metamodel.facetapi.Facet;
 
 /**
- * Implemented by some of the {@link InteractionContext} subclasses, making it easier for facets to process multiple at
- * the same time.
+ * Implemented by some of the {@link InteractionContext} subclasses, making it
+ * easier for facets to process multiple at the same time.
  * 
  * <p>
- * Where this is used most often is for {@link PropertyModifyContext} and {@link ActionArgumentContext}, where a
- * {@link Facet} (such as <tt>MandatoryFacet</tt> or <tt>MaxLengthFacet</tt>) would want to perform the same checks on
- * the {@link #getProposed() proposed} value/argument.
+ * Where this is used most often is for {@link PropertyModifyContext} and
+ * {@link ActionArgumentContext}, where a {@link Facet} (such as
+ * <tt>MandatoryFacet</tt> or <tt>MaxLengthFacet</tt>) would want to perform the
+ * same checks on the {@link #getProposed() proposed} value/argument.
  */
 public interface ProposedHolder {
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/UsabilityContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/UsabilityContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/UsabilityContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/UsabilityContext.java Sun Jan 22 06:58:43 2012
@@ -27,12 +27,12 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link UsabilityEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link UsabilityEvent}.
  */
 public abstract class UsabilityContext<T extends UsabilityEvent> extends InteractionContext<T> {
 
-    public UsabilityContext(final InteractionContextType interactionType, final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final Identifier identifier, final ObjectAdapter target) {
+    public UsabilityContext(final InteractionContextType interactionType, final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final Identifier identifier, final ObjectAdapter target) {
         super(interactionType, session, invocationMethod, identifier, target);
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidatingInteractionAdvisor.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidatingInteractionAdvisor.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidatingInteractionAdvisor.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidatingInteractionAdvisor.java Sun Jan 22 06:58:43 2012
@@ -24,11 +24,13 @@ import org.apache.isis.core.metamodel.fa
 import org.apache.isis.core.metamodel.facetapi.FacetAbstract.Validating;
 
 /**
- * Mix-in interface for {@link Facet}s that can advise as to whether a proposed value is valid.
+ * Mix-in interface for {@link Facet}s that can advise as to whether a proposed
+ * value is valid.
  * 
  * <p>
- * For example, <tt>MaxLengthFacet</tt> does constrain the length of candidate values, whereas <tt>DebugFacet</tt> or
- * <tt>MemberOrderFacet</tt> do not - they are basically just UI hints.
+ * For example, <tt>MaxLengthFacet</tt> does constrain the length of candidate
+ * values, whereas <tt>DebugFacet</tt> or <tt>MemberOrderFacet</tt> do not -
+ * they are basically just UI hints.
  * 
  * @see DisablingInteractionAdvisor
  * @see HidingInteractionAdvisor
@@ -39,9 +41,11 @@ public interface ValidatingInteractionAd
      * Whether the validation represented by this facet passes or fails.
      * 
      * <p>
-     * Implementations should use the provided {@link ValidityContext} to determine whether they declare the interaction
-     * invalid. They must however guard against a <tt>null</tt> {@link ValidityContext#getTarget() target} and
-     * {@link ValidityContext#getSession() session} - neither are guaranteed to be populated.
+     * Implementations should use the provided {@link ValidityContext} to
+     * determine whether they declare the interaction invalid. They must however
+     * guard against a <tt>null</tt> {@link ValidityContext#getTarget() target}
+     * and {@link ValidityContext#getSession() session} - neither are guaranteed
+     * to be populated.
      */
     String invalidates(final ValidityContext<? extends ValidityEvent> ic);
 }

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidityContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidityContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidityContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/ValidityContext.java Sun Jan 22 06:58:43 2012
@@ -27,12 +27,12 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link ValidityEvent}
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link ValidityEvent}
  */
 public abstract class ValidityContext<T extends ValidityEvent> extends InteractionContext<T> {
 
-    public ValidityContext(final InteractionContextType interactionType, final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final Identifier identifier, final ObjectAdapter target) {
+    public ValidityContext(final InteractionContextType interactionType, final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final Identifier identifier, final ObjectAdapter target) {
         super(interactionType, session, invocationMethod, identifier, target);
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/VisibilityContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/VisibilityContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/VisibilityContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/interactions/VisibilityContext.java Sun Jan 22 06:58:43 2012
@@ -27,12 +27,12 @@ import org.apache.isis.core.metamodel.co
 import org.apache.isis.core.metamodel.consent.InteractionInvocationMethod;
 
 /**
- * See {@link InteractionContext} for overview; analogous to {@link VisibilityEvent}.
+ * See {@link InteractionContext} for overview; analogous to
+ * {@link VisibilityEvent}.
  */
 public abstract class VisibilityContext<T extends VisibilityEvent> extends InteractionContext<T> {
 
-    public VisibilityContext(final InteractionContextType interactionType, final AuthenticationSession session,
-        final InteractionInvocationMethod invocationMethod, final Identifier identifier, final ObjectAdapter target) {
+    public VisibilityContext(final InteractionContextType interactionType, final AuthenticationSession session, final InteractionInvocationMethod invocationMethod, final Identifier identifier, final ObjectAdapter target) {
         super(interactionType, session, invocationMethod, identifier, target);
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArranger.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArranger.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArranger.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArranger.java Sun Jan 22 06:58:43 2012
@@ -23,10 +23,11 @@ import org.apache.isis.core.metamodel.sp
 
 /**
  * Orders both associations (properties and collections) and also actions.
- *
+ * 
  * <p>
- * TODO: for now, 'arranging' means merely ordering.  In the future, this might evolve into a more general specification
- * of a layout, eg columns and other layouts.
+ * TODO: for now, 'arranging' means merely ordering. In the future, this might
+ * evolve into a more general specification of a layout, eg columns and other
+ * layouts.
  */
 public interface MemberLayoutArranger {
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArrangerComposite.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArrangerComposite.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArrangerComposite.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/MemberLayoutArrangerComposite.java Sun Jan 22 06:58:43 2012
@@ -26,7 +26,7 @@ import org.apache.log4j.Logger;
 import com.google.common.collect.Lists;
 
 public abstract class MemberLayoutArrangerComposite implements MemberLayoutArranger {
-    
+
     private static final Logger LOG = Logger.getLogger(MemberLayoutArrangerComposite.class);
 
     private final List<MemberLayoutArranger> arrangers = Lists.newArrayList();
@@ -35,50 +35,47 @@ public abstract class MemberLayoutArrang
     // constructor
     // ////////////////////////////////////////////////////////////////////////////
 
-    public MemberLayoutArrangerComposite(MemberLayoutArranger... arrangers ) {
+    public MemberLayoutArrangerComposite(final MemberLayoutArranger... arrangers) {
         this.arrangers.addAll(Arrays.asList(arrangers));
     }
 
-
     // ////////////////////////////////////////////////////////////////////////////
     // associations
     // ////////////////////////////////////////////////////////////////////////////
 
-
     @Override
-    public OrderSet createAssociationOrderSetFor(ObjectSpecification spec, final List<FacetedMethod> associationMethods) {
+    public OrderSet createAssociationOrderSetFor(final ObjectSpecification spec, final List<FacetedMethod> associationMethods) {
         if (LOG.isDebugEnabled()) {
             LOG.debug("MemberLayoutArrangerComposite: createAssociationOrderSetFor " + spec.getFullIdentifier());
         }
 
-        for (MemberLayoutArranger arranger : arrangers) {
+        for (final MemberLayoutArranger arranger : arrangers) {
             final OrderSet orderSet = arranger.createAssociationOrderSetFor(spec, associationMethods);
-            if(orderSet != null) {
+            if (orderSet != null) {
                 return orderSet;
             }
         }
         return null;
     }
 
-
     // ////////////////////////////////////////////////////////////////////////////
     // actions
     // ////////////////////////////////////////////////////////////////////////////
-    
+
     @Override
-    public OrderSet createActionOrderSetFor(ObjectSpecification spec, List<FacetedMethod> actionMethods) {
+    public OrderSet createActionOrderSetFor(final ObjectSpecification spec, final List<FacetedMethod> actionMethods) {
         if (LOG.isDebugEnabled()) {
             LOG.debug("MemberLayoutArrangerDefault: createAssociationOrderSetFor " + spec.getFullIdentifier());
         }
 
-        for (MemberLayoutArranger arranger : arrangers) {
+        for (final MemberLayoutArranger arranger : arrangers) {
             final OrderSet orderSet = arranger.createActionOrderSetFor(spec, actionMethods);
-            if(orderSet != null) {
+            if (orderSet != null) {
                 return orderSet;
             }
         }
         return null;
 
     }
-    
+
 }

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/OrderSet.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/OrderSet.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/OrderSet.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/OrderSet.java Sun Jan 22 06:58:43 2012
@@ -17,7 +17,6 @@
  *  under the License.
  */
 
-
 package org.apache.isis.core.metamodel.layout;
 
 import java.util.ArrayList;
@@ -32,7 +31,6 @@ import org.apache.isis.core.metamodel.sp
 
 import com.google.common.collect.Lists;
 
-
 /**
  * This represents a particular layout of {@link ObjectMember}s.
  * 
@@ -49,8 +47,8 @@ public class OrderSet implements Compara
     private OrderSet parent;
 
     /**
-     * A staging area until we are ready to add the child sets to the collection of elements owned by the
-     * superclass.
+     * A staging area until we are ready to add the child sets to the collection
+     * of elements owned by the superclass.
      */
     protected SortedSet<DeweyOrderSet> childOrderSets = new TreeSet<DeweyOrderSet>();
 
@@ -64,12 +62,14 @@ public class OrderSet implements Compara
     // ///////////////// Group Name etc ////////////////////
 
     /**
-     * Last component of the comma-separated group name supplied in the constructor (analogous to the file
-     * name extracted from a fully qualified file name).
-     *
+     * Last component of the comma-separated group name supplied in the
+     * constructor (analogous to the file name extracted from a fully qualified
+     * file name).
+     * 
      * <p>
-     * For example, if supplied <tt>abc,def,ghi</tt> in the constructor, then this will return <tt>ghi</tt>.
-     *
+     * For example, if supplied <tt>abc,def,ghi</tt> in the constructor, then
+     * this will return <tt>ghi</tt>.
+     * 
      * @return
      */
     public String getGroupName() {
@@ -77,13 +77,13 @@ public class OrderSet implements Compara
     }
 
     /**
-     * The group name exactly as it was supplied in the constructor (analogous to a fully qualified file
-     * name).
-     *
+     * The group name exactly as it was supplied in the constructor (analogous
+     * to a fully qualified file name).
+     * 
      * <p>
-     * For example, if supplied <tt>abc,def,ghi</tt> in the constructor, then this will return the same string
-     * <tt>abc,def,ghi</tt>.
-     *
+     * For example, if supplied <tt>abc,def,ghi</tt> in the constructor, then
+     * this will return the same string <tt>abc,def,ghi</tt>.
+     * 
      * @return
      */
     public String getGroupFullName() {
@@ -91,13 +91,14 @@ public class OrderSet implements Compara
     }
 
     /**
-     * Represents the parent groups, derived from the group name supplied in the constructor (analogous to the
-     * directory portion of a fully qualified file name).
-     *
+     * Represents the parent groups, derived from the group name supplied in the
+     * constructor (analogous to the directory portion of a fully qualified file
+     * name).
+     * 
      * <p>
-     * For example, if supplied <tt>abc,def,ghi</tt> in the constructor, then this will return
-     * <tt>abc,def</tt>.
-     *
+     * For example, if supplied <tt>abc,def,ghi</tt> in the constructor, then
+     * this will return <tt>abc,def</tt>.
+     * 
      * @return
      */
     public String getGroupPath() {
@@ -106,7 +107,7 @@ public class OrderSet implements Compara
 
     /**
      * Splits name by comma, then title case the last component.
-     *
+     * 
      * @param groupFullName
      * @return
      */
@@ -127,7 +128,7 @@ public class OrderSet implements Compara
 
     /**
      * Everything upto the last comma, else empty string if none.
-     *
+     * 
      * @param groupFullName
      * @return
      */
@@ -141,7 +142,6 @@ public class OrderSet implements Compara
 
     // ///////////////////// Parent & Children ///////////////////
 
-
     protected void setParent(final DeweyOrderSet parent) {
         this.parent = parent;
     }
@@ -150,7 +150,6 @@ public class OrderSet implements Compara
         return parent;
     }
 
-
     protected void addChild(final DeweyOrderSet childOrderSet) {
         childOrderSets.add(childOrderSet);
     }
@@ -169,7 +168,7 @@ public class OrderSet implements Compara
 
     /**
      * Returns a copy of the elements, in sequence.
-     *
+     * 
      * @return
      */
     public List<Object> elementList() {
@@ -190,8 +189,8 @@ public class OrderSet implements Compara
     }
 
     protected void addAll(final SortedSet<DeweyOrderSet> sortedMembers) {
-        for (final Iterator<DeweyOrderSet> iter = sortedMembers.iterator(); iter.hasNext();) {
-            this.addElement(iter.next());
+        for (final DeweyOrderSet deweyOrderSet : sortedMembers) {
+            this.addElement(deweyOrderSet);
         }
     }
 
@@ -202,24 +201,31 @@ public class OrderSet implements Compara
      */
     @Override
     public int compareTo(final OrderSet o) {
-        if (this.equals(o)) { return 0; }
+        if (this.equals(o)) {
+            return 0;
+        }
         return groupFullName.compareTo(o.groupFullName);
     }
 
     @Override
-    public boolean equals(Object obj) {
-        if (this == obj)
+    public boolean equals(final Object obj) {
+        if (this == obj) {
             return true;
-        if (obj == null)
+        }
+        if (obj == null) {
             return false;
-        if (getClass() != obj.getClass())
+        }
+        if (getClass() != obj.getClass()) {
             return false;
-        OrderSet other = (OrderSet) obj;
+        }
+        final OrderSet other = (OrderSet) obj;
         if (groupFullName == null) {
-            if (other.groupFullName != null)
+            if (other.groupFullName != null) {
                 return false;
-        } else if (!groupFullName.equals(other.groupFullName))
+            }
+        } else if (!groupFullName.equals(other.groupFullName)) {
             return false;
+        }
         return true;
     }
 
@@ -231,6 +237,4 @@ public class OrderSet implements Compara
         return result;
     }
 
-
 }
-

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/DeweyOrderSet.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/DeweyOrderSet.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/DeweyOrderSet.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/DeweyOrderSet.java Sun Jan 22 06:58:43 2012
@@ -19,7 +19,6 @@
 
 package org.apache.isis.core.metamodel.layout.memberorderfacet;
 
-import java.util.Iterator;
 import java.util.List;
 import java.util.Set;
 import java.util.SortedMap;
@@ -37,8 +36,9 @@ import com.google.common.collect.Sets;
  * Represents a nested hierarchy of ordered members.
  * 
  * <p>
- * At each level the elements are either {@link FacetedMethod}s or they are instances of {@link OrderSet} represent a
- * group of {@link FacetedMethod}s that have a {@link MemberOrderFacet} of the same name.
+ * At each level the elements are either {@link FacetedMethod}s or they are
+ * instances of {@link OrderSet} represent a group of {@link FacetedMethod}s
+ * that have a {@link MemberOrderFacet} of the same name.
  * 
  * <p>
  * With no name, (ie <tt>name=""</tt> is the default), at the top level
@@ -90,32 +90,35 @@ public class DeweyOrderSet extends Order
         // create OrderSets, wiring up parents and children.
 
         // since sortedMembersByGroup is a SortedMap, the
-        // iteration will be in alphabetical order (ie parent groups before their children).
+        // iteration will be in alphabetical order (ie parent groups before
+        // their children).
         final Set<String> groupNames = sortedMembersByGroup.keySet();
-        final SortedMap<String,DeweyOrderSet> orderSetsByGroup = Maps.newTreeMap();
+        final SortedMap<String, DeweyOrderSet> orderSetsByGroup = Maps.newTreeMap();
 
-        for (final Iterator<String> iter = groupNames.iterator(); iter.hasNext();) {
-            final String groupName = (String) iter.next();
+        for (final String string : groupNames) {
+            final String groupName = string;
             final DeweyOrderSet deweyOrderSet = new DeweyOrderSet(groupName);
             orderSetsByGroup.put(groupName, deweyOrderSet);
             ensureParentFor(orderSetsByGroup, deweyOrderSet);
         }
 
         // now populate the OrderSets
-        for (final String groupName: groupNames) {
-            final DeweyOrderSet deweyOrderSet = (DeweyOrderSet) orderSetsByGroup.get(groupName);
-            // REVIEW: something fishy happens here with casting, hence warnings left in
-            final SortedSet sortedMembers = (SortedSet) sortedMembersByGroup.get(groupName);
+        for (final String groupName : groupNames) {
+            final DeweyOrderSet deweyOrderSet = orderSetsByGroup.get(groupName);
+            // REVIEW: something fishy happens here with casting, hence warnings
+            // left in
+            final SortedSet sortedMembers = sortedMembersByGroup.get(groupName);
             deweyOrderSet.addAll(sortedMembers);
             deweyOrderSet.copyOverChildren();
         }
 
-        return (DeweyOrderSet) orderSetsByGroup.get("");
+        return orderSetsByGroup.get("");
     }
 
     /**
-     * Recursively creates parents all the way up to root (<tt>""</tt>), along the way associating each child with its
-     * parent and adding the child as an element of its parent.
+     * Recursively creates parents all the way up to root (<tt>""</tt>), along
+     * the way associating each child with its parent and adding the child as an
+     * element of its parent.
      * 
      * @param orderSetsByGroup
      * @param deweyOrderSet
@@ -138,7 +141,8 @@ public class DeweyOrderSet extends Order
     }
 
     /**
-     * Gets the SortedSet with the specified group from the supplied Map of SortedSets.
+     * Gets the SortedSet with the specified group from the supplied Map of
+     * SortedSets.
      * 
      * <p>
      * If there is no such SortedSet, creates.
@@ -147,7 +151,7 @@ public class DeweyOrderSet extends Order
      * @param groupName
      * @return
      */
-    private static SortedSet<FacetedMethod> getSortedSet(final SortedMap<String,SortedSet<FacetedMethod>> sortedMembersByGroup, final String groupName) {
+    private static SortedSet<FacetedMethod> getSortedSet(final SortedMap<String, SortedSet<FacetedMethod>> sortedMembersByGroup, final String groupName) {
         SortedSet<FacetedMethod> sortedMembersForGroup;
         sortedMembersForGroup = sortedMembersByGroup.get(groupName);
         if (sortedMembersForGroup == null) {
@@ -166,13 +170,13 @@ public class DeweyOrderSet extends Order
     /**
      * Format is: <tt>abc,def:XXel/YYm/ZZch</tt>
      * <p>
-     * where <tt>abc,def</tt> is group name, <tt>XX</tt> is number of elements, <tt>YY is number of members, and 
+     * where <tt>abc,def</tt> is group name, <tt>XX</tt> is number of elements,
+     * <tt>YY is number of members, and 
      * <tt>ZZ</tt> is number of child order sets.
      */
     @Override
     public String toString() {
-        return getGroupFullName() + ":" + size() + "el/" + (size() - childOrderSets.size()) + "m/"
-            + childOrderSets.size() + "ch";
+        return getGroupFullName() + ":" + size() + "el/" + (size() - childOrderSets.size()) + "m/" + childOrderSets.size() + "ch";
     }
 
 }

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberLayoutArrangerUsingMemberOrderFacet.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberLayoutArrangerUsingMemberOrderFacet.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberLayoutArrangerUsingMemberOrderFacet.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberLayoutArrangerUsingMemberOrderFacet.java Sun Jan 22 06:58:43 2012
@@ -40,11 +40,9 @@ public class MemberLayoutArrangerUsingMe
     // ////////////////////////////////////////////////////////////////////////////
 
     @Override
-    public OrderSet createAssociationOrderSetFor(final ObjectSpecification spec,
-        final List<FacetedMethod> associationMethods) {
+    public OrderSet createAssociationOrderSetFor(final ObjectSpecification spec, final List<FacetedMethod> associationMethods) {
         if (LOG.isDebugEnabled()) {
-            LOG.debug("MemberLayoutArrangerUsingMemberOrderFacet: createAssociationOrderSetFor "
-                + spec.getFullIdentifier());
+            LOG.debug("MemberLayoutArrangerUsingMemberOrderFacet: createAssociationOrderSetFor " + spec.getFullIdentifier());
         }
 
         return DeweyOrderSet.createOrderSet(associationMethods);
@@ -55,11 +53,9 @@ public class MemberLayoutArrangerUsingMe
     // ////////////////////////////////////////////////////////////////////////////
 
     @Override
-    public OrderSet createActionOrderSetFor(final ObjectSpecification spec,
-        final List<FacetedMethod> actionFacetedMethodList) {
+    public OrderSet createActionOrderSetFor(final ObjectSpecification spec, final List<FacetedMethod> actionFacetedMethodList) {
         if (LOG.isDebugEnabled()) {
-            LOG.debug("MemberLayoutArrangerUsingMemberOrderFacet: createAssociationOrderSetFor "
-                + spec.getFullIdentifier());
+            LOG.debug("MemberLayoutArrangerUsingMemberOrderFacet: createAssociationOrderSetFor " + spec.getFullIdentifier());
         }
 
         return DeweyOrderSet.createOrderSet(actionFacetedMethodList);

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberOrderComparator.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberOrderComparator.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberOrderComparator.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/MemberOrderComparator.java Sun Jan 22 06:58:43 2012
@@ -28,14 +28,17 @@ import org.apache.isis.core.metamodel.fa
 import org.apache.isis.core.metamodel.layout.OrderSet;
 
 /**
- * Compares by {@link MemberOrderFacet} obtained from each {@link FacetedMethod}).
+ * Compares by {@link MemberOrderFacet} obtained from each {@link FacetedMethod}
+ * ).
  * 
  * <p>
- * Will also compare {@link OrderSet}s; these are put after any {@link FacetedMethod}s. If there is more than one
- * OrderSet then these are compared by an {@link OrderSetGroupNameComparator}.
+ * Will also compare {@link OrderSet}s; these are put after any
+ * {@link FacetedMethod}s. If there is more than one OrderSet then these are
+ * compared by an {@link OrderSetGroupNameComparator}.
  * 
  * <p>
- * If there is no annotation on either member, then will compare the members by name instead.
+ * If there is no annotation on either member, then will compare the members by
+ * name instead.
  * 
  * <p>
  * Can specify if requires that members are in the same (group) name.
@@ -93,7 +96,7 @@ public class MemberOrderComparator imple
 
         final String sequence1 = m1.sequence();
         final String sequence2 = m2.sequence();
-        
+
         final String[] components1 = componentsFor(sequence1);
         final String[] components2 = componentsFor(sequence2);
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/OrderSetGroupNameComparator.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/OrderSetGroupNameComparator.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/OrderSetGroupNameComparator.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/memberorderfacet/OrderSetGroupNameComparator.java Sun Jan 22 06:58:43 2012
@@ -28,8 +28,8 @@ import org.apache.isis.core.metamodel.la
  * Compares by (simple) group name of each {@link OrderSet}.
  * 
  * <p>
- * Note that it only makes sense to use this comparator for {@link OrderSet}s that are known to have the same parent
- * {@link OrderSet}s.
+ * Note that it only makes sense to use this comparator for {@link OrderSet}s
+ * that are known to have the same parent {@link OrderSet}s.
  */
 public class OrderSetGroupNameComparator implements Comparator<OrderSet>, Serializable {
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/ordermethod/SimpleOrderSet.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/ordermethod/SimpleOrderSet.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/ordermethod/SimpleOrderSet.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/layout/ordermethod/SimpleOrderSet.java Sun Jan 22 06:58:43 2012
@@ -65,8 +65,7 @@ public class SimpleOrderSet extends Orde
         parent = null;
     }
 
-    private SimpleOrderSet(final SimpleOrderSet set, final String groupName, final String name,
-        final List<FacetedMethod> members) {
+    private SimpleOrderSet(final SimpleOrderSet set, final String groupName, final String name, final List<FacetedMethod> members) {
         super(groupName);
         parent = set;
         parent.addElement(this);

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/methodutils/MethodFinderUtils.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/methodutils/MethodFinderUtils.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/methodutils/MethodFinderUtils.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/methodutils/MethodFinderUtils.java Sun Jan 22 06:58:43 2012
@@ -31,7 +31,8 @@ import org.apache.isis.core.commons.lang
 import com.google.common.collect.Maps;
 
 /**
- * TODO: remove duplication with {@link WrapperUtils} and {@link PrimitiveUtils}.
+ * TODO: remove duplication with {@link WrapperUtils} and {@link PrimitiveUtils}
+ * .
  */
 public final class MethodFinderUtils {
     private static Map<Class<?>, Class<?>> boxedClasses = Maps.newHashMap();
@@ -53,17 +54,19 @@ public final class MethodFinderUtils {
     }
 
     /**
-     * Searches the supplied array of methods for specific method and returns it, also removing it from supplied array
-     * if found (by setting to <tt>null</tt>).
+     * Searches the supplied array of methods for specific method and returns
+     * it, also removing it from supplied array if found (by setting to
+     * <tt>null</tt>).
      * 
      * <p>
-     * Any methods that do not meet the search criteria are left in the array of methods.
+     * Any methods that do not meet the search criteria are left in the array of
+     * methods.
      * 
      * <p>
-     * The search algorithm is as specified in {@link #findMethodIndex(List, MethodScope, String, Class, Class[])}.
+     * The search algorithm is as specified in
+     * {@link #findMethodIndex(List, MethodScope, String, Class, Class[])}.
      */
-    public static Method removeMethod(final List<Method> methods, final MethodScope methodScope, final String name,
-        final Class<?> returnType, final Class<?>[] paramTypes) {
+    public static Method removeMethod(final List<Method> methods, final MethodScope methodScope, final String name, final Class<?> returnType, final Class<?>[] paramTypes) {
         final int idx = findMethodIndex(methods, methodScope, name, returnType, paramTypes);
         if (idx != -1) {
             final Method method = methods.get(idx);
@@ -74,31 +77,33 @@ public final class MethodFinderUtils {
     }
 
     /**
-     * Searches the supplied array of methods for specific method and returns it,.
+     * Searches the supplied array of methods for specific method and returns
+     * it,.
      * 
      * <p>
-     * The search algorithm is as specified in {@link #findMethodIndex(List, MethodScope, String, Class, Class[])}.
+     * The search algorithm is as specified in
+     * {@link #findMethodIndex(List, MethodScope, String, Class, Class[])}.
      */
-    public static Method findMethod(final List<Method> methods, final MethodScope methodScope, final String name,
-        final Class<?> returnType, final Class<?>[] paramTypes) {
+    public static Method findMethod(final List<Method> methods, final MethodScope methodScope, final String name, final Class<?> returnType, final Class<?>[] paramTypes) {
         final int idx = findMethodIndex(methods, methodScope, name, returnType, paramTypes);
         return idx != -1 ? methods.get(idx) : null;
     }
 
     /**
-     * Searches the supplied array of methods for specific method and returns its index, otherwise returns <tt>-1</tt>.
+     * Searches the supplied array of methods for specific method and returns
+     * its index, otherwise returns <tt>-1</tt>.
      * 
      * <p>
      * The search algorithm is:
      * <ul>
      * <li>has the specified prefix</li>
-     * <li>has the specified return type, or <tt>void</tt> if canBeVoid is <tt>true</tt> (but see below)</li>
+     * <li>has the specified return type, or <tt>void</tt> if canBeVoid is
+     * <tt>true</tt> (but see below)</li>
      * <li>has the specified number of parameters</li>
      * </ul>
      * If the returnType is specified as null then the return type is ignored.
      */
-    public static int findMethodIndex(final List<Method> methods, final MethodScope methodScope, final String name,
-        final Class<?> returnType, final Class<?>[] paramTypes) {
+    public static int findMethodIndex(final List<Method> methods, final MethodScope methodScope, final String name, final Class<?> returnType, final Class<?>[] paramTypes) {
         int idx = -1;
         method: for (int i = 0; i < methods.size(); i++) {
             if (methods.get(i) == null) {
@@ -153,17 +158,19 @@ public final class MethodFinderUtils {
     }
 
     /**
-     * Searches the supplied array of methods for all specific methods and returns them, also removing them from
-     * supplied array if found.
+     * Searches the supplied array of methods for all specific methods and
+     * returns them, also removing them from supplied array if found.
      * 
      * <p>
-     * Any methods that do not meet the search criteria are left in the array of methods.
+     * Any methods that do not meet the search criteria are left in the array of
+     * methods.
      * 
      * <p>
      * The search algorithm is:
      * <ul>
      * <li>has the specified prefix</li>
-     * <li>has the specified return type, or <tt>void</tt> if canBeVoid is <tt>true</tt> (but see below)</li>
+     * <li>has the specified return type, or <tt>void</tt> if canBeVoid is
+     * <tt>true</tt> (but see below)</li>
      * <li>has the specified number of parameters</li>
      * </ul>
      * If the returnType is specified as null then the return type is ignored.
@@ -172,11 +179,11 @@ public final class MethodFinderUtils {
      * @param name
      * @param returnType
      * @param paramTypes
-     *            the set of parameters the method should have, if null then is ignored
+     *            the set of parameters the method should have, if null then is
+     *            ignored
      * @return Method
      */
-    public static List<Method> removeMethods(final List<Method> methods, final MethodScope forClass,
-        final String prefix, final Class<?> returnType, final boolean canBeVoid, final int paramCount) {
+    public static List<Method> removeMethods(final List<Method> methods, final MethodScope forClass, final String prefix, final Class<?> returnType, final boolean canBeVoid, final int paramCount) {
 
         final List<Method> validMethods = new ArrayList<Method>();
 
@@ -221,11 +228,11 @@ public final class MethodFinderUtils {
     }
 
     /**
-     * From the supplied method array, finds but <i>does not remove</i> methods that have the required prefix, and adds
-     * to the supplied candidates vector.
+     * From the supplied method array, finds but <i>does not remove</i> methods
+     * that have the required prefix, and adds to the supplied candidates
+     * vector.
      */
-    public static void findPrefixedInstanceMethods(final Method[] methods, final String prefix,
-        final List<Method> candidates) {
+    public static void findPrefixedInstanceMethods(final Method[] methods, final String prefix, final List<Method> candidates) {
         for (final Method method : methods) {
             if (method == null) {
                 continue;

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/progmodel/ProgrammingModelAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/progmodel/ProgrammingModelAbstract.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/progmodel/ProgrammingModelAbstract.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/progmodel/ProgrammingModelAbstract.java Sun Jan 22 06:58:43 2012
@@ -29,8 +29,7 @@ import org.apache.isis.core.metamodel.fa
 public abstract class ProgrammingModelAbstract implements ProgrammingModel {
 
     private final List<FacetFactory> facetFactories = new ArrayList<FacetFactory>();
-    private final List<Class<? extends FacetFactory>> facetFactoryClasses =
-        new ArrayList<Class<? extends FacetFactory>>();
+    private final List<Class<? extends FacetFactory>> facetFactoryClasses = new ArrayList<Class<? extends FacetFactory>>();
 
     @Override
     public final List<FacetFactory> getList() {

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContext.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContext.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContext.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContext.java Sun Jan 22 06:58:43 2012
@@ -40,11 +40,12 @@ import org.apache.isis.core.metamodel.sp
 public interface RuntimeContext extends Injectable {
 
     /**
-     * A mechanism for returning the <tt>current</tt> {@link AuthenticationSession}.
+     * A mechanism for returning the <tt>current</tt>
+     * {@link AuthenticationSession}.
      * 
      * <p>
-     * Note that the scope of {@link RuntimeContext} is global, whereas {@link AuthenticationSession} may change over
-     * time.
+     * Note that the scope of {@link RuntimeContext} is global, whereas
+     * {@link AuthenticationSession} may change over time.
      */
     public AuthenticationSessionProvider getAuthenticationSessionProvider();
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContextAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContextAbstract.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContextAbstract.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/RuntimeContextAbstract.java Sun Jan 22 06:58:43 2012
@@ -32,8 +32,7 @@ import org.apache.isis.core.metamodel.sp
 import org.apache.isis.core.metamodel.spec.SpecificationLookup;
 import org.apache.isis.core.metamodel.spec.SpecificationLookupDelegator;
 
-public abstract class RuntimeContextAbstract implements RuntimeContext, SpecificationLoaderAware,
-    DomainObjectContainerAware {
+public abstract class RuntimeContextAbstract implements RuntimeContext, SpecificationLoaderAware, DomainObjectContainerAware {
 
     private final SpecificationLookupDelegator specificationLookupDelegator;
     private DomainObjectContainer container;
@@ -72,7 +71,8 @@ public abstract class RuntimeContextAbst
     }
 
     /**
-     * Is injected into when the reflector is {@link ObjectReflectorAbstract#init() initialized}.
+     * Is injected into when the reflector is
+     * {@link ObjectReflectorAbstract#init() initialized}.
      */
     @Override
     public void setSpecificationLoader(final SpecificationLoader specificationLoader) {
@@ -100,7 +100,8 @@ public abstract class RuntimeContextAbst
     }
 
     /**
-     * So that {@link #injectDependenciesInto(Object)} can also inject the {@link DomainObjectContainer}.
+     * So that {@link #injectDependenciesInto(Object)} can also inject the
+     * {@link DomainObjectContainer}.
      */
     @Override
     public void setContainer(final DomainObjectContainer container) {

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/runtimecontext/noruntime/RuntimeContextNoRuntime.java Sun Jan 22 06:58:43 2012
@@ -68,8 +68,9 @@ public class RuntimeContextNoRuntime ext
         dependencyInjector = new DependencyInjectorAbstract() {
 
             /**
-             * Unlike most of the methods in this implementation, does nothing (because this will always be called, even
-             * in a no-runtime context).
+             * Unlike most of the methods in this implementation, does nothing
+             * (because this will always be called, even in a no-runtime
+             * context).
              */
             @Override
             public void injectDependenciesInto(final Object domainObject) {
@@ -90,8 +91,7 @@ public class RuntimeContextNoRuntime ext
             }
 
             @Override
-            public ObjectAdapter adapterFor(final Object pojo, final ObjectAdapter ownerAdapter,
-                final IdentifiedHolder identifiedHolder) {
+            public ObjectAdapter adapterFor(final Object pojo, final ObjectAdapter ownerAdapter, final IdentifiedHolder identifiedHolder) {
                 throw new UnsupportedOperationException("Not supported by this implementation of RuntimeContext");
             }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjector.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjector.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjector.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjector.java Sun Jan 22 06:58:43 2012
@@ -29,8 +29,8 @@ import org.apache.isis.core.commons.comp
  * The repository of services, also able to inject into any object.
  * 
  * <p>
- * The {@link #getContainer() domain object container} is always injected but is not a {@link #getRegisteredServices()
- * registered service}.
+ * The {@link #getContainer() domain object container} is always injected but is
+ * not a {@link #getRegisteredServices() registered service}.
  */
 public interface ServicesInjector extends SessionScopedComponent, Injectable {
 
@@ -56,7 +56,8 @@ public interface ServicesInjector extend
      * Services to be injected.
      * 
      * <p>
-     * Should automatically inject all services into each other (though calling {@link #open()} will also do this).
+     * Should automatically inject all services into each other (though calling
+     * {@link #open()} will also do this).
      * 
      * @param services
      */
@@ -80,7 +81,8 @@ public interface ServicesInjector extend
     void injectDependencies(Object domainObject);
 
     /**
-     * As per {@link #injectDependencies(Object)}, but for all objects in the list.
+     * As per {@link #injectDependencies(Object)}, but for all objects in the
+     * list.
      */
     void injectDependencies(List<Object> objects);
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjectorAbstract.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjectorAbstract.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjectorAbstract.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/ServicesInjectorAbstract.java Sun Jan 22 06:58:43 2012
@@ -153,8 +153,7 @@ public abstract class ServicesInjectorAb
                     continue;
                 }
                 final Class<?>[] parameterTypes = methods[j].getParameterTypes();
-                if (parameterTypes.length != 1 || parameterTypes[0] == Object.class
-                    || !parameterTypes[0].isAssignableFrom(serviceClass)) {
+                if (parameterTypes.length != 1 || parameterTypes[0] == Object.class || !parameterTypes[0].isAssignableFrom(serviceClass)) {
                     continue;
                 }
 
@@ -169,13 +168,11 @@ public abstract class ServicesInjectorAb
         try {
             method.invoke(target, parameters);
         } catch (final SecurityException e) {
-            throw new MetaModelException(String.format("Cannot access the %s method in %s", method.getName(), target
-                .getClass().getName()));
+            throw new MetaModelException(String.format("Cannot access the %s method in %s", method.getName(), target.getClass().getName()));
         } catch (final IllegalArgumentException e1) {
             throw new MetaModelException(e1);
         } catch (final IllegalAccessException e1) {
-            throw new MetaModelException(String.format("Cannot access the %s method in %s", method.getName(), target
-                .getClass().getName()));
+            throw new MetaModelException(String.format("Cannot access the %s method in %s", method.getName(), target.getClass().getName()));
         } catch (final InvocationTargetException e) {
             final Throwable targetException = e.getTargetException();
             if (targetException instanceof RuntimeException) {

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/DomainObjectContainerDefault.java Sun Jan 22 06:58:43 2012
@@ -57,8 +57,7 @@ import org.apache.isis.core.metamodel.sp
 import org.apache.isis.core.metamodel.spec.SpecificationLookup;
 import org.apache.isis.core.metamodel.spec.SpecificationLookupAware;
 
-public class DomainObjectContainerDefault implements DomainObjectContainer, QuerySubmitterAware, ObjectDirtierAware,
-    DomainObjectServicesAware, ObjectPersistorAware, SpecificationLookupAware, AuthenticationSessionProviderAware, AdapterMapAware, LocalizationProviderAware {
+public class DomainObjectContainerDefault implements DomainObjectContainer, QuerySubmitterAware, ObjectDirtierAware, DomainObjectServicesAware, ObjectPersistorAware, SpecificationLookupAware, AuthenticationSessionProviderAware, AdapterMapAware, LocalizationProviderAware {
 
     private ObjectDirtier objectDirtier;
     private ObjectPersistor objectPersistor;
@@ -73,18 +72,16 @@ public class DomainObjectContainerDefaul
 
     }
 
-
     // //////////////////////////////////////////////////////////////////
     // titleOf
     // //////////////////////////////////////////////////////////////////
 
     @Override
-    public String titleOf(Object domainObject) {
+    public String titleOf(final Object domainObject) {
         final ObjectAdapter objectAdapter = adapterMap.adapterFor(domainObject);
         return objectAdapter.getSpecification().getTitle(objectAdapter, localizationProvider.getLocalization());
     }
 
-
     // //////////////////////////////////////////////////////////////////
     // newInstance, disposeInstance
     // //////////////////////////////////////////////////////////////////
@@ -104,35 +101,41 @@ public class DomainObjectContainerDefaul
             return (T) adapter.getObject();
         }
     }
-    
+
     @Override
     @SuppressWarnings("unchecked")
-    public <T> T newAggregatedInstance(Object parent, Class<T> ofClass) {
+    public <T> T newAggregatedInstance(final Object parent, final Class<T> ofClass) {
         final ObjectSpecification spec = getSpecificationLookup().loadSpecification(ofClass);
         // TODO check aggregation is supported
         if (!spec.isAggregated()) {
-            throw new IsisException("Cannot instantiate an object unless it is marked as Aggregated using the newAggregatedInstance method: " + ofClass); // TODO proper type
+            throw new IsisException("Cannot instantiate an object unless it is marked as Aggregated using the newAggregatedInstance method: " + ofClass); // TODO
+                                                                                                                                                          // proper
+                                                                                                                                                          // type
         }
         final ObjectAdapter adapter = doCreateAggregatedInstance(spec, parent);
         if (adapter.getOid() instanceof AggregatedOid) {
             return (T) adapter.getObject();
         } else {
-            throw new IsisException("Object instatiated but was not given a AggregatedOid: " + ofClass); // TODO proper type            
+            throw new IsisException("Object instatiated but was not given a AggregatedOid: " + ofClass); // TODO
+                                                                                                         // proper
+                                                                                                         // type
         }
     }
 
     /**
-     * Returns a new instance of the specified class that will have been persisted.
+     * Returns a new instance of the specified class that will have been
+     * persisted.
      */
     @Override
     public <T> T newPersistentInstance(final Class<T> ofClass) {
-        T newInstance = newTransientInstance(ofClass);
+        final T newInstance = newTransientInstance(ofClass);
         persist(newInstance);
         return newInstance;
     }
 
     /**
-     * Returns a new instance of the specified class that has the sane persisted state as the specified object.
+     * Returns a new instance of the specified class that has the sane persisted
+     * state as the specified object.
      */
     @Override
     public <T> T newInstance(final Class<T> ofClass, final Object object) {
@@ -149,9 +152,9 @@ public class DomainObjectContainerDefaul
     protected ObjectAdapter doCreateTransientInstance(final ObjectSpecification spec) {
         return getDomainObjectServices().createTransientInstance(spec);
     }
-    
-    private ObjectAdapter doCreateAggregatedInstance(ObjectSpecification spec, Object parent) {
-        ObjectAdapter parentAdapter = getAdapterMap().getAdapterFor(parent);
+
+    private ObjectAdapter doCreateAggregatedInstance(final ObjectSpecification spec, final Object parent) {
+        final ObjectAdapter parentAdapter = getAdapterMap().getAdapterFor(parent);
         return getDomainObjectServices().createAggregatedInstance(spec, parentAdapter);
     }
 
@@ -221,7 +224,7 @@ public class DomainObjectContainerDefaul
     @Override
     public String validate(final Object domainObject) {
         final ObjectAdapter adapter = getAdapterMap().adapterFor(domainObject);
-        InteractionResult validityResult = adapter.getSpecification().isValidResult(adapter);
+        final InteractionResult validityResult = adapter.getSpecification().isValidResult(adapter);
         return validityResult.getReason();
     }
 
@@ -271,10 +274,10 @@ public class DomainObjectContainerDefaul
         return user;
     }
 
-    private List<RoleMemento> asRoleMementos(List<String> roles) {
-        List<RoleMemento> mementos = new ArrayList<RoleMemento>();
+    private List<RoleMemento> asRoleMementos(final List<String> roles) {
+        final List<RoleMemento> mementos = new ArrayList<RoleMemento>();
         if (roles != null) {
-            for (String role : roles) {
+            for (final String role : roles) {
                 mementos.add(new RoleMemento(role));
             }
         }
@@ -286,13 +289,13 @@ public class DomainObjectContainerDefaul
     // //////////////////////////////////////////////////////////////////
 
     @Override
-    public String getProperty(String name) {
+    public String getProperty(final String name) {
         return getDomainObjectServices().getProperty(name);
     }
 
     @Override
-    public String getProperty(String name, String defaultValue) {
-        String value = getProperty(name);
+    public String getProperty(final String name, final String defaultValue) {
+        final String value = getProperty(name);
         return value == null ? defaultValue : value;
     }
 
@@ -333,7 +336,7 @@ public class DomainObjectContainerDefaul
     public <T> List<T> allMatches(final Class<T> cls, final Filter<? super T> filter) {
         final List<T> allInstances = allInstances(cls);
         final List<T> filtered = new ArrayList<T>();
-        for (T instance : allInstances) {
+        for (final T instance : allInstances) {
             if (filter.accept(instance)) {
                 filtered.add(instance);
             }
@@ -354,7 +357,7 @@ public class DomainObjectContainerDefaul
 
     @Override
     public <T> List<T> allMatches(final Query<T> query) {
-        List<ObjectAdapter> allMatching = getQuerySubmitter().allMatchingQuery(query);
+        final List<ObjectAdapter> allMatching = getQuerySubmitter().allMatchingQuery(query);
         return AdapterUtils.unwrap(allMatching);
     }
 
@@ -365,7 +368,7 @@ public class DomainObjectContainerDefaul
     @Override
     public <T> T firstMatch(final Class<T> cls, final Filter<T> filter) {
         final List<T> allInstances = allInstances(cls);
-        for (T instance : allInstances) {
+        for (final T instance : allInstances) {
             if (filter.accept(instance)) {
                 return instance;
             }
@@ -388,7 +391,7 @@ public class DomainObjectContainerDefaul
     @Override
     @SuppressWarnings("unchecked")
     public <T> T firstMatch(final Query<T> query) {
-        ObjectAdapter firstMatching = getQuerySubmitter().firstMatchingQuery(query);
+        final ObjectAdapter firstMatching = getQuerySubmitter().firstMatchingQuery(query);
         return (T) AdapterUtils.unwrap(firstMatching);
     }
 
@@ -445,7 +448,7 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setQuerySubmitter(QuerySubmitter querySubmitter) {
+    public void setQuerySubmitter(final QuerySubmitter querySubmitter) {
         this.querySubmitter = querySubmitter;
     }
 
@@ -454,7 +457,7 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setDomainObjectServices(DomainObjectServices domainObjectServices) {
+    public void setDomainObjectServices(final DomainObjectServices domainObjectServices) {
         this.domainObjectServices = domainObjectServices;
     }
 
@@ -463,7 +466,7 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setSpecificationLookup(SpecificationLookup specificationLookup) {
+    public void setSpecificationLookup(final SpecificationLookup specificationLookup) {
         this.specificationLookup = specificationLookup;
     }
 
@@ -472,7 +475,7 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setAuthenticationSessionProvider(AuthenticationSessionProvider authenticationSessionProvider) {
+    public void setAuthenticationSessionProvider(final AuthenticationSessionProvider authenticationSessionProvider) {
         this.authenticationSessionProvider = authenticationSessionProvider;
     }
 
@@ -481,7 +484,7 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setAdapterMap(AdapterMap adapterManager) {
+    public void setAdapterMap(final AdapterMap adapterManager) {
         this.adapterMap = adapterManager;
     }
 
@@ -490,7 +493,7 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setObjectDirtier(ObjectDirtier objectDirtier) {
+    public void setObjectDirtier(final ObjectDirtier objectDirtier) {
         this.objectDirtier = objectDirtier;
     }
 
@@ -499,12 +502,12 @@ public class DomainObjectContainerDefaul
     }
 
     @Override
-    public void setObjectPersistor(ObjectPersistor objectPersistor) {
+    public void setObjectPersistor(final ObjectPersistor objectPersistor) {
         this.objectPersistor = objectPersistor;
     }
-    
+
     @Override
-    public void setLocalizationProvider(LocalizationProvider localizationProvider) {
+    public void setLocalizationProvider(final LocalizationProvider localizationProvider) {
         this.localizationProvider = localizationProvider;
     }
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByPattern.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByPattern.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByPattern.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByPattern.java Sun Jan 22 06:58:43 2012
@@ -25,8 +25,10 @@ import org.apache.isis.applib.query.Quer
 import org.apache.isis.applib.query.QueryBuiltInAbstract;
 
 /**
- * Although implements {@link Query} and thus is ought to be {@link Serializable}, it will be converted into a
- * <tt>PersistenceQuery</tt> in the runtime for remoting purposes and so does not need to be (and indeed isn't).
+ * Although implements {@link Query} and thus is ought to be
+ * {@link Serializable}, it will be converted into a <tt>PersistenceQuery</tt>
+ * in the runtime for remoting purposes and so does not need to be (and indeed
+ * isn't).
  * 
  * <p>
  * See discussion in {@link QueryBuiltInAbstract} for further details.

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByTitle.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByTitle.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByTitle.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/services/container/query/QueryFindByTitle.java Sun Jan 22 06:58:43 2012
@@ -26,8 +26,9 @@ import org.apache.isis.applib.query.Quer
 import org.apache.isis.applib.query.QueryBuiltInAbstract;
 
 /**
- * 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/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ActionType.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ActionType.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ActionType.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ActionType.java Sun Jan 22 06:58:43 2012
@@ -31,15 +31,29 @@ public enum ActionType {
         return name();
     }
 
-    public boolean matchesTypeOf(ObjectAction action) {
+    public boolean matchesTypeOf(final ObjectAction action) {
         return action != null && action.getType().equals(this);
     }
-    
-    public boolean isDebug() { return this == DEBUG; }
-    public boolean isSet() { return this == SET; }
-    public boolean isExploration() { return this == EXPLORATION; }
-    public boolean isPrototype() { return this == PROTOTYPE; }
-    public boolean isUser() { return this == USER; }
-    
+
+    public boolean isDebug() {
+        return this == DEBUG;
+    }
+
+    public boolean isSet() {
+        return this == SET;
+    }
+
+    public boolean isExploration() {
+        return this == EXPLORATION;
+    }
+
+    public boolean isPrototype() {
+        return this == PROTOTYPE;
+    }
+
+    public boolean isUser() {
+        return this == USER;
+    }
+
     public static final List<ActionType> ALL_EXCEPT_SET = Arrays.asList(DEBUG, EXPLORATION, PROTOTYPE, USER);
 }

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DefaultProvider.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DefaultProvider.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DefaultProvider.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DefaultProvider.java Sun Jan 22 06:58:43 2012
@@ -22,8 +22,9 @@ package org.apache.isis.core.metamodel.s
 public interface DefaultProvider {
 
     /**
-     * Default value to be provided for properties or parameters that are not declared as <tt>@Optional</tt> but where
-     * the UI has not (yet) provided a value.
+     * Default value to be provided for properties or parameters that are not
+     * declared as <tt>@Optional</tt> but where the UI has not (yet) provided a
+     * value.
      */
     Object getDefaultValue();
 

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Described.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Described.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Described.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Described.java Sun Jan 22 06:58:43 2012
@@ -25,7 +25,8 @@ package org.apache.isis.core.metamodel.s
 public interface Described {
 
     /**
-     * Returns a description of how the member is used - this complements the help text.
+     * Returns a description of how the member is used - this complements the
+     * help text.
      * 
      * @see #getHelp()
      */

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Dirtiable.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Dirtiable.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Dirtiable.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/Dirtiable.java Sun Jan 22 06:58:43 2012
@@ -24,8 +24,8 @@ import org.apache.isis.core.metamodel.ad
 public interface Dirtiable {
 
     /**
-     * Clear the dirty flag so that a call to <tt>isDirty()</tt>, and before <tt>markDirty()</tt> is called, will return
-     * false;
+     * Clear the dirty flag so that a call to <tt>isDirty()</tt>, and before
+     * <tt>markDirty()</tt> is called, will return false;
      * 
      * @see #isDirty(ObjectAdapter)
      * @see #markDirty(ObjectAdapter)
@@ -33,7 +33,8 @@ public interface Dirtiable {
     void clearDirty(ObjectAdapter object);
 
     /**
-     * Checks if the specified object has been changed, and hence needs persisting.
+     * Checks if the specified object has been changed, and hence needs
+     * persisting.
      * 
      * @see #markDirty(ObjectAdapter)
      * @see #clearDirty(ObjectAdapter)
@@ -41,7 +42,8 @@ public interface Dirtiable {
     boolean isDirty(ObjectAdapter object);
 
     /**
-     * Mark the specified object as having been changed, and hence needing persisting.
+     * Mark the specified object as having been changed, and hence needing
+     * persisting.
      * 
      * @see #isDirty(ObjectAdapter)
      * @see #clearDirty(ObjectAdapter)

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DomainModelException.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DomainModelException.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DomainModelException.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/DomainModelException.java Sun Jan 22 06:58:43 2012
@@ -22,7 +22,8 @@ package org.apache.isis.core.metamodel.s
 import org.apache.isis.core.commons.exceptions.IsisException;
 
 /**
- * Thrown when a problem is found with the domain model, and Isis cannot proceed.
+ * Thrown when a problem is found with the domain model, and Isis cannot
+ * proceed.
  */
 public class DomainModelException extends IsisException {
     private static final long serialVersionUID = 1L;

Modified: incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ElementSpecificationProvider.java
URL: http://svn.apache.org/viewvc/incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ElementSpecificationProvider.java?rev=1234453&r1=1234452&r2=1234453&view=diff
==============================================================================
--- incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ElementSpecificationProvider.java (original)
+++ incubator/isis/trunk/framework/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/ElementSpecificationProvider.java Sun Jan 22 06:58:43 2012
@@ -17,10 +17,12 @@
 package org.apache.isis.core.metamodel.spec;
 
 /**
- * A mechanism to provide the {@link ObjectSpecification type} of a (stand-alone) collection.
+ * A mechanism to provide the {@link ObjectSpecification type} of a
+ * (stand-alone) collection.
  * 
  * <p>
- * Introduced to decouple the <tt>facets</tt> package from the <tt>spec</tt> package.
+ * Introduced to decouple the <tt>facets</tt> package from the <tt>spec</tt>
+ * package.
  */
 public interface ElementSpecificationProvider {
 



Mime
View raw message