Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
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 #10779 (closed)

Opened 11 years ago

Closed 11 years ago

Pass SOFT flag on delete of MIF-image

Reported by: jamoore Owned by: bpindelski
Priority: critical Milestone: 5.0.0-beta1
Component: Insight Version: n.a.
Keywords: fs Cc: fs@…, ux@…, wmoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: FS demo 4.1

Description

Once #10776 has been implemented, attempting to delete a single image from a MIF will result in an exception ("foreign key constraint"). If the SOFT flag is passed for the /Image/Fileset relationship, then the delete of the main image can proceed without affecting the rest of the MIF. It's unclear whether:

  • SOFT should always be passed;
  • MIFs should be detected and SOFT passed; or
  • An attempt made without SOFT with a dialog provided to the user explaining the situation (possibly with highlighting, see #10778) and a confirmation to use SOFT. This would depend on a clear explanation of the situation in the ERR object returned by graph deletes (#10776)

Either of the first 2 options would be an initial win.

Change History (3)

comment:1 Changed 11 years ago by wmoore

In discussion with Josh:

  • Always use 'SOFT' (clients don't have to know about this)
  • Clients need to handle return message, E.g. {"remaining-filesets":[596, 492]}. See #10806

comment:2 Changed 11 years ago by jamoore

Just to be clear, the clients (or at least the gateways) will need to be aware because SOFT will need to be passed for DELETE but not for CHGRP. (At least that's the fastest way to have this implemented. Perhaps we can hard code the difference server-side)

comment:3 Changed 11 years ago by jamoore

  • Resolution set to invalid
  • Status changed from new to closed

Server-side changes for #10776 have made this unnecessary. Closing.

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.66414 sec.)

We're Hiring!