fixes issue sorter not recognizing issues as "equal" if their ETAs are both null

Sat, 31 Oct 2020 09:56:00 +0100

author
Mike Becker <universe@uap-core.de>
date
Sat, 31 Oct 2020 09:56:00 +0100
changeset 155
3ace475224f7
parent 154
3d10f2a390a1
child 156
c5d6820d884e

fixes issue sorter not recognizing issues as "equal" if their ETAs are both null

src/main/java/de/uapcore/lightpit/viewmodel/util/IssueSorter.java file | annotate | diff | comparison | revisions
     1.1 --- a/src/main/java/de/uapcore/lightpit/viewmodel/util/IssueSorter.java	Sat Oct 31 09:50:15 2020 +0100
     1.2 +++ b/src/main/java/de/uapcore/lightpit/viewmodel/util/IssueSorter.java	Sat Oct 31 09:56:00 2020 +0100
     1.3 @@ -37,6 +37,9 @@
     1.4      }
     1.5  
     1.6      private int compare(Issue left, Issue right, Criteria criteria) {
     1.7 +        if (left.equals(right))
     1.8 +            return 0;
     1.9 +
    1.10          int result;
    1.11          switch (criteria.field) {
    1.12              case DONE:
    1.13 @@ -47,6 +50,8 @@
    1.14              case ETA:
    1.15                  if (left.getEta() != null && right.getEta() != null)
    1.16                      result = left.getEta().compareTo(right.getEta());
    1.17 +                else if (left.getEta() == null && right.getEta() == null)
    1.18 +                    result = 0;
    1.19                  else
    1.20                      result = left.getEta() != null ? -1 : 1;
    1.21                  break;

mercurial