Fix config is null on hooks

[stesi561]

Overview

Encountered error when expecting config to be initialised when using hook_civicrm_config.

Found due to https://github.com/eileenmcnaughton/nz.co.fuzion.civixero/blob/bfa6ec6e0d1906fe9b96e0d364e6a19d7b453331/Civixero.php#L16

which defines the hook as:
function civixero_civicrm_config(CRM_Core_Config $config) {

Seems like in the D7 code config is initialised but somewhere along the way this didn't make it do D8.

https://docs.civicrm.org/dev/en/latest/hooks/hook_civicrm_config/

Before

hook_civicrm_config is called with null, resulting in a Fatal error if nz.co.fuzion.civixero extension latest version is installed.

After

hook_civicrm_config is called with config object.

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

@totten can you check this - the other CMS pass config through - these 2 don't as it turns out.... and haven't - so 'fixing' could have unexpected consquences ... maybe

[totten]

@eileenmcnaughton @stesi561 - I would be optimistic about this change. If there were some edge-case where an extension only worked at moment when $config===NULL, then it probably would've been broken already (from the POV of D7/WP/BD/J).

For due diligence, I've fired a test-run on top of D9 (https://test.civicrm.org/job/CiviCRM-Manual-Test/67/parameters/). One might also do some light r-run with cv (since that's one of the few places where it's still possible to hit loadBootStrap on D8+)

[eileenmcnaughton]

OK - let's merge it - it does make me wonder if we really need to pass out the config object - but whatever