update for release 1.3.4

Author: timpavone1990

pom.xml

@@ -5,7 +5,7 @@
 
     <groupId>com.espirit.moddev</groupId>
     <artifactId>basicworkflows</artifactId>
-    <version>1.3.2</version>
+    <version>1.3.4</version>
     <packaging>jar</packaging>
     <name>${project.displayName}</name>
     <inceptionYear>2018</inceptionYear>

src/main/java/com/espirit/moddev/basicworkflows/util/StoreComparator.java

@@ -49,14 +49,24 @@ public int compare(final Object o1, final Object o2) {
         }
         final Type type1 = storeElement1.getStore().getType();
         final Type type2 = storeElement2.getStore().getType();
-        if (isPageStoreObject(type1)) {
-            if (isPageStoreObject(type2)) {
+        // MediaStoreObject < PageStoreObject < SiteStoreObject < others
+        if (isMediaStoreObject(type1)) {
+            if (isMediaStoreObject(type2)) {
                 return 0;
             } else {
                 return -1;
             }
-        } else { 
-            if (isPageStoreObject(type2)) {
+        } else if (isPageStoreObject(type1)) {
+            if (isMediaStoreObject(type2)) {
+                return 1;
+            }
+            else if (isPageStoreObject(type2)) {
+                return 0;
+            } else {
+                return -1;
+            }
+        } else {
+            if (isMediaStoreObject(type2) || isPageStoreObject(type2)) {
                 return 1;
             } else if (isSiteStoreObject(type2)) {
                 return 0;
@@ -66,6 +76,9 @@ public int compare(final Object o1, final Object o2) {
         }
     }
 
+    private boolean isMediaStoreObject(final Type type) {
+        return type == Store.Type.MEDIASTORE;
+    }
 
     private boolean isPageStoreObject(final Type type) {
         return type == Store.Type.PAGESTORE;

src/test/java/com/espirit/moddev/basicworkflows/util/StoreComparatorTest.java

@@ -42,12 +42,12 @@ public void testStoreComparator() throws Exception {
 
 		Collections.sort(releaseObjects, new StoreComparator());
 
-		assertEquals(Store.Type.PAGESTORE, releaseObjects.get(0).getStore().getType());
-		assertEquals(Store.Type.PAGESTORE, releaseObjects.get(1).getStore().getType());
-		assertEquals(Store.Type.SITESTORE, releaseObjects.get(2).getStore().getType());
-		assertEquals(Store.Type.SITESTORE, releaseObjects.get(3).getStore().getType());
-		assertEquals(Store.Type.MEDIASTORE, releaseObjects.get(4).getStore().getType());
-		assertEquals(Store.Type.MEDIASTORE, releaseObjects.get(5).getStore().getType());
+		assertEquals(Store.Type.MEDIASTORE, releaseObjects.get(0).getStore().getType());
+		assertEquals(Store.Type.MEDIASTORE, releaseObjects.get(1).getStore().getType());
+		assertEquals(Store.Type.PAGESTORE, releaseObjects.get(2).getStore().getType());
+		assertEquals(Store.Type.PAGESTORE, releaseObjects.get(3).getStore().getType());
+		assertEquals(Store.Type.SITESTORE, releaseObjects.get(4).getStore().getType());
+		assertEquals(Store.Type.SITESTORE, releaseObjects.get(5).getStore().getType());
 	}