Basis\Cache

Complexity

Total Complexity

16

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

92.5%

Importance

Changes	4
Bugs	3	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	exists()	0	10	3
A	get()	0	6	2
A	set()	0	12	1
B	wrap()	0	18	8

<?php

namespace Basis;

use Basis\Converter;
use Carbon\Carbon;

class Cache
{
    private $converter;

    public function __construct(Converter $converter)
    {
        $this->converter = $converter;
        if (!is_dir('.cache')) {
            mkdir('.cache');
            @chown('.cache', 'www-data');

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

            @chgrp('.cache', 'www-data');

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

        }
    }

    private $cache = [];

    public function exists($key)
    {
        $filename = '.cache/'.$key;
        if (file_exists($filename)) {
            if (!array_key_exists($key, $this->cache)) {
                $this->cache[$key] = include $filename;
            }
            return $this->cache[$key]['expire'] > Carbon::now()->getTimestamp();
        }
    }

    public function get($key)
    {
        if ($this->exists($key)) {
            return $this->converter->toObject($this->cache[$key]);
        }
    }

    public function set($key, $value)
    {
        $filename = '.cache/'.$key;
        $data = $this->converter->toArray($value);
        $string = '<?php return '.var_export($data, true).';';

        file_put_contents($filename, $string);
        @chown($filename, 'www-data');

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

        @chgrp($filename, 'www-data');

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

        $this->cache[$key] = $data;
    }

    public function wrap($key, $callback)
    {
        $hash = md5(json_encode($key));
        if ($this->exists($hash)) {
            return $this->get($hash);
        }
        $result = call_user_func($callback);
        $expire = null;
        if (is_array($result) && array_key_exists('expire', $result)) {
            $expire = $result['expire'];
        } elseif (is_object($result) && property_exists($result, 'expire')) {
            $expire = $result->expire;
        }
        if ($expire && $expire > Carbon::now()->getTimestamp()) {
            $this->set($hash, $result);
        }
        return $result;
    }
}