[Zope-CMF] [dev] delete members: related changes and updated proposal

[Yuppie]

Hi!


Based on the feedback and a closer look at the issue, I updated my 
proposal and included some related changes:


related changes:
================

1.) createMemberarea

Camel case of member area is MemberArea, not Memberarea. I'd rather like 
to change createMemberarea to createMemberArea (and provide an alias for 
backwards compatibility) than adding a deleteMemberarea method.

2.) getMembersFolder

Currently getMembersFolder exists only in CMFDefault. I'd like to add a 
hardcoded getMembersFolder method (always returning 'Members') to 
CMFCore for use in the new deleteMemberArea method.

3.) deleteLocalRoles

For implementing deleteMembers, two issues have to be resolved:
a) deleteLocalRoles is protected by roles, not by permissions. I propose 
to protected it by ManageUsers of the object.
b) deleteLocalRoles should have a 'recursive' argument.


updated proposal:
=================

portal_membership:

def deleteMembers(member_ids, delete_memberareas=1,
                    delete_localroles=1):
   """ Delete members specified by member_ids.

   Delete members in acl_users and member data in portal_memberdata.
   If delete_memberareas is true, delete members' home folders including
   all content items. If delete_localroles is true, recursively delete
   members' local roles, starting from the portal root.

   Permission -- Manage users

   Returns -- Tuple listing member_ids of deleted members
   """

def deleteMemberArea(member_id):
   """ Delete member area of member specified by member_id.

   Permission -- Manage users
   """

portal_memberdata:

def deleteMemberData(member_id):
   """ Delete member data of specified member.

   Permission -- Python only
   """


Comments are welcome! If there are no objections, I'll check this in on 
a branch within the next days.


Cheers,
	Yuppie