feature request #3836
closedImplement modifiers and modifying text for Description Elements in TaxEditor
Added by Andreas Müller over 10 years ago. Updated over 4 years ago.
100%
Description
but not for TextData as we expect that TextData will handle modifiers with its text.
see also #3746
Only the recommended modifiers should be shown for the given feature (Feature.recommendedModifierEnumeration)
Related issues
Updated by Andreas Müller almost 6 years ago
- Description updated (diff)
- Assignee changed from Cherian Mathew to Katja Luther
- Target version changed from TaxEditor Next Major Release to Unassigned CDM tickets
Updated by Andreas Müller almost 6 years ago
- Related to feature request #3746: Implement time period scope for description elements in TaxEditor added
Updated by Andreas Müller almost 6 years ago
- Related to feature request #3747: Check if time period is handled correctly for description elements in data portals added
Updated by Andreas Kohlbecker about 5 years ago
- Subject changed from Implement modifiers and modifying text for Description Elements to Implement modifiers and modifying text for Description Elements in TaxEditor
Updated by Andreas Kohlbecker about 5 years ago
- Related to feature request #8144: Check if modifiers and modifyingText is handled correctly for description elements in data portals added
Updated by Andreas Müller about 5 years ago
- Target version changed from Unassigned CDM tickets to Release 5.6
As we have decided to check for old unimplemented model issues in this version I add it to current milestone. But we may discuss if this makes sense.
Updated by Katja Luther about 5 years ago
- Priority changed from New to Priority14
Updated by Katja Luther about 5 years ago
- Status changed from New to Resolved
- Assignee changed from Katja Luther to Andreas Müller
This is implemented now, please review
Updated by Andreas Müller about 5 years ago
- Assignee changed from Andreas Müller to Patrick Plitzner
please do first review on nightly
Updated by Patrick Plitzner about 5 years ago
- Assignee changed from Patrick Plitzner to Andreas Müller
Tested on nightly:
- Created a specimen description element and set modifier and Modifying text
- Closed and re-opened, re-connected and everything was stored correctly
- TextData like e.g. citation do not have modifiers
Updated by Andreas Müller about 5 years ago
- Description updated (diff)
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Only the recommended modifiers should be shown for the given feature (Feature.recommendedModifierEnumeration) - sorry this requirement was missing in the initial description.
Also for some reason when editing a feature in term editor currently it is not possible to edit the recommended modifiers. Patrick, is this maybe a regression from additivity?
Updated by Andreas Müller about 5 years ago
We may also discuss if the modifiers should only be shown if the recommended modifier enumeration is not empty.
Also we may want to make modifers configurable (show only if switched on) as some projects may consider them as not necessary.
But these might be issues of a new ticket.
Updated by Katja Luther about 5 years ago
Andreas Müller wrote:
Only the recommended modifiers should be shown for the given feature (Feature.recommendedModifierEnumeration) - sorry this requirement was missing in the initial description.
Also for some reason when editing a feature in term editor currently it is not possible to edit the recommended modifiers. Patrick, is this maybe a regression from additivity?
This is only possible for categorical data and quantitative data and for these the drop down is filled with the recommended modifiers, if no recommended modifiers exist all modifiers from Vocabulary Modifier are loaded.
Updated by Katja Luther about 5 years ago
Andreas Müller wrote:
We may also discuss if the modifiers should only be shown if the recommended modifier enumeration is not empty.
Also we may want to make modifers configurable (show only if switched on) as some projects may consider them as not necessary.
But these might be issues of a new ticket.
created new ticket #8226
Updated by Katja Luther about 5 years ago
- Related to feature request #8226: make modifiers for description elements configurable added
Updated by Katja Luther about 5 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
for description elements only the recommended modifiers are shown. configurability is moved to a new ticket.
Updated by Andreas Müller about 5 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Katja Luther wrote:
Andreas Müller wrote:
Only the recommended modifiers should be shown for the given feature (Feature.recommendedModifierEnumeration) - sorry this requirement was missing in the initial description.
Also for some reason when editing a feature in term editor currently it is not possible to edit the recommended modifiers. Patrick, is this maybe a regression from additivity?
This is only possible for categorical data and quantitative data and for these the drop down is filled with the recommended modifiers, if no recommended modifiers exist all modifiers from Vocabulary Modifier are loaded.
Hmm, this needs to be discussed. CategoricalData and QuantitativeData have extra modifiers in StateData and StatisticalMeasurmentValue. Maybe therefore only they allow to define the recommended enumerations currently. However, for the DescriptionElements either all (except text data) or none should evaluate the enumeration.
But definetely IF we evaluate the enumeration an empty list should be handled as NO modifier available and not as All modifiers. This is because there are too many different modifier types which may not match at all.
Not sure how we handle this for this release now. We need to discuss if we need 2 different modifier enumerations for catergorical+quantitative data or if we can use the same enumeration for both places.
As we havn't really time to discuss this until release I suggeset to hide the field until we have decided. Can you open a ticket to decide how to handle the modifier enumerationsß
Updated by Katja Luther about 5 years ago
now the modifier section is shown only if the feature has recommended modifier voc, so it is hided for all features it is not possible to add recommended modifier vocabularies.
Updated by Katja Luther about 5 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller about 5 years ago
- Related to feature request #8236: [DISCUSS] Should recommended modifiers also hold for DescriptionElements added
Updated by Andreas Müller about 5 years ago
The discussion on recommended modifiers I moved to #8236.
Updated by Andreas Müller about 5 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Katja Luther wrote:
now the modifier section is shown only if the feature has recommended modifier voc, so it is hided for all features it is not possible to add recommended modifier vocabularies.
Only the structured modifiers may need to be hidden. The freetext modifier (modifying text) is vocabulary independent and therefore can always be shown (except for configuration via preferences that it should not be shown for UI simplification)
Updated by Katja Luther about 5 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Katja Luther to Andreas Müller
Andreas Müller wrote:
Katja Luther wrote:
now the modifier section is shown only if the feature has recommended modifier voc, so it is hided for all features it is not possible to add recommended modifier vocabularies.
Only the structured modifiers may need to be hidden. The freetext modifier (modifying text) is vocabulary independent and therefore can always be shown (except for configuration via preferences that it should not be shown for UI simplification)
this is implemented now.
Updated by Andreas Müller over 4 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 70 to 100
I think I tested this in the context of #8226 and I think it worked.