Quantum\Logger\LoggerManager::getHandler()

Complexity

Conditions	4
Paths	5

Size

Total Lines	24
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public static function getHandler(): Logger
    {
        if(is_debug_mode()) {
            return LoggerFactory::createLogger();
        }

        if (!config()->has('logging')) {
            config()->import(new Setup('config', 'logging'));
        }

        $logAdapter = config()->get('logging.current');

        if (!in_array($logAdapter, self::ADAPTERS)) {
            throw LoggerException::unsupportedAdapter($logAdapter);

It seems like $logAdapter can also be of type null; however, parameter $name of Quantum\Logger\LoggerException::unsupportedAdapter() does only seem to accept string, maybe add an additional type check?

        }

        $logLevel = config()->get('logging.level', 'error');
        LoggerConfig::setAppLogLevel($logLevel);

It seems like $logLevel can also be of type null; however, parameter $level of Quantum\Logger\LoggerConfig::setAppLogLevel() does only seem to accept string, maybe add an additional type check?

        $logAdapterClass = __NAMESPACE__ . '\\Adapters\\' . ucfirst($logAdapter) . 'Adapter';

It seems like $logAdapter can also be of type null; however, parameter $string of ucfirst() does only seem to accept string, maybe add an additional type check?

        $logAdapter = new $logAdapterClass(config()->get('logging.' . $logAdapter));

        return LoggerFactory::createLogger($logAdapter);
    }
}