[Zope-CMF] IndexableObjectWrapper

Tres Seaver tseaver at palladion.com
Wed Mar 18 08:12:19 EDT 2009


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

Miles wrote:
> 
> Hi
> 
> <snip>
> 
>  >> Can I suggest the following logic:
>  >>
>  >>   1. if the object already implements the IIndexableObject marker
>  >> interface, no wrapping is required;
>  >>
>  >>   2. otherwise, adapt to IIndexableObject to do the wrapping;
>  >>
>  >>   3. if no adapter is registered, fall back to the existing
>  >> IndexableObjectWrapper class for BBB.
>  >
>  > That sounds like what I had in mind, but not for BBB.  I think of the
>  > adapter scheme as a way to choose a non-default wrapper, rather than a
>  > quasi-mandatory replacement for it.
> 
> Ok, well this logic is checked in now on the branch, and tests adjusted 
> accordingly.  Without any warnings.

Thanks, looks good.


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

iD8DBQFJwOUj+gerLs4ltQ4RAolpAKC0o4oox/IYKe4curmEgKFiLnRjqQCg2kd2
Soip97b3h35ItbJhdjKjxNA=
=+baG
-----END PGP SIGNATURE-----



More information about the Zope-CMF mailing list