bytic / dispatcher

src/Resolver/Cache/FileManager.php

<?php

namespace Nip\Dispatcher\Resolver\Cache;

/**
 * Class FileManager
 * @package Nip\Dispatcher\Resolver\Cache
 */
class FileManager
{
    /**
     * @param DefinitionsCollection $definitionCollection
     * @param null $path

Are you sure the doc-type for parameter $path is correct as it would always require null to be passed?

     */
    public static function write($definitionCollection, $path = null)
    {
        $path = $path ? $path : static::filePath();

$path is of type null, thus it always evaluated to false.

        $content = '<?php return ' . var_export($definitionCollection->all(), true) . ';';
        file_put_contents($path, $content);
    }

    /**
     * @param DefinitionsCollection $definitionCollection
     * @param null $path

Are you sure the doc-type for parameter $path is correct as it would always require null to be passed?

     */
    public static function read($definitionCollection, $path = null)
    {
        $path = $path ? $path : static::filePath();

$path is of type null, thus it always evaluated to false.

        /** @noinspection PhpIncludeInspection */
        $items = require $path;
        $definitionCollection->replace($items);
    }

    /**
     * @param null $path

Are you sure the doc-type for parameter $path is correct as it would always require null to be passed?

     */
    public static function empty($path = null)
    {
        $path = $path ? $path : static::filePath();

$path is of type null, thus it always evaluated to false.

        $content = '<?php return [];';
        file_put_contents($path, $content);
    }

    /**
     * @return bool
     */
    public static function hasCacheFile()
    {
        $path = static::filePath();
        return is_file($path);
    }

    /**
     * @return string
     */
    public static function filePath()
    {
        return static::filePathBase() . '/dispatcher.php';
    }

    /**
     * @return string
     */
    public static function filePathBase()
    {
        return function_exists('app') && app()->has('path.storage')
            ? app('path.storage') . DIRECTORY_SEPARATOR . 'cache'

Are you sure app('path.storage') of type mixed|object can be used in concatenation?

            : dirname(dirname(dirname(__DIR__))) . '/cache';
    }
}