Gacela\Framework\ClassResolver\Cache\GacelaFileCache

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

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

<?php

declare(strict_types=1);

namespace Gacela\Framework\ClassResolver\Cache;

use Gacela\Framework\Bootstrap\CacheConfigurationInterface;
use Gacela\Framework\Config\ConfigInterface;

final class GacelaFileCache
{
    public const KEY_ENABLED = 'gacela-cache-enabled';

    public const DEFAULT_ENABLED_VALUE = false;

    public const DEFAULT_DIRECTORY_VALUE = null;

    private static ?bool $isEnabled = null;

    public function __construct(
        private readonly ConfigInterface $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()->isFileCacheEnabled();
        }

        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.

    }

    public static function isEnabledFromCacheConfig(CacheConfigurationInterface $cacheConfig): bool
    {
        if (self::$isEnabled === null) {
            self::$isEnabled = $cacheConfig->isFileCacheEnabled();
        }

        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.

    }
}