MidasSoft\DominicanBankParser\Cache\FileCacheDriver

Complexity

Total Complexity

4

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	remove()	0	3	1
A	add()	0	8	1
A	get()	0	7	1
A	__construct()	0	3	1

<?php

namespace MidasSoft\DominicanBankParser\Cache;

class FileCacheDriver extends AbstractCacheDriver
{
    /**
     * The config of the driver.
     *
     * @var array
     */
    protected $config = [
        'path' => '',
        'timezone' => 'America/Santo_Domingo',
    ];

    /**
     * Creates a new FileCacheDriver instance.
     *
     * @param array $config
     */
    public function __construct(array $config)
    {
        $this->config = $config;
    }

    /**
     * {@inheritDoc}
     */
    public function add($key, $value)
    {
        $filename = sprintf('%s/%s.txt', $this->config['path'], $key);
        $file = fopen($filename, 'w');

        fwrite($file, serialize($value));

It seems like $file can also be of type false; however, parameter $handle of fwrite() does only seem to accept resource, maybe add an additional type check?

        fclose($file);

It seems like $file can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

        $this->keys[] = $key;
    }

    /**
     * {@inheritDoc}
     */
    public function get($key)
    {
        $filename = sprintf('%s/%s.txt', $this->config['path'], $key);
        $file = fopen($filename, 'r');
        $content = fread($file, filesize($filename));

It seems like $file can also be of type false; however, parameter $handle of fread() does only seem to accept resource, maybe add an additional type check?

        return unserialize($content);
    }

    /**
     * {@inheritDoc}
     */
    public function remove($key)
    {
        unlink(sprintf('%s/%s.txt', $this->config['path'], $key));
    }
}