[Freeipa-devel] [PATCH] user and group deletion

[Simo Sorce]

On Fri, 2007-10-26 at 12:14 -0400, Rob Crittenden wrote:
> Kevin McCarthy wrote:
> > This adds user and group deletion to the gui.
> > 
> > It sidesteps the issues of:
> > 1) What if the user is a manager or secretary of another user
> > 2) What if the user is a member of a group
> > 3) What if the group is a member of a group.
> > 
> > I'm assuming these kinds of things will be solved with a server-side
> > plugin.
> > 
> > The patch also fixes a bug with add user brought on by the
> > manager/secretary patch.  Apparently hidden fields don't have a "" value
> > by default and so were passing none if not set.
> 
> OK, so we never came to a conclusion with this.
> 
> 1. Are we going to allow completely removing user entries (delete versus 
> inactivate)? The CLI currently only inactivates.

I think we should allow deletion too.

> 2. Are we going to allow completely removing group entries? CLI 
> currently deletes groups. There is no group inactivation.

I don't see how we could easily do group inactivation, deletion is ok
imo.

> So we need to reconcile these.
> 
> And this patch also contains an important fix to the GUI (add user is 
> currently broken).

I think you can push it.

Simo.