Desarrolla2\Cache\Option\FilenameTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Test Coverage

Coverage

70.59%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getFilenameOption()	0	7	2
A	setFilenameOption()	0	11	3
A	getFilename()	0	6	1
A	getWildcard()	0	5	1

<?php
/**
 * This file is part of the Cache package.
 *
 * Copyright (c) Daniel González
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @author Daniel González <daniel@desarrolla2.com>
 * @author Arnold Daniels <arnold@jasny.net>
 */

declare(strict_types=1);

namespace Desarrolla2\Cache\Option;

use TypeError;
use Desarrolla2\Cache\File\BasicFilename;

/**
 * Use filename generator
 */
trait FilenameTrait
{
    /**
     * @var callable
     */
    protected $filename;


    /**
     * Filename format or callable.
     * The filename format will be applied using sprintf, replacing `%s` with the key.
     *
     * @param string|callable $filename
     * @return void
     */
    protected function setFilenameOption($filename): void
    {
        if (is_string($filename)) {
            $filename = new BasicFilename($filename);
        }

        if (!is_callable($filename)) {
            throw new TypeError("Filename should be a string or callable");
        }

        $this->filename = $filename;
    }

    /**
     * Get the filename callable
     *
     * @return callable
     */
    protected function getFilenameOption(): callable
    {
        if (!isset($this->filename)) {
            $this->filename = new BasicFilename('%s.' . $this->getPacker()->getType());

It seems like getPacker() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $this->filename;
    }

    /**
     * Create a filename based on the key
     *
     * @param string|mixed $key
     * @return string
     */
    protected function getFilename($key): string
    {
        $id = $this->keyToId($key);

It seems like keyToId() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $generator = $this->getFilenameOption();

        return $this->cacheDir . DIRECTORY_SEPARATOR . $generator($id);
    }

    /**
     * Get a wildcard for all files
     *
     * @return string
     */
    protected function getWildcard(): string
    {
        $generator = $this->getFilenameOption();

        return $this->cacheDir . DIRECTORY_SEPARATOR . $generator('');
    }
}