[z3-five] Re: __fallback_traverse__ exception

[Tarek Ziadé]

Philipp von Weitershausen wrote:

>Tarek Ziadé wrote:
>  
>
>>If i make OFS.Folder five:traversable, some CPS code that is based on
>>the fact that the CMF raises
>>a KeyError when an id doesn't exists fails, because Five raises an
>>AttributeError.
>>
>>To have the same behavior, __fallback_traverse__() should raise a
>>KeyError as well.
>>
>>Does anyone sees a problem if we change this behavior ?
>>    
>>
>
>I don't, but I remember that the behaviour of the whole Five/Zope 2
>traversal system is as stable as a house of cards :). So, there might be
>implications for other systems than the CMF.
>
>That's all *I* can say...
>
>Philipp
>  
>
we've talked about it with Lennart off list and it seems that KeyError,
NotFoundError, and AttributeError
can all happen in Zope 2 when an id is  not found in the traversal,
depending on the container,
so i guess this behavior is right.

Tarek