Gacela\Framework\ClassResolver\Profiler\GacelaProfiler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	resetCache()	0	3	1
A	__construct()	0	3	1
A	isEnabled()	0	9	3

<?php

declare(strict_types=1);

namespace Gacela\Framework\ClassResolver\Profiler;

use Gacela\Framework\Config\ConfigInterface;

final class GacelaProfiler
{
    public const KEY_ENABLED = 'gacela-profiler-enabled';
    public const DEFAULT_ENABLED_VALUE = false;
    public const DEFAULT_DIRECTORY_VALUE = '/.gacela/profiler';

    private static ?bool $isEnabled = null;

    private ConfigInterface $config;

    public function __construct(ConfigInterface $config)
    {
        $this->config = $config;
    }

    /**
     * @internal
     */
    public static function resetCache(): void
    {
        self::$isEnabled = null;
    }

    public function isEnabled(): bool
    {
        if (self::$isEnabled === null) {
            self::$isEnabled = $this->config->hasKey(self::KEY_ENABLED)
                ? (bool)$this->config->get(self::KEY_ENABLED)
                : $this->config->getSetupGacela()->isProfilerEnabled();
        }

        return self::$isEnabled;

The expression return self::isEnabled could return the type null which is incompatible with the type-hinted return boolean. Consider adding an additional type-check to rule them out.

    }
}