Shlinkio\Shlink\Common\Logger\LoggerFactory::__invoke()

Complexity

Conditions	3
Paths	4

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

0

<?php
    public function __invoke(ContainerInterface $container, $requestedName, ?array $options = null)
    {
        $config = $container->has('config') ? $container->get('config') : [];
        Cascade::fileConfig($config['logger'] ?? ['loggers' => []]);

It seems like $config['logger'] ?? array('loggers' => array()) can also be of type array<string,array>; however, parameter $resource of Cascade\Cascade::fileConfig() does only seem to accept string, maybe add an additional type check?

        // Compose requested logger name
        $loggerName = $options['logger_name'] ?? 'Logger';
        $nameParts = explode('_', $requestedName);
        if (count($nameParts) > 1) {
            $loggerName = $nameParts[1];
        }

        return Cascade::getLogger($loggerName);
    }