Do not break when indexing multiple indexes

[icemac]

When trying to reindex multiple indexes while having a progress handler it was breaking before with:

...
  File ".../Products.ZCatalog-5.0.1-py3.7.egg/Products/ZCatalog/ZCatalog.py", line 420, in reindexIndex
    pghandler.init('reindexing %s' % name, len(paths))
TypeError: not all arguments converted during string formatting

[jugmac00]

I'd like to see a docstring added, as it is not entirely clear what name is, or should be. Obviously it can be a string or something else.

[d-maurer]

Jürgen Gmach wrote at 2019-12-5 23:24 -0800:

jugmac00 commented on this pull request. > @@ -410,14 +410,12 @@ def manage_clearIndex(self, ids=None, @security.protected(manage_zcatalog_entries) def reindexIndex(self, name, REQUEST, pghandler=None): I'd like to see a docstring added, as it is not entirely clear what name is, or should be. Obviously it can be a string or something else.

Note that a "docstring" has as side effect that the object becames "Web publishable" (i.e. can be directly called via the Web). Reindexing can take a long time; therefore, it might be interesting to quite tightly control its use.

[jugmac00]

@d-maurer Thank you for the hint. I know why I am a big fan of "explicit is better than implicit" :-)