PhpMob\Settings\Manager\CachedManager

Complexity

Total Complexity

11

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getCacheKey()	0	7	2
A	setSetting()	0	14	3
A	getSetting()	0	16	2
A	flush()	0	4	1
A	get()	0	6	1
A	set()	0	6	1

<?php

/*
 * This file is part of the PhpMob package.
 *
 * (c) Ishmael Doss <nukboon@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace PhpMob\Settings\Manager;

use Psr\Cache\CacheItemPoolInterface;

/**
 * @author Ishmael Doss <nukboon@gmail.com>
 */
class CachedManager implements SettingManagerInterface
{
    /**
     * @var CacheItemPoolInterface
     */
    private $cache;

    /**
     * @var int|\DateInterval|null
     */
    private $ttl = 0;

    /**
     * @var SettingManagerInterface
     */
    private $decoratedManager;

    public function __construct(SettingManagerInterface $manager, CacheItemPoolInterface $cache, $ttl = 0)
    {
        $this->decoratedManager = $manager;
        $this->cache = $cache;
        $this->ttl = $ttl;
    }

    /**
     * @param $section
     * @param $key
     * @param string|null $owner
     *
     * @return string
     */
    private function getCacheKey($section, $key, ?string $owner = null)
    {
        return 'phpmob-settings.' . ($owner
                ? sprintf('%s.%s.%s', $section, $key, $owner)
                : sprintf('%s.%s', $section, $key)
            );
    }

    /**
     * {@inheritdoc}
     */
    public function setSetting(string $section, string $key, $value, ?string $owner = null, $autoFlush = false): void
    {
        $cacheKey = $this->getCacheKey($section, $key, $owner);

        if ($this->cache->hasItem($cacheKey) && $value === $this->cache->getItem($cacheKey)->get()) {
            return;
        }

        $this->decoratedManager->setSetting($section, $key, $value, $owner, $autoFlush);

        $this->cache->save(
            $this->cache->getItem($cacheKey)->set($value)->expiresAfter($this->ttl)
        );
    }

    /**
     * {@inheritdoc}
     */
    public function getSetting(string $section, string $key, ?string $owner = null)
    {
        $cacheKey = $this->getCacheKey($section, $key, $owner);

        if ($this->cache->hasItem($cacheKey)) {
            return $this->cache->getItem($cacheKey)->get();
        }

        $value = $this->decoratedManager->getSetting($section, $key, $owner);

        $this->cache->save(
            $this->cache->getItem($cacheKey)->set($value)->expiresAfter($this->ttl)
        );

        return $value;
    }

    /**
     * {@inheritdoc}
     */
    public function flush(): void
    {
        $this->decoratedManager->flush();
    }

    /**
     * {@inheritdoc}
     */
    public function get(string $path, ?string $owner = null)
    {
        @list($section, $key) = explode('.', $path);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        return $this->getSetting($section, $key, $owner);
    }

    /**
     * {@inheritdoc}
     */
    public function set(string $path, $value, ?string $owner = null): void
    {
        @list($section, $key) = explode('.', $path);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $this->setSetting($section, $key, $value, $owner, true);
    }
}