feature request #8257
closedRemove factory methods for term node creation
100%
Description
Currently it is possible to create a FeatureNode
on its own, e.g. with a given term
public static <T extends DefinedTermBase<T>> FeatureNode<T> NewInstance(T term){
FeatureNode<T> result = new FeatureNode<>(term.getTermType());
result.setTerm(term);
return result;
}
It should only be possible to create a term node based on its parent or its term tree.
AM: Aus einem ähnlichen Grund hatte ich bei TaxonNodes glaube ich mal grundsätzlich keine alleinstehende Instanzerzeugung zugelassen, weil es gefährlich ist, einen Knoten ohne seinen Baum zu erzeugen.
Ich denke, in diesem Fall sollten wir das auch anpassen. Erzeugen von Knoten sollte nur über Parent oder den Baum möglich sein mittels add-Methoden oder ähnlichem. In diesem Fall ist dann der termType des Baums immer bekannt und wird immer korrekt gesetzt.
Updated by Patrick Plitzner about 4 years ago
- Status changed from New to In Progress
Updated by Patrick Plitzner about 4 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 50
Applied in changeset cdmlib|48e5bbde635e7d2fd347cf5c2eef0d103c3699db.
Updated by Patrick Plitzner about 4 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Patrick Plitzner to Andreas Müller
@Andreas M.: Can you please have look at the last remaining factory method which is used in one last spot:
@Deprecated
public static FeatureNode<Feature> NewInstance(){
return new FeatureNode<>(TermType.Feature);
}
Updated by Andreas Müller almost 4 years ago
- Status changed from Feedback to Resolved
Applied in changeset cdmlib|179a846773ab7eec76e786dfafe5a84cdbdf7103.
Updated by Andreas Müller almost 4 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Patrick Plitzner
- % Done changed from 50 to 80
should be fixed now and can be closed I guess
Updated by Patrick Plitzner almost 4 years ago
- Status changed from Feedback to Closed
- % Done changed from 80 to 100