Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

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)

  1. import an image
  2. put it in a dataset
  3. project the image
  4. unlink the projection from the dataset
  5. delete the dataset
  6. 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

Is this a candidate for 5.1.2?

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.

Last edited 4 years ago by mtbcarroll (previous) (diff)

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
Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.79035 sec.)

We're Hiring!