search

collective / collective.nitf

A Dexterity-based content type inspired on the News Industry Text Format specification
8 stars 3 forks source link

Remove indexes #251

Closed wesleybl closed 2 years ago

wesleybl commented 3 years ago

Today indexes are created in Nitf:

https://github.com/collective/collective.nitf/blob/3bfff5b606f260db385abed14e0561da6516f4c4/src/collective/nitf/setuphandlers.py#L48-L56

to be used in collections:

https://github.com/collective/collective.nitf/blob/master/src/collective/nitf/profiles/default/registry.xml#L5-L77

As we know, the fewer indexes in the catalog the better. So my question is: Is it worth it to inflate the catalog with these indexes? Is someone really using it? Maybe we are cataloging things that are never used.

My suggestion is to remove these indexes. If anyone has a case for them, we can back.

What do you think?

@hvelarde @ericof @rodfersou @cleberjsantos

hvelarde commented 3 years ago

IMO, yes they are needed as they add a lot of functionality to the collections.

wesleybl commented 2 years ago

Ok let's keep the indexes then.