[Zope3-dev] Bug in checkPermission

[Dmitry Vasiliev]

Jim Fulton wrote:
>> Dmitry Vasiliev wrote:
>>
>> If "checkPermission is guaranteed to return True if permission is 
>> CheckerPublic or None" why we just can't do:
>>
>> if permission is CheckerPublic or permission is None:
>>     return True
>>
>> in zope.security.management.checkPermission?
> 
> Sigh.
> 
> That's right.  I just broke this the other day.  I missed the last part 
> of the
> doc string and removed the code that checked for CheckerPublic or None.
> This didn't break any tests.
> 
> Wanna add this back with a test?

Added on trunk at revision 26753.

-- 
Dmitry Vasiliev (dima at hlabs.spb.ru)
     http://hlabs.spb.ru