Fix bug where log hook is ignored on custom field create
authorEileen McNaughton <emcnaughton@wikimedia.org>
Tue, 21 Sep 2021 00:52:26 +0000 (12:52 +1200)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Wed, 22 Sep 2021 19:54:35 +0000 (07:54 +1200)
commit78b7cb8930864ab3965a7eb7a701daf8d2e4135e
treeb882ec171cde583e9c79ca647fbb69c89874f992
parente7d877e11f1aed601fd280afba439d5280c83610
Fix bug where log hook is ignored on custom field create

It is possible to specify that various tables should not
result in a log table via the `alterLogTables` hook.

This is useful for excluding tables of low information
value or tables like summary fields that are calculated
values. However, when altering custom fields this
hook's output was being ignored as it was going through
the specific path of
which did not have awareness of this hook.

This code path is the primary use for this function.

It's also called via createMissingLogTables - but in
that case the calculation is already done. It is not
elsewhere in git universe and the return value
is never used, so I removed it
CRM/Core/BAO/CustomField.php
CRM/Core/DAO.php
CRM/Logging/Schema.php
tests/phpunit/CRM/Logging/LoggingTest.php