This is an automated email from the ASF dual-hosted git repository.

ahuber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/isis.git


The following commit(s) were added to refs/heads/master by this push:
     new 742eacc  FIX ISIS-1738 (#97)
742eacc is described below

commit 742eacc6587c8e09632c438ec8c88cc7c8593bc0
Author: Andi Huber <hob...@gmx.at>
AuthorDate: Fri Oct 6 23:02:05 2017 +0200

    FIX ISIS-1738 (#97)
---
 .../core/metamodel/spec/feature/ObjectMember.java  | 30 +++++++++-------------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git 
a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/feature/ObjectMember.java
 
b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/feature/ObjectMember.java
index 39b2177..c7721e5 100644
--- 
a/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/feature/ObjectMember.java
+++ 
b/core/metamodel/src/main/java/org/apache/isis/core/metamodel/spec/feature/ObjectMember.java
@@ -25,10 +25,6 @@ import java.util.List;
 
 import javax.annotation.Nullable;
 
-import com.google.common.base.Function;
-import com.google.common.collect.ImmutableMap;
-import com.google.common.collect.Maps;
-
 import org.apache.isis.applib.annotation.When;
 import org.apache.isis.applib.annotation.Where;
 import org.apache.isis.core.metamodel.adapter.ObjectAdapter;
@@ -36,7 +32,10 @@ import org.apache.isis.core.metamodel.consent.Consent;
 import org.apache.isis.core.metamodel.consent.InteractionInitiatedBy;
 import org.apache.isis.core.metamodel.facets.all.hide.HiddenFacet;
 import org.apache.isis.core.metamodel.facets.members.order.MemberOrderFacet;
-import org.apache.isis.core.metamodel.util.DeweyOrderComparator;
+import 
org.apache.isis.core.metamodel.layout.memberorderfacet.MemberOrderFacetComparator;
+
+import com.google.common.base.Function;
+import com.google.common.collect.Maps;
 
 /**
  * Provides reflective access to an action or a field on a domain object.
@@ -205,19 +204,14 @@ public interface ObjectMember extends ObjectFeature {
     public static class Comparators {
         public static Comparator<ObjectMember> byMemberOrderSequence() {
             return new Comparator<ObjectMember>() {
-                private final DeweyOrderComparator deweyOrderComparator = new 
DeweyOrderComparator();
-                @Override
-                public int compare(final ObjectMember o1, final ObjectMember 
o2) {
-                    final MemberOrderFacet o1Facet = 
o1.getFacet(MemberOrderFacet.class);
-                    final MemberOrderFacet o2Facet = 
o2.getFacet(MemberOrderFacet.class);
-                    String memberId1 = o1.getId();
-                    String memberId2 = o2.getId();
-                    String o1Sequence = o1Facet != null ? o1Facet.sequence() : 
"0";
-                    String o2Sequence = o2Facet != null ? o2Facet.sequence() : 
"0";
-                    return o1Facet == null? +1:
-                            o2Facet == null? -1:
-                                    deweyOrderComparator.compare(o1Sequence, 
o2Sequence);
-                }
+                private final MemberOrderFacetComparator 
memberOrderFacetComparator = 
+                               new MemberOrderFacetComparator(false);
+                 @Override
+                 public int compare(final ObjectMember o1, final ObjectMember 
o2) {
+                       return memberOrderFacetComparator.compare(
+                                       o1.getFacet(MemberOrderFacet.class),
+                                       o2.getFacet(MemberOrderFacet.class));
+                 }
             };
         }
 

-- 
To stop receiving notification emails like this one, please contact
['"commits@isis.apache.org" <commits@isis.apache.org>'].

Reply via email to