Silviooosilva\CacheerPhp\Helpers\FlushHelper

Complexity

Total Complexity

2

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	pathFor()	0	9	2

<?php

namespace Silviooosilva\CacheerPhp\Helpers;

/**
 * Class FlushHelper
 * 
 * @author Sílvio Silva <https://github.com/silviooosilva>
 * @package Silviooosilva\CacheerPhp
 * 
 * Builds deterministic file paths for last-flush timestamps per store type.
 */
class FlushHelper
{
    /**
     * Returns a path to store a last-flush timestamp.
     * @param string $storeType e.g., 'redis' or 'db'
     * @param string $identifier e.g., namespace or table name
     * @return string
     */
    public static function pathFor(string $storeType, string $identifier): string
    {
        $root = EnvHelper::getRootPath();
        $dir = $root . DIRECTORY_SEPARATOR . 'CacheerPHP' . DIRECTORY_SEPARATOR . 'Flush';
        if (!is_dir($dir)) {
            @mkdir($dir, 0755, true);

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

        }
        $safeId = preg_replace('/[^a-zA-Z0-9_-]+/', '_', $identifier);
        return $dir . DIRECTORY_SEPARATOR . $storeType . '_' . $safeId . '.time';
    }
}