Tres Seaver wrote:
> yuppie wrote:
>> can't we make the adapter lookup in catalog_object optional and wouldn't
>> that make test setups simpler?
> Agreed. I had expected that the catalog would do a queryAdapter, and
> default to the existing wrapper class if not found.
Well. I had expected it would default to the unwrapped object.
Restricted catalog queries rely on allowedRolesAndUsers, but AFAICS it
is not essential for CMFCore that the object is wrapped before cataloged.
The current code on the branch requires the new IIndexableObject
interface. I'm not sure if we should make it required. It makes things
more explicit, but creates an additional burden for simple use cases and
unit tests. It should be sufficient to assume that objects without
registered adapter are indexable unwrapped.
Zope-CMF maillist - Zope-CMF@lists.zope.org
See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests