Bug #2633
Deleting a cloned result also deletes the original result
Statut: | Closed | Début: | 30/10/2019 | |
---|---|---|---|---|
Priorité: | High | Echéance: | ||
Assigné à: | - | % réalisé: | 100% |
|
Catégorie: | Commands | Temps passé: | - | |
Version cible: | TXM 0.8.1 |
Description
When a Result is created with hidden parents, the delete command deletes the parent.
But if the result is cloned, both results are deleted
Solution¶
Delete parent only if :- hidden
- AND has no more child
Révisions associées
fix TXMResult delete command refs #2633
Fixes: Deleting a cloned result also deletes the original result (refs #2633)
Historique
#1 Mis à jour par Sebastien Jacquot il y a presque 6 ans
- Sujet changé de Deleting a cloned specificities also delete the original specificities à Deleting a cloned specificities also deletes the original specificities
#2 Mis à jour par Sebastien Jacquot il y a presque 6 ans
- Sujet changé de Deleting a cloned specificities also deletes the original specificities à Deleting a cloned result also deletes the original result
- Description mis à jour (diff)
#3 Mis à jour par Matthieu Decorde il y a presque 6 ans
- Sujet changé de Deleting a cloned result also deletes the original result à Deleting a cloned specificties or AFC result computed from partition
- Description mis à jour (diff)
- Catégorie mis à Commands
#4 Mis à jour par Matthieu Decorde il y a presque 6 ans
- % réalisé changé de 0 à 80
#5 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
- Version cible changé de TXM 0.8.2 à TXM 0.8.1
#6 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
- Sujet changé de Deleting a cloned specificties or AFC result computed from partition à Deleting a cloned result also deletes the original result
#7 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
- % réalisé changé de 80 à 0
The fix doesn't work...
Update: my bad... I reintroduced the bug by commenting:
parent.getChildren().size() == 1 // AND if there is no sibling
in DeleteObject, line 127, several days ago.
I think this test is not sufficient to manage all the case of deletion.
I'll do some tests.
#8 Mis à jour par Sebastien Jacquot il y a plus de 5 ans
- % réalisé changé de 0 à 80
#9 Mis à jour par Sebastien Jacquot il y a plus d'un an
- % réalisé changé de 80 à 100
#10 Mis à jour par Sebastien Jacquot il y a plus d'un an
- Statut changé de New à Closed