Browse Source

statistics sorting bug fixed

Julien Clauter 10 years ago
parent
commit
8c36cbda33

+ 10 - 3
src/de/tudarmstadt/informatik/hostage/ui2/fragment/StatisticsFragment.java

@@ -1040,7 +1040,7 @@ public class StatisticsFragment extends Fragment implements ChecklistDialog.Chec
          Collections.sort(plotItems, new Comparator<PlotComparisonItem>() {
              @Override
              public int compare(PlotComparisonItem s1, PlotComparisonItem s2) {
-                 return s1.getValue2().compareTo(s2.getValue2());
+                 return s2.getValue2().compareTo(s1.getValue2());
              }
          });
          return this.resizeData(plotItems);
@@ -1153,7 +1153,7 @@ public class StatisticsFragment extends Fragment implements ChecklistDialog.Chec
         Collections.sort(plotItems, new Comparator<PlotComparisonItem>() {
             @Override
             public int compare(PlotComparisonItem s1, PlotComparisonItem s2) {
-                return s1.getValue2().compareTo(s2.getValue2());
+                return s2.getValue2().compareTo(s1.getValue2());
             }
         });
         return this.resizeData(plotItems);
@@ -1190,7 +1190,7 @@ public class StatisticsFragment extends Fragment implements ChecklistDialog.Chec
         Collections.sort(plotItems, new Comparator<PlotComparisonItem>() {
             @Override
             public int compare(PlotComparisonItem s1, PlotComparisonItem s2) {
-                return s1.getValue2().compareTo(s2.getValue2());
+                return s2.getValue2().compareTo(s1.getValue2());
             }
         });
 
@@ -1220,6 +1220,13 @@ public class StatisticsFragment extends Fragment implements ChecklistDialog.Chec
                 otherItem.setOtherData(others);
                 copy.add(otherItem);
 
+                Collections.sort(copy, new Comparator<PlotComparisonItem>() {
+                    @Override
+                    public int compare(PlotComparisonItem s1, PlotComparisonItem s2) {
+                        return s2.getValue2().compareTo(s1.getValue2());
+                    }
+                });
+
                 return copy;
             }
         }