Update function to repair indexes to take table as a parameter

[eileenmcnaughton]

Overview

Fixes the System.getmissingindices & System.updateindexes apis to accept 'tables' (array or string) as a parameter & limit to the specified tables if supplied

Before

Table index repair is all or nothing

After

Table index repair can be filtered per

civicrm_api3('System', 'getmissingindices', ['tables' => ['civicrm_contact']]);
civicrm_api3('System', 'updateindexes', ['tables' => ['civicrm_contact']]);

Technical Details

The inconsistency on indexes vs indices preceeds this - I have used the right English rather than copy the existing api & left tidying that up (perhaps a wrapper api with the right name) out of scope.

Comments

I'd like to go a step further & allow individual indices to be dropped - this would allow a UI for this cleanup

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

New api actually added separately #16145

[colemanw]

Non-blocking but grumpy cat comment: 🙀 :
Arguably new api functionality should be added to v4.

[eileenmcnaughton]

@colemanw yeah - I just feel we don't have enough precedent for odd api functions in v4 to be sure how we want things like this to look

[eileenmcnaughton]

Thanks @mattwire