[CAUSEWAY-324] ToDoItem.compare compares against itself.

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: archetype-wrj-1.0.2
• Fix Version/s: archetype-wrj-1.0.3
• Assignee: Daniel Keir Haywood
• Reporter: Minto van der Sluis
• Created: 03/Feb/13 09:31
• Updated: 20/Feb/15 14:07
• Resolved: 03/Feb/13 14:58

Description

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());