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

[Dieter Maurer]

Yuppie wrote at 2003-9-16 10:45 +0200:
 > ...
 > > 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):
 > 
 > Strange! There are some methods in CMF using true as default and it 
 > never seemed strange to me:

Very fine!

It is *MUCH* more important to choose good argument names
than to ensure that the default argument values are true...


Dieter