PhpProfiler\Lib\File\CatFileReader

Complexity

Total Complexity

1

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	readAll()	0	19	1

<?php

/**
 * This file is part of the sj-i/php-profiler package.
 *
 * (c) sji <sji@sj-i.dev>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace PhpProfiler\Lib\File;

/**
 * Class CatFileReader
 *
 * workaround for a problem that PHP cannot open files in /proc/<pid>/root/
 *
 * @package PhpProfiler\Lib\File
 */
final class CatFileReader implements FileReaderInterface
{
    /**
     * @param string $path
     * @return string
     */
    public function readAll(string $path): string
    {
        /** @psalm-suppress InvalidArgument */
        $process = proc_open(
            [

array('cat', $path) of type array<integer,string> is incompatible with the type string expected by parameter $cmd of proc_open().

                'cat',
                $path
            ],
            [
                0 => ['pipe', 'r'],
                1 => ['pipe', 'w'],
                2 => ['pipe', 'w'],
            ],
            $pipes
        );
        $contents = stream_get_contents($pipes[1]);
        proc_close($process);

It seems like $process can also be of type false; however, parameter $process of proc_close() does only seem to accept resource, maybe add an additional type check?

        return $contents;
    }
}