AppLoggerFactory::addHandlers() - Code Metrics - Inspection of "addressing scrutinizer issues;" - SlayerBirden/dataflow-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8cccfe...3ec575 )

by Oleg

created 2018-03-07 13:51 UTC

AppLoggerFactory::addHandlers() B

↳ Parent: AppLoggerFactory

Complexity

Conditions	8
Paths	5

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	8

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	9
cts	9
cp	1
rs	7.7777
c	0
b	0
f	0
cc	8
eloc	8
nc	5
nop	3
crap	8

<?php
declare(strict_types=1);

namespace SlayerBirden\DataFlowServer\Logger;

use Monolog\Handler\HandlerInterface;
use Monolog\Handler\StreamHandler;
use Monolog\Logger;
use Psr\Container\ContainerInterface;

class AppLoggerFactory
{
    /**
     * @param ContainerInterface $container
     * @return Logger
     * @throws \Psr\Container\ContainerExceptionInterface
     * @throws \Psr\Container\NotFoundExceptionInterface
     * @throws \Exception
     */
    public function __invoke(ContainerInterface $container)
    {
        $config = $container->has('config') ? $container->get('config') : [];
        $loggerConfig = $config['logger'] ?? [];

        $logger = new Logger('app');

        if (!empty($loggerConfig)) {
            $this->addHandlers($logger, $loggerConfig, $container);
        }

        if (!$logger->getHandlers()) {
            // add default handler
            $baseHandler = new StreamHandler('data/log/app.log');
            $logger->pushHandler($baseHandler);
        }

        return $logger;
    }

    private function addHandlers(Logger $logger, array $loggerConfig, ContainerInterface $container): void
    {
        $handlers = $loggerConfig['handlers'] ?? [];
        if (!empty($handlers) && is_array($handlers)) {
            foreach ($handlers as $handler) {
                if (is_object($handler) && $handler instanceof HandlerInterface) {
                    $logger->pushHandler($handler);
                } elseif (is_string($handler) && $container->has($handler)) {
                    $logger->pushHandler($container->get($handler));
                }
            }
        }
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace SlayerBirden\DataFlowServer\Logger;
5
6		use Monolog\Handler\HandlerInterface;
7		use Monolog\Handler\StreamHandler;
8		use Monolog\Logger;
9		use Psr\Container\ContainerInterface;
10
11		class AppLoggerFactory
12		{
13		/**
14		* @param ContainerInterface $container
15		* @return Logger
16		* @throws \Psr\Container\ContainerExceptionInterface
17		* @throws \Psr\Container\NotFoundExceptionInterface
18		* @throws \Exception
19		*/
20	34	public function __invoke(ContainerInterface $container)
21		{
22	34	$config = $container->has('config') ? $container->get('config') : [];
23	34	$loggerConfig = $config['logger'] ?? [];
24
25	34	$logger = new Logger('app');
26
27	34	if (!empty($loggerConfig)) {
28	34	$this->addHandlers($logger, $loggerConfig, $container);
29		}
30
31	34	if (!$logger->getHandlers()) {
32		// add default handler
33	1	$baseHandler = new StreamHandler('data/log/app.log');
34	1	$logger->pushHandler($baseHandler);
35		}
36
37	34	return $logger;
38		}
39
40	34	private function addHandlers(Logger $logger, array $loggerConfig, ContainerInterface $container): void
41		{
42	34	$handlers = $loggerConfig['handlers'] ?? [];
43	34	if (!empty($handlers) && is_array($handlers)) {
44	34	foreach ($handlers as $handler) {
45	34	if (is_object($handler) && $handler instanceof HandlerInterface) {
46	1	$logger->pushHandler($handler);
47	33	} elseif (is_string($handler) && $container->has($handler)) {
48	34	$logger->pushHandler($container->get($handler));
49		}
50		}
51		}
52	34	}
53		}
54

SlayerBirden / dataflow-server

Push — master ( 8cccfe...3ec575 )

AppLoggerFactory::addHandlers() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like