feature request #8283
closedAllow adding media to terms in taxeditor
100%
Description
This could be done in the details view for example
Related issues
Updated by Patrick Plitzner almost 5 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 50
Applied in changeset taxeditor|d418413628bd797221e1c9101ec57af041f1dab5.
Updated by Patrick Plitzner almost 5 years ago
- Assignee changed from Patrick Plitzner to Andreas Müller
Updated by Andreas Müller almost 5 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Patrick Plitzner
The data may got lost the following way.
- Open term
- add media
- open referencing objects view (-> Details View is empty which is anyway not wanted)
- go back to term => media is not shown anymore
Updated by Andreas Müller almost 5 years ago
With the current implementation it is not possible to switch between advanced and simple view but sometimes it is essential to add >1 representations to a media.
In general it is a bit critical to have media together with other data in the details view, because the media part can become large and also because (until now) we do not yet have asynchronous loading for media which makes everything very slow once media are selected.
I can see that we can not use the media view with the current model as it requires a DescriptionBase, but maybe we can a switch in the media view which decides if it should show a media gallery or a simple media list depending on the selected parent.
In general for UX I think it is better if media are always handled the same way (in media view).
These thoughts are only for discussion, I am not sure what the best solution really is.
Updated by Andreas Müller almost 5 years ago
Katja, can you please cherry pick however. The above problems are minor issues and the media are not used much anyway but urgently needed for additivity.
Updated by Andreas Müller almost 5 years ago
Andreas Müller wrote:
The data may got lost the following way.
- Open term
- add media
- open referencing objects view (-> Details View is empty which is anyway not wanted)
- go back to term => media is not shown anymore
Is this maybe related to new implementation in #8316?
Updated by Katja Luther almost 5 years ago
- Target version changed from Release 5.8 to Release 5.7
Updated by Patrick Plitzner almost 5 years ago
- Status changed from Feedback to Resolved
Applied in changeset taxeditor|909326e9b74be422e80071325ae93e53102404a5.
Updated by Andreas Müller almost 5 years ago
- Target version changed from Release 5.7 to Release 5.8
PP just added some more code. Could you check if this also needs to be cherry picked.
Revert milestone as not sure if this is already fully implemented.
Updated by Andreas Müller almost 5 years ago
Or was this code only from the cherry picking? A bit confusing.
Updated by Katja Luther almost 5 years ago
Andreas Müller wrote:
Or was this code only from the cherry picking? A bit confusing.
Yes this seems to be the cherry-pick.
Updated by Katja Luther almost 5 years ago
- Status changed from Resolved to Feedback
Updated by Patrick Plitzner almost 5 years ago
Andreas Müller wrote:
PP just added some more code. Could you check if this also needs to be cherry picked.
Revert milestone as not sure if this is already fully implemented.
Interesting to see that interaction of cherry-picking and redmine. Not only is my name shown but also the 'fix' set the status back to resolved again.
Updated by Patrick Plitzner almost 5 years ago
- Assignee changed from Patrick Plitzner to Andreas Müller
- % Done changed from 50 to 80
Andreas Müller wrote:
The data may got lost the following way.
- Open term
- add media
- open referencing objects view (-> Details View is empty which is anyway not wanted)
- go back to term => media is not shown anymore
This does not happen anymore for me. Andreas, can you also try one more time and then we close this ticket and open a new one for the ideas mentioned in #8283#note-5
Updated by Katja Luther almost 5 years ago
- Status changed from Feedback to Resolved
- Assignee changed from Andreas Müller to Patrick Plitzner
this seems to work now. (the details view is still empty when changing to referencing objects view, but adding the media is fixed)
Updated by Patrick Plitzner almost 5 years ago
- Related to bug #8367: Advanced media view is not available for media not edited in media details view added
Updated by Patrick Plitzner almost 5 years ago
- Assignee changed from Patrick Plitzner to Andreas Müller
Updated by Patrick Plitzner almost 5 years ago
Added #8367 for remaining issues and discussion
All the rest should be implemented and functional now
Updated by Patrick Plitzner almost 5 years ago
- Related to bug #8129: Load images asynchronously in media details view added
Updated by Andreas Müller almost 5 years ago
- Status changed from Resolved to Feedback
Patrick Plitzner wrote:
Andreas Müller wrote:
The data may got lost the following way.
- Open term
- add media
- open referencing objects view (-> Details View is empty which is anyway not wanted)
- go back to term => media is not shown anymore
This does not happen anymore for me. Andreas, can you also try one more time and then we close this ticket and open a new one for the ideas mentioned in #8283#note-5
It still happens somehow but not so easy to reproduce.
I could reproduce this way:
On rem_conf_am open features.
- Got to Vocabulary "Untitled" and open term "test2".
- Open a media by simply clicking the "+" button
- Open "User Vocabulary"
- Click on first term (Citation Chromosome Numbers)
- Click back on test2 Repeat 4. and 5. a couple of times
After a while the media is not shown anymore.
Also there is another issue that sometimes when peforming step 5 the details view does not switch to "test2". Which is probably another issue and needs a new ticket.
Updated by Andreas Müller almost 5 years ago
- Assignee changed from Andreas Müller to Patrick Plitzner
Updated by Patrick Plitzner almost 5 years ago
- Assignee changed from Patrick Plitzner to Andreas Müller
Andreas Müller wrote:
Patrick Plitzner wrote:
Andreas Müller wrote:
The data may got lost the following way.
- Open term
- add media
- open referencing objects view (-> Details View is empty which is anyway not wanted)
- go back to term => media is not shown anymore
This does not happen anymore for me. Andreas, can you also try one more time and then we close this ticket and open a new one for the ideas mentioned in #8283#note-5
It still happens somehow but not so easy to reproduce.
I could reproduce this way:On rem_conf_am open features.
- Got to Vocabulary "Untitled" and open term "test2".
- Open a media by simply clicking the "+" button
- Open "User Vocabulary"
- Click on first term (Citation Chromosome Numbers)
- Click back on test2 Repeat 4. and 5. a couple of times
After a while the media is not shown anymore.
Also there is another issue that sometimes when peforming step 5 the details view does not switch to "test2". Which is probably another issue and needs a new ticket.
I tried these steps on rem_conf_am about 20 times and could not reproduce this behavior. This issue in general never happens when the new media is saved, right? I would say that this is a very rare corner case then and we may close this ticket.
Updated by Patrick Plitzner almost 5 years ago
Just when I wrote the last comment it happened. But still, it is very unlikely that this will happen to a user. We may open another ticket to investigate why this can happen at all. I suspect that it's either a problem with the widget management or the term cache.
I tried also with saving the term and then it did not occur anymore.
Updated by Andreas Müller almost 5 years ago
- Assignee changed from Andreas Müller to Patrick Plitzner
- % Done changed from 80 to 90
Patrick Plitzner wrote:
Just when I wrote the last comment it happened. But still, it is very unlikely that this will happen to a user. We may open another ticket to investigate why this can happen at all. I suspect that it's either a problem with the widget management or the term cache.
I tried also with saving the term and then it did not occur anymore.
Why do you think it is very unlikely to happen? It happened to me several times, though of course adding media to terms in general is not really core functionality - and save in general works so only returning to the term creates a problem.
But opening a ticket and close this one is ok. Anyway we should handle media via Media view soon so maybe the problem will not occurr this way anyway so it is not worth speding much time on it.
Updated by Patrick Plitzner almost 5 years ago
- Related to bug #8380: Unsaved media gets lost in details view added
Updated by Patrick Plitzner almost 5 years ago
- Status changed from Feedback to Closed
- % Done changed from 90 to 100
Created new ticket for remaining issues -> #8380