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

[J Cameron Cooper]

> 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

While the idea that this method should delete the folders by default is 
fine and expected, it seems strange to have flags that default to true. 
If it's a "hidden" option, one should turn it on, rather than off, when 
over-riding. How about:

def deleteMembers(member_ids, keep_memberareas=0,
                   keep_localroles=0):

          --jcc

-- 
"My point and period will be throughly wrought,
Or well or ill, as this day's battle's fought."