You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Dan Haywood (JIRA)" <ji...@apache.org> on 2014/06/10 01:05:04 UTC
[jira] [Closed] (ISIS-324) ToDoItem.compare compares against
itself.
[ https://issues.apache.org/jira/browse/ISIS-324?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dan Haywood closed ISIS-324.
----------------------------
> ToDoItem.compare compares against itself.
> -----------------------------------------
>
> Key: ISIS-324
> URL: https://issues.apache.org/jira/browse/ISIS-324
> Project: Isis
> Issue Type: Bug
> Components: Archetype: ToDoApp WRJ
> Affects Versions: archetype-wrj-1.0.2
> Reporter: Minto van der Sluis
> Assignee: Dan Haywood
> Priority: Minor
> Fix For: archetype-wrj-1.0.3
>
>
> final statements in the compare method compares with itself:
> if (getDueBy() == null && other.getDueBy() == null || getDueBy().equals(this.getDueBy())) {
> return getDescription().compareTo(other.getDescription());
> }
> return getDueBy().compareTo(getDueBy());
> Should be:
> if (getDueBy() == null && other.getDueBy() == null || getDueBy().equals(other.getDueBy())) {
> return getDescription().compareTo(other.getDescription());
> }
> return getDueBy().compareTo(other.getDueBy());
--
This message was sent by Atlassian JIRA
(v6.2#6252)