yuppie <y.2015@...> writes:
> yuppie wrote:
> > There is no test that shows what was broken and fixed. AFAICS there is
> > nothing wrong with using None for uninitialized objects. We just have a
> > new requirement in Products.ZCatalog 3: Attributes with None values can
> > no longer be indexed.
> > 
> > Wouldn't it be better to adjust the portal_type just for indexing in the
> > IndexableObjectWrapper?
> 
> Fixed the issue this way:
> 
> - reverted r130422
> 
> - adjusted the IndexableObjectWrapper
> 
> I guess other indexed attributes can also contain None. Not sure if
> converting them to an empty string is always the best solution.

Thanks for cleaning up after me. I took the easy way out when trying to make
CMF trunk and Zope trunk work together.

Adjusting the indexing behavior is the better approach.

Hanno

_______________________________________________
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests

Reply via email to