feature request #6581closed
Make all microreferenced references OriginalSources
Currently we have OriginalSource usually used for collections of sources while single references such as nomenclaturalReference or references for relations (or even secundum reference) use 2 fields for defining the reference, the references and the microreference fields. This is generally problematic e.g. when formatting the nomenclatural citation. Also it reduces the possibility to add further information that is available in OriginalSource such as name used in source (could be used for original spelling), source information (currently "name in source"), or further features such as URL(s) that do link to an online version of the source together with a comment (see #6588).
We may want to handle all these references as OriginalSources. Also this eases the usage of the Reference class as it is not linked from so many places anymore (but therefore OriginalSource is linked more). Also we can easier implement common formatters for sources.
Also this could improve the handling for protologues and similar "proves" for nomenclatural acts or references in general.
Possible candidates are
- nomenclatural reference
- RelationshipBase reference (TaxonRelationship, NameRelationship, HybridRelationship)
- TypeDesignation (rename to designationSource => #9332)
- Classification => needs further discussion as classification is SourcedEntityBase #9211
- Secundum reference => #9327
Note: We already have the class ReferencedEntityBase which inherits from AnnotatableEntity and is the base class for RelationshipBase and TypeDesignation. This can be kept and the 3 parameters citation, microCitation and originalNameString can be replaced by according OriginalSource fields.
do we want to keep the direct methods, e.g. getNomenclaturalReference() in the main classes=> for now we keep it, may be removed later how to delete a source if it became completely empty how to decide if a source is completely empty (is an attached annotation or import source enough to make it not empty, or is it required that a "citation" exist. What about external links as only information=> isEmpty has been implemented and works on completely empty sources where to delete it (in model or only during persistence)=> #9069
should we support one common interface which makes it easier to program against but methods are difficult to name, getSource() or getOriginalSource() might not be an expected label for getNomenclaturalSource() in TaxonName or TypeDesignationBase. Maybe we use 2 interfaces, one for nomenclatural sources.=> we do have a common interface where appropriate now What OriginalSourceType to use: NomenclaturalReference could be a very strict source type for names and type designations, OriginalSource might be a more general one, maybe NomenclaturalReference might become a subtype; how is the relationship to PrimaryTaxonomicSource=> partly handled in #9330 IMPORTANT: should we store this source together with other sources if available. This is the case for TaxonName.nomenclaturalReference and TaxonBase.secundum where the main classes are identifiable and already have sources. There might be performance issues and lazy loading issues if the specific source is stored together with the set of OrginalSources.=> we decided not to do so Should we have a strict type checking for the OriginalSourceType?=> we do not handle source type for these sources in TaxEditor yet, maybe this becomes more interesting if we have something like SecondaryTaxonomicSource Should we allow to always return a placeholder empty source in case no source exists, this reduces NPEs but the semantic must be strictly followed that this placeholder may not be attached to the parent and therefore changes will have no effect; we may have a singleton for this to improve performance and to throw exceptions when any changes are done on this placeholder=> we do not use such an placeholder but we offer a get method with parameter createIfNotExists Rename DescriptionElementSource to NamedSource as the only reason for distinguishing it from IdentifiableSource is now the additional attribute; the back link to the sourced entity does not exist anymore=> #9331
Updated by Andreas Müller over 6 years ago
AK: > Viel besser sollten wir das im ValidationFramework als Rule definieren.
das Ticket https://dev.e-taxonomy.eu/redmine/issues/6581 freut mich wirklich sehr, denn die Nomenklatorischen Referenzen sind auch in Portal sehr lästig, da man mit citation und citationDetail immer noch einen extra Request an den Server schicken muss um den kompletten Citation String zu bekommen.