micmania1\config\CachedConfigCollection

Complexity

Total Complexity

10

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	set()	0	18	3
A	get()	0	10	3
A	exists()	0	4	1
A	clear()	0	4	1
A	__destruct()	0	4	1

<?php

namespace micmania1\config;

use Psr\Cache\CacheItemPoolInterface;

class CachedConfigCollection extends ConfigCollectionInterface

There is one abstract method delete in this class; you could implement it, or declare this class as abstract.

{
    /**
     * @var CachePoolInterface
     */
    protected $pool;

    /**
     * @var boolean
     */
    protected $trackMetadata = false;

    /**
     * @param boolean $trackHistory

There is no parameter named $trackHistory. Was it maybe removed?

     * @param CacheItemPoolInterface $pool
     */
    public function __construct(CacheItemPoolInterface $pool, $trackMetadata = false)
    {
        $this->pool = $pool;

It seems like $pool of type object<Psr\Cache\CacheItemPoolInterface> is incompatible with the declared type object<micmania1\config\CachePoolInterface> of property $pool.

        $this->trackMetadata = (bool) $trackMetadata;
    }

    /**
     * {@inheritdoc}
     */
    public function set($key, ConfigItemInterface $item)
    {
        // We use null as the key to return an empty cache item
        $cacheItem = $this->pool->getItem($key);

        // If we're tracking history and already have an existing record, we grab
        // that and set the new record.
        if($this->Metadata && $existing = $cacheItem->get()) {

The property Metadata does not seem to exist. Did you mean trackMetadata?

            $existing->trackMetadata();
            $existing->set($item->getValue(), $item->getMetadata());
            $item = $existing;
        }

        $cacheItem->set($item);

        // We defer saving until later
        $this->pool->saveDeffered($cacheItem);
    }

    /**
     * {@inheritdoc}
     */
    public function get($key)
    {
        if(!$this->exists($key)) {
            return null;
        }

        $cacheItem = $this->pool->getItem($key);

        return $cacheItem->get() ?: new ConfigItem(null, [], $this->trackHistory);

The property trackHistory does not exist. Did you maybe forget to declare it?

    }

    /**
     * {@inheritdoc}
     */
    public function exists($key)
    {
        return $this->pool->hasItem($key);
    }

    /**
     * {@inheritdoc}
     */
    public function clear($key)
    {
        $this->pool->deleteItem($key);
    }

    /**
     * Commits the cache
     */
    public function __destruct()
    {
        $this->pool->commit();
    }
}