[Zope-CMF] Re: CVS: CMF/CMFSetup - actions.py:1.18

yuppie y.2005- at wcm-solutions.de
Tue Feb 22 06:19:56 EST 2005


Florent Guillaume wrote:
>>=== CMF/CMFSetup/actions.py 1.17 => 1.18 ===
>>+            if p_info['actions']:
>>+                for a_info in p_info['actions']:
> [...]
> 
> Nitpick: I prefer to write such code in a way that indentation is minimal:
> 
>             for a_info in p_info['actions'] or ():

Looking again at this code, I believe p_info['actions'] is always a 
sequence. So just removing the 'if' statement should suffice.

Feel free to change that code if you like.

Cheers, Yuppie



More information about the Zope-CMF mailing list