[Zope-CMF] Re: [dev] delete members: related changes and updated
proposal
Dieter Maurer
dieter at handshake.de
Tue Sep 16 14:26:01 EDT 2003
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
More information about the Zope-CMF
mailing list