Ekino\Drupal\Debug\Configuration\Model\DefaultsConfiguration

Complexity

Total Complexity

10

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Test Coverage

Coverage

86.96%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	serialize()	0	5	2
A	__construct()	0	5	1
A	getLogger()	0	11	3
A	getFileLinkFormat()	0	3	1
A	unserialize()	0	3	1
A	getCharset()	0	3	1
A	getCacheDirectory()	0	3	1

<?php

declare(strict_types=1);

/*
 * This file is part of the ekino Drupal Debug project.
 *
 * (c) ekino
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Ekino\Drupal\Debug\Configuration\Model;

use Monolog\Handler\StreamHandler;
use Monolog\Logger;

class DefaultsConfiguration extends AbstractConfiguration
{
    /**
     * @var false|Logger|null
     */
    private $logger;

    /**
     * {@inheritdoc}
     */
    public function __construct(array $processedConfiguration)
    {
        parent::__construct($processedConfiguration);

        $this->logger = false;
    }

    /**
     * @return string
     */
    public function getCacheDirectory(): string
    {
        return $this->processedConfiguration['cache_directory'];
    }

    /**
     * @return Logger|null
     */
    public function getLogger(): ?Logger
    {
        if (false === $this->logger) {
            $loggerProcessedConfiguration = $this->processedConfiguration['logger'];

            $this->logger = $loggerProcessedConfiguration['enabled'] ? new Logger($loggerProcessedConfiguration['channel'], array(
                new StreamHandler($loggerProcessedConfiguration['file_path']),
            )) : null;
        }

        return $this->logger;

The expression return $this->logger could return the type true which is incompatible with the type-hinted return Monolog\Logger|null. Consider adding an additional type-check to rule them out.

    }

    /**
     * @return string|null
     */
    public function getCharset(): ?string
    {
        return $this->processedConfiguration['charset'];
    }

    /**
     * @return string|null
     */
    public function getFileLinkFormat(): ?string
    {
        return $this->processedConfiguration['file_link_format'];
    }

    /**
     * {@inheritdoc}
     */
    public function serialize(): ?string
    {
        return \serialize(array(
            $this->processedConfiguration,
            null === $this->logger ? null : false,
        ));
    }

    /**
     * {@inheritdoc}
     */
    public function unserialize($serialized): void
    {
        list($this->processedConfiguration, $this->logger) = \unserialize($serialized);
    }
}