bug #824
closedRank labes and Rank names inconsitent
100%
Description
The rank labels and the names used in Rank.getRankByName() are inconsistent thus the following line of code fails:
Rank.getRankByName(subfamily.getRank().getLabel());
Updated by Andreas Müller over 14 years ago
- Assignee changed from Andreas Müller to k.luther -
- Target version changed from CDM lib Release 2.0 to CDM lib Release 2.3
Rank.getRankByName() is a preliminary implementation to meet the Berlin Model and other import requirements. That has been documented in the java doc.
Anyway we should think about dictionaries for rank names and other defined term vocabularies.
Maybe we should open a new ticket for this and close this one.
Updated by Andreas Müller over 2 years ago
- Description updated (diff)
- Status changed from New to Feedback
- Assignee changed from Katja Luther to Andreas Kohlbecker
- Target version changed from cdmlib - Old Next Major Release to Release 5.45
I guess we can close this ticket?
Updated by Andreas Kohlbecker over 2 years ago
- Assignee changed from Andreas Kohlbecker to Andreas Müller
Yes, still open. I created a Test to reproduce this issue. However it is completely unclear what is being meant by "byName" in the method name ("Returns the rank identified through a name.") There is no way to find out the supported names without reading the code.
Another point the UnknownCdmTypeException that can be thrown by the getRankByName method. This should be EntityNotFoundException instead
Updated by Andreas Müller over 2 years ago
- Assignee changed from Andreas Müller to Andreas Kohlbecker
This seems to be a misunderstanding. I did not mean that the issue is "fixed" but according to note-1 it is not an issue. All getRankByName methods do have a javadoc saying that they are preliminary implementation and that they serve only supporting the parser.
I did now additionally change the name of the method to getRankByLatinName() to make the semantics of these methods clearer as they actually only check for Latin labels. Checking for English labels is done by the method getRankByEnglishName or so.
Updated by Andreas Kohlbecker over 2 years ago
I agree with this solution. Please decide how to further proceed with this ticket.
Shall we delete the test method?
Updated by Andreas Müller over 2 years ago
- Status changed from Feedback to Closed
- % Done changed from 0 to 100
I removed the test and close the ticket.
Updated by Andreas Müller about 2 years ago
- Target version changed from Release 5.45 to Release 5.29