[Zope-CMF] DynamicType.portal_type

[Hanno Schlichting]

yuppie <y.2015 at ...> 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