[CMF-checkins] CVS: CMF/CMFDefault - MembershipTool.py:1.43

[Florent Guillaume]

Casey Duncan wrote:
>>You're right about the aq_base, but I'll code it differently:
>>   if hasattr(aq_base(members), id):
>>     folder = getattr(members, id)
>>     ...
> 
> Except that hasattr swallows conflict errors... 8^/

It does ? Could you expand on this, that's news to me...

Florent

>>But if you still think it's problematic I'll remove it and
>>monkey-patch it from CPS.
> 
> Nah, it seems ok. Just muddles up the code a bit. I'm ok with it.

-- 
Florent Guillaume, Nuxeo (Paris, France)
+33 1 40 33 79 87  http://nuxeo.com  mailto:fg at nuxeo.com