PhpProfiler\Lib\Concurrency\Amphp\Task\PhpReaderTask

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	run()	0	22	2

<?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\Concurrency\Amphp\Task;

use Amp\Parallel\Sync\Channel;

The type Amp\Parallel\Sync\Channel was not found. Maybe you did not declare it correctly or list all dependencies?

use Generator;
use PhpProfiler\Command\Inspector\GetTraceCommand;

final class PhpReaderTask
{
    private Channel $channel;
    private GetTraceCommand $get_trace_command;

    public function __construct(Channel $channel, GetTraceCommand $get_trace_command)
    {
        $this->channel = $channel;
        $this->get_trace_command = $get_trace_command;
    }

    public function run(): Generator
    {
        /** @var int $pid */
        $pid = yield $this->channel->receive();
        $process = proc_open(
            [

array(PhpProfiler\Lib\Concurrency\Amphp\Task\PHP_BINARY, __DIR__ . '/../../../../../php-profiler', 'inspector:trace', '-p', $pid) of type array<integer,integer|string> is incompatible with the type string expected by parameter $cmd of proc_open().

                PHP_BINARY,
                __DIR__ . '/../../../../../php-profiler',
                'inspector:trace',
                '-p',
                $pid
            ],
            [
                0 => ['pipe', 'r'],
                1 => ['pipe', 'w'],
                2 => ['pipe', 'w'],
            ],
            $pipes
        );
        stream_set_blocking($pipes[1], false);
        for ($status = proc_get_status($process); $status['running']; $status = proc_get_status($process)) {

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

            yield $this->channel->send(stream_get_contents($pipes[1]));
        }
    }
}