BrainExe\Core\Traits\FileCacheTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

93.33%

Importance

Changes	3
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getCacheFileName()	0	4	1
A	includeFile()	0	9	2
A	dumpCacheFile()	0	7	1
A	dumpVariableToCache()	0	4	1

<?php

namespace BrainExe\Core\Traits;

/**
 * @api
 */
trait FileCacheTrait
{

    /**
     * @param string $name
     * @return mixed
     */
    protected function includeFile(string $name)
    {
        $filename = $this->getCacheFileName($name);
        if (!is_file($filename)) {
            return null;
        }

        return include $filename;
    }

    /**
     * @param string $name
     * @param string $content
     */
    protected function dumpCacheFile(string $name, string $content)
    {
        $fileName = $this->getCacheFileName($name);

        file_put_contents($fileName, '<?php' . PHP_EOL . $content);
        @chmod($fileName, 0777);

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

    }

    /**
     * @param string $name
     * @param mixed $variable
     */
    protected function dumpVariableToCache(string $name, $variable)
    {
        $this->dumpCacheFile($name, 'return ' . var_export($variable, true) . ';');
    }

    /**
     * @param string $name
     * @return string
     */
    private function getCacheFileName($name)
    {
        return ROOT . 'cache/' . basename($name, '.php')  . '.php';
    }
}