Task #12856 (closed)
Opened 4 years ago
Closed 4 years ago
Bug: delete dataset does not delete contained image that somebody projected
| Reported by: | mtbcarroll | Owned by: | mtbcarroll |
|---|---|---|---|
| Priority: | minor | Milestone: | 5.1.3 |
| Component: | ApplicationServer | Version: | 5.1.1 |
| Keywords: | n.a. | Cc: | ux@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description (last modified by mtbcarroll)
- import an image
- put it in a dataset
- project the image
- unlink the projection from the dataset
- delete the dataset
- be surprised that the imported image remains
This is probably due to,
<bean parent="graphPolicyRule" p:matches="!ILink[E]{r} = X:[E]{i}" p:changes="X:{r}"/>
which is one of the many alarmingly overgeneral rules that https://trello.com/c/Ryvi7Cra/ is intended to replace. (In deletion, for determining if an image is an orphan, there is no reason to consider projections of it.)
Change History (6)
comment:1 Changed 4 years ago by jamoore
comment:2 Changed 4 years ago by mtbcarroll
There is enough graph and chmod stuff for 5.1.2 that I am hesitant to yet add further: after we get that spreadsheet done (next week?), there is a lot of follow-up activity for me to make the rules far easier to change safely. This bug may be quite solvable with but a couple of hours' thought so if it's important then I could prioritize it accordingly.
In the meantime, I have created an integration test for it at https://github.com/openmicroscopy/openmicroscopy/pull/3780.
comment:3 Changed 4 years ago by mtbcarroll
- Description modified (diff)
- Milestone changed from 5.x to 5.1.2
Okay, let's shoot for 5.1.2.
comment:4 Changed 4 years ago by mtbcarroll
- Milestone changed from 5.1.2 to 5.1.3
This ticket may yet be fixed for 5.1.2 but I wouldn't want any major delete changes released without more time for testing.
comment:5 Changed 4 years ago by mtbcarroll
- Status changed from new to accepted
I already have this test passing under the work for https://trello.com/c/Ryvi7Cra/ so this ticket oughtn't again need pushing.
comment:6 Changed 4 years ago by mtbcarroll
- Resolution set to fixed
- Status changed from accepted to closed
Is this a candidate for 5.1.2?