Project

General

Profile

Bug #4424

KeplerLSID should implement hashCode

Added by Derik Barseghian over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
core
Target version:
Start date:
09/29/2009
Due date:
% Done:

0%

Estimated time:
Bugzilla-Id:
4424

Description

KeplerLSID overrides equals, and so it should therefore override hashCode as well.

I suspect this is why doing a map.containsKey(lsid) does not work (the lsid key is not found, even if it's in there). There was a bug due to this issue in KARFile. I implemented a little workflow-around and will add a note in the code referring to this bug. If implementing hashCode() does fix the containsKey issue, we should revert to the commented out code, as it's cleaner.

History

#1 Updated by Aaron Aaron over 9 years ago

hashCode is implemented now, there's a JUnit test called KeplerLSIDTest that shows the contains method is working properly now on hashed KeplerLSID objects.

#2 Updated by Redmine Admin about 6 years ago

Original Bugzilla ID was 4424

Also available in: Atom PDF