Fix IllegalArgumentException on files with dots in name, add test case

src/main/java/third_parties/daveKoeller/AlphanumComparator.java

@@ -168,9 +168,9 @@ public class AlphanumComparator<T> implements Comparator<T>, Serializable {
                 }
             } else if (isSpecialChar(thisChunk.charAt(0)) && isSpecialChar(thatChunk.charAt(0))) {
                 for (int i = 0; i < thisChunk.length(); i++) {
-                    if (thisChunk.charAt(i) == '.') {
+                    if (thisChunk.charAt(i) == '.' && thatChunk.charAt(i) != '.') {
                         return -1;
-                    } else if (thatChunk.charAt(i) == '.') {
+                    } else if (thatChunk.charAt(i) == '.' && thisChunk.charAt(i) != '.') {
                         return 1;
                     } else {
                         result = thisChunk.charAt(i) - thatChunk.charAt(i);

src/test/java/com/owncloud/android/utils/TestSorting.java

@@ -158,6 +158,215 @@ public class TestSorting {
         fileSortOrderByName.sortCloudFiles(array);
     }
 
+    @Test
+    public void testSortCloudFilesWithDots() {
+        List<OCFile> array = new ArrayList<>();
+
+        array.add(new OCFile("/2013.10.804"));
+        array.add(new OCFile("/2013.05.1754"));
+        array.add(new OCFile("/2012.05.9095"));
+        array.add(new OCFile("/2015.07.845"));
+        array.add(new OCFile("/2017.07.2056"));
+        array.add(new OCFile("/2016.05.8586"));
+        array.add(new OCFile("/2011.05.5902"));
+        array.add(new OCFile("/2016.06.782"));
+        array.add(new OCFile("/2010.01.8447"));
+        array.add(new OCFile("/2010.07.8126"));
+        array.add(new OCFile("/2015.05.7767"));
+        array.add(new OCFile("/2013.05.2216"));
+        array.add(new OCFile("/2015.08.3915"));
+        array.add(new OCFile("/2014.09.9809"));
+        array.add(new OCFile("/2017.01.962"));
+        array.add(new OCFile("/2011.09.1398"));
+        array.add(new OCFile("/2015.09.7108"));
+        array.add(new OCFile("/2013.07.6003"));
+        array.add(new OCFile("/2010.05.7857"));
+        array.add(new OCFile("/2016.05.4529"));
+        array.add(new OCFile("/2017.05.1116"));
+        array.add(new OCFile("/2017.09.3978"));
+        array.add(new OCFile("/2015.05.5681"));
+        array.add(new OCFile("/2010.07.4053"));
+        array.add(new OCFile("/2015.07.8494"));
+        array.add(new OCFile("/2014.09.4869"));
+        array.add(new OCFile("/2017.03.786"));
+        array.add(new OCFile("/2017.08.7493"));
+        array.add(new OCFile("/2016.06.1030"));
+        array.add(new OCFile("/2013.02.861"));
+        array.add(new OCFile("/2016.10.1857"));
+        array.add(new OCFile("/2016.02.5648"));
+        array.add(new OCFile("/2015.06.7667"));
+        array.add(new OCFile("/2017.04.4579"));
+        array.add(new OCFile("/2015.03.3019"));
+        array.add(new OCFile("/2011.08.2122"));
+        array.add(new OCFile("/2017.11.5763"));
+        array.add(new OCFile("/2014.04.3249"));
+        array.add(new OCFile("/2017.01.2733"));
+        array.add(new OCFile("/2010.03.5635"));
+        array.add(new OCFile("/2011.08.2065"));
+        array.add(new OCFile("/2014.05.7808"));
+        array.add(new OCFile("/2013.04.1710"));
+        array.add(new OCFile("/2012.08.8905"));
+        array.add(new OCFile("/2011.11.5763"));
+        array.add(new OCFile("/2011.04.5001"));
+        array.add(new OCFile("/2014.06.2181"));
+        array.add(new OCFile("/2017.09.1846"));
+        array.add(new OCFile("/2010.11.8127"));
+        array.add(new OCFile("/2012.08.841"));
+        array.add(new OCFile("/2017.03.2849"));
+        array.add(new OCFile("/2014.08.7792"));
+        array.add(new OCFile("/2017.09.3665"));
+        array.add(new OCFile("/2014.07.7095"));
+        array.add(new OCFile("/2014.07.7342"));
+        array.add(new OCFile("/2017.11.920"));
+        array.add(new OCFile("/2010.02.6786"));
+        array.add(new OCFile("/2016.09.3652"));
+        array.add(new OCFile("/2010.04.5241"));
+        array.add(new OCFile("/2011.08.2295"));
+        array.add(new OCFile("/2015.01.8283"));
+        array.add(new OCFile("/2015.05.9830"));
+        array.add(new OCFile("/2011.06.1760"));
+        array.add(new OCFile("/2015.06.7024"));
+        array.add(new OCFile("/2012.03.3599"));
+        array.add(new OCFile("/2010.08.5827"));
+        array.add(new OCFile("/2017.01.1958"));
+        array.add(new OCFile("/2013.11.2766"));
+        array.add(new OCFile("/2017.04.7168"));
+        array.add(new OCFile("/2015.10.645"));
+        array.add(new OCFile("/2014.02.8980"));
+        array.add(new OCFile("/2017.04.5291"));
+        array.add(new OCFile("/2013.05.7761"));
+        array.add(new OCFile("/2012.09.5068"));
+        array.add(new OCFile("/2015.02.6614"));
+        array.add(new OCFile("/2015.04.7370"));
+        array.add(new OCFile("/2017.01.4330"));
+        array.add(new OCFile("/2011.07.6014"));
+        array.add(new OCFile("/2011.03.7712"));
+        array.add(new OCFile("/2015.11.1950"));
+        array.add(new OCFile("/2016.04.6305"));
+        array.add(new OCFile("/2012.11.2195"));
+        array.add(new OCFile("/2010.06.7521"));
+        array.add(new OCFile("/2010.05.1230"));
+        array.add(new OCFile("/2016.08.8918"));
+        array.add(new OCFile("/2013.05.8741"));
+        array.add(new OCFile("/2012.07.2853"));
+        array.add(new OCFile("/2016.03.8389"));
+        array.add(new OCFile("/2012.02.935"));
+        array.add(new OCFile("/2012.01.2319"));
+        array.add(new OCFile("/2010.04.9515"));
+        array.add(new OCFile("/2015.05.3916"));
+        array.add(new OCFile("/2010.06.5763"));
+        array.add(new OCFile("/2016.04.6043"));
+        array.add(new OCFile("/2013.08.8660"));
+        array.add(new OCFile("/2013.04.3372"));
+        array.add(new OCFile("/2013.11.8259"));
+        array.add(new OCFile("/2014.02.4056"));
+        array.add(new OCFile("/2011.11.4597"));
+        array.add(new OCFile("/2015.05.4675"));
+        array.add(new OCFile("/2016.08.7836"));
+        array.add(new OCFile("/2015.08.2715"));
+        array.add(new OCFile("/2017.01.7891"));
+        array.add(new OCFile("/2010.11.7728"));
+        array.add(new OCFile("/2012.01.2966"));
+        array.add(new OCFile("/2014.02.6094"));
+        array.add(new OCFile("/2011.06.1937"));
+        array.add(new OCFile("/2017.02.5664"));
+        array.add(new OCFile("/2011.06.2733"));
+        array.add(new OCFile("/2013.10.1946"));
+        array.add(new OCFile("/2017.08.1603"));
+        array.add(new OCFile("/2015.05.7043"));
+        array.add(new OCFile("/2014.09.8531"));
+        array.add(new OCFile("/2010.09.6730"));
+        array.add(new OCFile("/2014.04.8584"));
+        array.add(new OCFile("/2012.01.9713"));
+        array.add(new OCFile("/2014.05.2610"));
+        array.add(new OCFile("/2014.02.5390"));
+        array.add(new OCFile("/2014.05.5375"));
+        array.add(new OCFile("/2012.08.170"));
+        array.add(new OCFile("/2016.01.9777"));
+        array.add(new OCFile("/2014.02.3514"));
+        array.add(new OCFile("/2016.09.6503"));
+        array.add(new OCFile("/2011.11.5326"));
+        array.add(new OCFile("/2017.07.6682"));
+        array.add(new OCFile("/2011.10.2285"));
+        array.add(new OCFile("/2012.03.3784"));
+        array.add(new OCFile("/2015.01.812"));
+        array.add(new OCFile("/2015.02.2764"));
+        array.add(new OCFile("/2014.03.8426"));
+        array.add(new OCFile("/2015.02.1406"));
+        array.add(new OCFile("/2012.03.2638"));
+        array.add(new OCFile("/2016.07.5437"));
+        array.add(new OCFile("/2017.02.41"));
+        array.add(new OCFile("/2011.08.7430"));
+        array.add(new OCFile("/2014.11.8557"));
+        array.add(new OCFile("/2010.09.2319"));
+        array.add(new OCFile("/2010.01.5466"));
+        array.add(new OCFile("/2015.04.5635"));
+        array.add(new OCFile("/2017.07.7894"));
+        array.add(new OCFile("/2015.09.9047"));
+        array.add(new OCFile("/2017.02.1957"));
+        array.add(new OCFile("/2011.07.1017"));
+        array.add(new OCFile("/2012.01.3098"));
+        array.add(new OCFile("/2011.11.3716"));
+        array.add(new OCFile("/2016.08.2250"));
+        array.add(new OCFile("/2011.11.5201"));
+        array.add(new OCFile("/2013.04.2152"));
+        array.add(new OCFile("/2017.03.4025"));
+        array.add(new OCFile("/2014.08.7971"));
+        array.add(new OCFile("/2012.11.6968"));
+        array.add(new OCFile("/2014.04.105"));
+        array.add(new OCFile("/2014.07.2094"));
+        array.add(new OCFile("/2010.02.2724"));
+        array.add(new OCFile("/2016.09.4720"));
+        array.add(new OCFile("/2014.11.3443"));
+        array.add(new OCFile("/2016.10.38"));
+        array.add(new OCFile("/2015.11.3243"));
+        array.add(new OCFile("/2014.08.365"));
+        array.add(new OCFile("/2012.02.9249"));
+        array.add(new OCFile("/2011.01.3175"));
+        array.add(new OCFile("/2017.06.1672"));
+        array.add(new OCFile("/2013.09.7070"));
+        array.add(new OCFile("/2012.11.9046"));
+        array.add(new OCFile("/2012.06.7947"));
+        array.add(new OCFile("/2012.03.8055"));
+        array.add(new OCFile("/2011.04.4308"));
+        array.add(new OCFile("/2016.05.6597"));
+        array.add(new OCFile("/2015.04.9059"));
+        array.add(new OCFile("/2011.09.7857"));
+        array.add(new OCFile("/2016.06.5603"));
+        array.add(new OCFile("/2011.06.2954"));
+        array.add(new OCFile("/2016.11.4425"));
+        array.add(new OCFile("/2015.11.9182"));
+        array.add(new OCFile("/2013.11.1929"));
+        array.add(new OCFile("/2016.08.7554"));
+        array.add(new OCFile("/2012.06.206"));
+        array.add(new OCFile("/2015.04.2022"));
+        array.add(new OCFile("/2013.07.1882"));
+        array.add(new OCFile("/2011.10.7269"));
+        array.add(new OCFile("/2017.08.7545"));
+        array.add(new OCFile("/2011.05.5452"));
+        array.add(new OCFile("/2017.03.4545"));
+        array.add(new OCFile("/2012.03.2416"));
+        array.add(new OCFile("/2011.06.2990"));
+        array.add(new OCFile("/2013.09.7388"));
+        array.add(new OCFile("/2016.10.8252"));
+        array.add(new OCFile("/2011.10.1147"));
+        array.add(new OCFile("/2010.09.3989"));
+        array.add(new OCFile("/2011.09.2769"));
+        array.add(new OCFile("/2010.06.1591"));
+        array.add(new OCFile("/2011.10.4875"));
+        array.add(new OCFile("/2011.02.9824"));
+        array.add(new OCFile("/2011.02.3068"));
+        array.add(new OCFile("/2010.04.9549"));
+        array.add(new OCFile("/2017.02.593"));
+        array.add(new OCFile("/2011.02.1659"));
+        array.add(new OCFile("/2012.08.3684"));
+        array.add(new OCFile("/2017.07.3936"));
+        array.add(new OCFile("/2011.04.8933"));
+
+        FileSortOrderByName fileSortOrderByName = new FileSortOrderByName("test", true);
+        fileSortOrderByName.sortCloudFiles(array);
+    }
+
     private List<Comparable> shuffle(List<? extends Comparable> files) {
         List<Comparable> shuffled = new ArrayList<>();
         shuffled.addAll(files);