FondBot\Cache\FilesystemCache

Complexity

Total Complexity

9

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

91.3%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	get()	0	15	3
A	store()	0	10	2
A	forget()	0	7	2
A	key()	0	4	1

<?php

declare(strict_types=1);

namespace FondBot\Cache;

use JsonSerializable;
use FondBot\Contracts\Cache;
use League\Flysystem\Filesystem;
use League\Flysystem\FileNotFoundException;

class FilesystemCache implements Cache
{
    private $filesystem;

    public function __construct(Filesystem $filesystem)
    {
        $this->filesystem = $filesystem;
    }

    /**
     * Retrieve an item from the cache by key.
     *
     * @param string $key
     * @param mixed  $default
     *
     * @return mixed
     */
    public function get(string $key, $default = null)
    {
        try {
            $contents = $this->filesystem->read($this->key($key));
            $json = json_decode($contents, true);

            if (json_last_error() === JSON_ERROR_NONE) {
                return $json;
            }

            return $contents;
        } catch (FileNotFoundException $exception) {
            return $default;
        }
    }

    /**
     * Store an item in the cache.
     *
     * @param string $key
     * @param mixed  $value
     */
    public function store(string $key, $value): void
    {
        if ($value instanceof JsonSerializable) {

The class JsonSerializable does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

            $value = json_encode($value->jsonSerialize());
        } else {
            $value = json_encode($value);
        }

        $this->filesystem->put($this->key($key), $value);
    }

    /**
     * Remove an item from the cache.
     *
     * @param string $key
     */
    public function forget(string $key): void
    {
        try {
            $this->filesystem->delete($this->key($key));
        } catch (FileNotFoundException $exception) {

Consider adding a comment why this CATCH block is empty.

        }
    }

    private function key(string $key): string
    {
        return md5($key);
    }
}