bug #9491
closedOrder of terms in not order relevant trees
100%
Description
The order of not order relevant trees in term tree editors should be alphabetic.
Updated by Katja Luther almost 3 years ago
- Status changed from New to Resolved
- Assignee changed from Katja Luther to Andreas Müller
Changed to client side ordering because the server does not know about the preference "Show Id in Vocabulary" for the term tree editor.
Please review
Updated by Andreas Müller almost 3 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
There was still an NPE when editing the title of a tree. I fixed this with a501f1e0 .
It generally works. Only I do not understand why the comparator implements the label provider again. Shouldn't we reuse the label provider code to be on the save side?
Updated by Katja Luther over 2 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
Andreas Müller wrote:
There was still an NPE when editing the title of a tree. I fixed this with a501f1e0 .
It generally works. Only I do not understand why the comparator implements the label provider again. Shouldn't we reuse the label provider code to be on the save side?
The comparator is adapted and uses the labelprovider, please have a short look whether everything is ok, then we can close the ticket.
Updated by Andreas Müller over 2 years ago
Can we delete the commented text in CreateDefinedTermOperation?
Updated by Andreas Müller over 2 years ago
Can we rename FeatureTreeViewerComparator to TermTreeViewerComparator? "FeatureTree" is deprecated. Maybe we can run search and replace for "FeatureTree" for the complete code (but take care for possible exceptions).
Updated by Andreas Müller over 2 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 40 to 90
Functionality and new code looks looks ok beside this and ticket can be closed.