bug #6952
open
Deleting terms does not get reflected in the UI and in the DB
Added by Patrick Plitzner over 6 years ago.
Updated about 5 years ago.
Description
If you delete a term in the term editor nothing happens in the UI.
After reconnecting and re-opening the term editor, the term is gone but it is still is the DB.
This seems to be related to #6951 because it works for terms that have the vocabulary as parent but not if they have another term as parent.
Another issue: If a term (with vocabulary as parent) is deleted, the term editor gets dirty but the term has actually been already deleted in the database making it impossible to revert your changes.
- Related to bug #6951: Drag and drop does not update term hierarchy in the UI added
- Description updated (diff)
After reconnecting and re-opening the term editor, the term is gone but it is still is the DB.
What happens in the DB is that the partOf relationship is deleted (partOf_id = NULL) and also the term is removed from the vocabulary (vocabulary_id = NULL), but the DefinedTermBase record itself is still in the DB. Therefore it does not appear in the Term Editor after reconnect.
- Status changed from New to Resolved
- Target version changed from Unassigned CDM tickets to Release 5.5
- % Done changed from 0 to 50
This issue is fixed with the implementation of DTOs in the term editor #7887
- Assignee changed from Katja Luther to Andreas Müller
This issue should be reviewed
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Patrick Plitzner
This is partly fixed. Deleting a single term works. The following issues do not yet work as expected:
- Deleting a vocabulary does not delete the terms within the vocabulary (do we need cascading.Delete for this?)
- Deleting a term which is not a leave but a node with children, this is not possible. A warning appears that the term is referenced (by it's children). Better handling here is to ask how the children should be handled (delete the whole subtree, attach children to parent, cancel). This is already correctly implemented in taxon navigator and should be standard for all tree editors. Please have a look there.
Andreas Müller wrote:
This is partly fixed. Deleting a single term works. The following issues do not yet work as expected:
- Deleting a vocabulary does not delete the terms within the vocabulary (do we need cascading.Delete for this?)
- Deleting a term which is not a leave but a node with children, this is not possible. A warning appears that the term is referenced (by it's children). Better handling here is to ask how the children should be handled (delete the whole subtree, attach children to parent, cancel). This is already correctly implemented in taxon navigator and should be standard for all tree editors. Please have a look there.
I did not change the logic of the delete operation when refactoring the term editor to use DTOs. So this must have worked the same way as before. There is also no specific delete method in the vocabulary service. So we have to decide if this should be done via Cascading or if we implement the recursive deletion in the service method.
For now, I just disallow the deletion of vocabularies that are not empty
- Assignee changed from Patrick Plitzner to Katja Luther
Katja, could you have a look on the delete issue as you have the most experience with delete handling.
- Status changed from Feedback to In Progress
- Priority changed from New to Highest
- Target version changed from Release 5.5 to Release 5.6
I will do that in the next release
- Target version changed from Release 5.6 to Reviewed Next Major Release
Also available in: Atom
PDF