Skip to content

First step towards a simplification of the indexer's code

David Douard requested to merge generated-differential-D990-source into generated-differential-D990-target

In this firs step, we attempt to simplify a bit the initialization process of all BaseIndexer derivative classes.

  • the configuration is managed entirely within the init method, and it can be adapted: any argument of the parse_config_file method can be given as constructor's parameter,

  • tools/tool attributes are managed within BaseIndexer.prepare() method. It uses a USE_TOOLS class attribute (bool) to check whether the 'tools' config paramter must be validated in the check() method.

  • config mess dedicated to ContentMetadataIndexer has been removed; ensure the proper 'tools' field is present in the 'config' parameter of the ContentMetadataIndexer's constructor (see metadat.py L216-218).

Migrated from D990 (view on Phabricator)

Merge request reports