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"

User Story #547 (new)

Opened 14 years ago

Last modified 5 years ago

Add deleteGroup and deleteUser to IAdmin

Reported by: jamoore Owned by: jamoore
Priority: critical Milestone: Unscheduled
Component: API Keywords: security
Cc: jburel, mtbcarroll, matthew.couchman@…, sethur2@… Story Points: n.a.
Sprint: n.a. Importance: n.a.
Total Remaining Time: 3.0d Estimated Remaining Time: n.a.

Description

Extracted from #545.

Deleting groups and users is obviously tricky. Either it must be done recursively, or can be done simply but will only succeed if there are no FK links to the entity.

Most likely the simple delete will suffice. If so, then IUpate.deleteObject() could be used if BasicACLVoter is updated to allow deleting Experimenter and ExperimenterGroup (test before attempting).

Change History (9)

comment:1 Changed 13 years ago by jmoore

  • Milestone changed from GatherReqs to 3.0-Beta4

comment:2 Changed 11 years ago by jmoore

  • Milestone changed from OMERO-Beta4 to OMERO-Beta4.1

Will have to wait for the next round of delete improvements.

comment:3 Changed 11 years ago by jmoore

See #1329

comment:4 Changed 8 years ago by jburel

  • Cc jburel added

if we let users create their groups, such functionality needs to be considered using the delete service.

comment:5 Changed 8 years ago by jmoore

Definitely, but I don't see us implementing and testing user/group deletion for 4.4.

comment:6 Changed 8 years ago by jmoore

  • Priority changed from minor to critical

comment:7 Changed 6 years ago by mtbcarroll

  • Cc mtbcarroll added

comment:8 Changed 6 years ago by mtbcarroll

A naïve look at this with my current experimental branch goes after the event log as if it were assigned to the Ministry of Truth, the most obvious initial problem being that parent directories in the repository may have an OriginalFile instance whose creationEvent has an experimenterGroup of the target group so it even goes after that instance and runs into PSQLException complaints of non-empty directories. Further work on this probably requires some initial scoping with such issues in mind: how to circumscribe this group deletion and how to adjust outstanding not-null properties that reference the target group across that border.

comment:9 Changed 5 years ago by mtbcarroll

  • Cc matthew.couchman@… sethur2@… added
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.206144 sec.)

We're Hiring!