Cubiche\Core\Metadata\Cache\FileCache

Complexity

Total Complexity

9

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	load()	0	9	2
A	save()	0	10	1
A	renameFile()	0	6	2
A	fileName()	0	4	1
A	__construct()	0	12	3

<?php
/**
 * This file is part of the Cubiche package.
 *
 * Copyright (c) Cubiche
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Cubiche\Core\Metadata\Cache;

use Cubiche\Core\Metadata\ClassMetadata;

/**
 * FileCache class.
 *
 * @author Ivannis Suárez Jerez <ivannis.suarez@gmail.com>
 */
class FileCache implements CacheInterface
{
    /**
     * @var string
     */
    protected $cacheDirectory;

    /**
     * FileCache constructor.
     *
     * @param string $cacheDirectory
     */
    public function __construct($cacheDirectory)
    {
        if (!is_dir($cacheDirectory)) {
            mkdir($cacheDirectory, 0755, true);
        }

        if (!is_writable($cacheDirectory)) {
            throw new \InvalidArgumentException(sprintf('The directory "%s" is not writable.', $cacheDirectory));
        }

        $this->cacheDirectory = rtrim($cacheDirectory, '\\/');
    }

    /**
     * {@inheritdoc}
     */
    public function load($className)
    {
        $filename = $this->fileName($className);
        if (!file_exists($filename)) {
            return;
        }

        return include $filename;
    }

    /**
     * {@inheritdoc}
     */
    public function save(ClassMetadata $metadata)
    {
        $filename = $this->fileName($metadata->className());

        $tmpFile = tempnam($this->cacheDirectory, 'metadata-cache');
        file_put_contents($tmpFile, '<?php return unserialize('.var_export(serialize($metadata), true).');');

        @chmod($tmpFile, 0666 & ~umask());

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

        $this->renameFile($tmpFile, $filename);
    }

    /**
     * Renames a file with fallback for windows.
     *
     * @param string $source
     * @param string $target
     */
    private function renameFile($source, $target)
    {
        if (false === @rename($source, $target)) {
            throw new \RuntimeException(sprintf('Could not write new cache file to %s.', $target));
        }
    }

    /**
     * @param string $className
     *
     * @return string
     */
    protected function fileName($className)
    {
        return $this->cacheDirectory.'/'.strtr($className, '\\', '-').'.cache.php';
    }
}