[Zope-CMF] IndexableObjectWrapper

[Tres Seaver]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

yuppie wrote:

>>   - The TraversingEventZCMLLayer now also loads implements.zcml, as the 
>> correct behaviour of both the catalog and the adapter class is required 
>> for the folder tests to pass.
>>
>>   - The CatalogTool tests set up the adapter at the moment, as a lot of 
>> the catalog tests require the adapter to work properly.  This is done in 
>> the _makeContent method as it applied to most tests that used the dummy 
>> content.  However, I think it belongs somewhere else, but I wasn't sure 
>> whether that place was a layer, a setup method or somewhere else.  Any 
>> suggestions?
> 
> I agree it belongs somewhere else. Maybe a registerWrapper method. But 
> 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.


Tres.
- --
===================================================================
Tres Seaver          +1 540-429-0999          tseaver at palladion.com
Palladion Software   "Excellence by Design"    http://palladion.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJv+To+gerLs4ltQ4RAvt/AJ9D75eOsZh7FINca9MJsnTU26998gCghu8C
TaiqRzwLe2WDoHIqaD7qGhs=
=kd1W
-----END PGP SIGNATURE-----