Bug #2633
Deleting a cloned result also deletes the original result
| Status: | New | Start date: | 10/30/2019 | ||
|---|---|---|---|---|---|
| Priority: | High | Due date: | |||
| Assignee: | - | % Done: | 80% |
||
| Category: | Commands | Spent time: | - | ||
| Target version: | 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
Associated revisions
fix TXMResult delete command refs #2633
Fixes: Deleting a cloned result also deletes the original result (refs #2633)
History
#1 Updated by Sebastien Jacquot 5 months ago
- Subject changed from Deleting a cloned specificities also delete the original specificities to Deleting a cloned specificities also deletes the original specificities
#2 Updated by Sebastien Jacquot 5 months ago
- Subject changed from Deleting a cloned specificities also deletes the original specificities to Deleting a cloned result also deletes the original result
- Description updated (diff)
#3 Updated by Matthieu Decorde 3 months ago
- Subject changed from Deleting a cloned result also deletes the original result to Deleting a cloned specificties or AFC result computed from partition
- Description updated (diff)
- Category set to Commands
#4 Updated by Matthieu Decorde 3 months ago
- % Done changed from 0 to 80
#5 Updated by Sebastien Jacquot 2 months ago
- Target version changed from TXM 0.8.2 to TXM 0.8.1
#6 Updated by Sebastien Jacquot 2 months ago
- Subject changed from Deleting a cloned specificties or AFC result computed from partition to Deleting a cloned result also deletes the original result
#7 Updated by Sebastien Jacquot 2 months ago
- % Done changed from 80 to 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 Updated by Sebastien Jacquot about 1 month ago
- % Done changed from 0 to 80