[REF] Replace deprecated code call

[eileenmcnaughton]

Overview

[REF] Replace deprecated code call

Before

Deprecated fn call

CRM_Core_DAO::triggerRebuild()

After

Civi::service('sql_triggers')->rebuild()

Technical Details

I hadn't made this change before because the Civi::service() facade means it takes a bit of effort to find the function being called & verify the signature is in fact the same - however I did it this time & it is.

Perhaps as a follow up we should rip through & do all the others - I'm tempted to perhaps add a comment to say what the actually called function is before each line but I guess in theory the Civi facade is dev-friendly

Also I'm very tempted to rip out the check to the url retrieve - CRM_Utils_Request::retrieve('triggerRebuild', 'Boolean', CRM_Core_DAO::$_nullObject, FALSE, 0, 'GET') = this seems to me like an undocumented feature that was used by devs when developing the triggers but which devs would do in other ways now...

Comments

[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).