[Zope-CMF] Re: [dev] delete members: small proposal

Chris Withers chrisw at nipltd.com
Thu Sep 11 16:10:39 EDT 2003


Yuppie wrote:
> I updated my proposal. deleteMembers() returns a tuple listing 
> member_ids of deleted members. Your code could test if the return value 
> is empty. But maybe deleteMembers() should raise an error instead?

Raising errors is good, but please make sure they're catchable in skin scripts!
(think security declarations and module declarations for the exception class so 
they can be imported...)

>> Well, CMFCore currently works with most userfolders.
>> I'd like to see it stay that way... provided deleteMember uses the 
>> standard acl_users interfaces, there shouldn't be a problem...
> 
> Judging from that interface, userfolders should either have a 
> userFolderDelUsers() method or raise 'NotImplemented'. Again my current 
> proposal is to return an empty tuple in that case. And again it would be 
> possible to raise an error instead.

error please, that's what they're there for ;-)

cheers,

Chris




More information about the Zope-CMF mailing list