(REF) CRM_Utils_Hook - Remove deprecated formulations of `invoke(int,...)`
authorTim Otten <totten@civicrm.org>
Tue, 21 Apr 2020 00:12:08 +0000 (17:12 -0700)
committerTim Otten <totten@civicrm.org>
Tue, 21 Apr 2020 04:28:58 +0000 (21:28 -0700)
commit413446611957fe022805a323a1261ed8d7738e8b
treeba0a4a83700c9da1fc1da72cdd02601eea1fc059
parent98368a89f0e80d5b6c412db1fbe8ced11985f5aa
(REF) CRM_Utils_Hook - Remove deprecated formulations of `invoke(int,...)`

Overview
--------

The signature of `CRM_Utils_Hook::invoke()` originally took an argument `int
$count` (number of hook args), but circa 4.7 it changed to prefer `string[]
$names` (symbolic names for each hook arg).  The `int` notation doesn't
provide enough information to fire via Symfony EventDispatcher and was
mostly replaced/deprecated, but `int` notation is still supported for
backward compatibility.

This cleans a couple oddballs using the `int` notation.

Before
------

Some stub functions in `CRM_Utils_Hook` still use the `invoke(int,...)` formulation:

* `hook_themes` fell through a crack because it had a long PR (years?) that
   overlapped with the change in `invoke()`.
* `hook_install` (etc) looks like `int`, but it's not actually used.

After
-----

None of the stub functions in `CRM_Utils_Hook::invoke(string[], ...)` use
`int` formulation.
CRM/Utils/Hook.php