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

[Tres Seaver]

On Mon, 2003-09-15 at 15:49, J Cameron Cooper wrote:

> 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):

-1.  Such a refactoring is a change to the contract, and will break any
callers which were not using the defaulted parameter.  OTOH, if this is
a new API, or a private method, then feel free. :)

Tres.
-- 
===============================================================
Tres Seaver                                tseaver at zope.com
Zope Corporation      "Zope Dealers"       http://www.zope.com