[z3-five] Re: making ObjectManager and Item Five-traversable by default

[Martijn Faassen]

Philipp von Weitershausen wrote:
> Martijn Faassen wrote:
> 
>> Sidnei da Silva wrote:
>> [not making ObjectManager/Item five-traversable by default]
>>
>>> I'm -1 for it until we find a solution for the following:
>>>
>>> - If two packages include the directive, even using includeOverrides,
>>>   a configuration conflict will happen.
>>>
>>> If there's a solution for that, I would be +1 on disabled by default.
>>
>> I thought Philipp had changed matters so that this wouldn't happen 
>> anymore. Perhaps I am mistaken about what was tackled, Philipp?
>  
> Before Sidnei renamed five:viewable to five:traversable and mangled with 
> it, it was indeed theoretically possible to call <five:viewable> more 
> than once. Theoretically because I forgot to get rid of the ZCML 
> descriminator, so you'd get a conflict indeed :).
> 
> In case Sidnei left five:traversable and five:defaultViewable in a state 
> where they can be called more than once w/o harm, we should lift the 
> descriminators.

I guess it's safe to do this for any ZCML directive that just takes a 
'class' and then gets monkey-patched by Five, such as five:traversable, 
five:defaultViewable and five:sendsEvents. I've added an item to the 
TODO list.

Perhaps we can event devise some tests for this by loading ZCML from the 
tests manually or something..

Regards,

Martijn