This is a low hanging fruit.
Old Pattern: global
Pattern a:
global $conf;
if (empty($conf['holidays']['enable'])) { // whatever}
Pattern b:
if (empty($GLOBALS['conf'][holidays']['enable'])) { // whatever}
New Pattern: KronolithConfig
At the top of file:
use Horde\Kronolith\KronolithConfig;
At the place where you found the pattern:
$config = $injector->get('KronolithConfig');
if (empty($config->get('holidays.enable'))) { // whatever}
PRs for individual or few conversions are accepted, you don't need to submit a large bulk transformation PR.
This is a low hanging fruit.
Old Pattern: global
Pattern a:
global $conf;
if (empty($conf['holidays']['enable'])) { // whatever}
Pattern b:
if (empty($GLOBALS['conf'][holidays']['enable'])) { // whatever}
New Pattern: KronolithConfig
At the top of file:
use Horde\Kronolith\KronolithConfig;
At the place where you found the pattern:
$config = $injector->get('KronolithConfig');
if (empty($config->get('holidays.enable'))) { // whatever}
PRs for individual or few conversions are accepted, you don't need to submit a large bulk transformation PR.