Hi!
Miles wrote:
> Thanks for the clarification. The bits that confused me were:
>
> class IndexableObjectSpecification(ObjectSpecificationDescriptor)
> ...
>
> class IndexableObjectWrapper(object):
>
> implements(IIndexableObjectWrapper)
> __providedBy__ = IndexableObjectSpecification()
>
> What does this code actually achieve (I get the implements bit, obviously)?!
This makes the wrapper transparent, allowing the index to look up
adapters for the interfaces of the object. TextIndexNG3 works that way.
>> Whether that is good or bad or should be changed is a different issue.
>
> I will write up a short proposal.
AFAICS wrapping the object before looking up adapters is unnecessary.
The catalog should do the lookup directly and the existing features
provided by IndexableObjectWrapper should be reimplemented as adapters.
Cheers,
Yuppie
_______________________________________________
Zope-CMF maillist - Zope-CMF@lists.zope.org
http://mail.zope.org/mailman/listinfo/zope-cmf
See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests
