Passed
Pull Request — master (#9)
by Shinji
01:31
created

PhpReaderTask::__construct()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 2
nc 1
nop 2
dl 0
loc 4
c 1
b 0
f 0
cc 1
rs 10
1
<?php
2
3
/**
4
 * This file is part of the sj-i/php-profiler package.
5
 *
6
 * (c) sji <[email protected]>
7
 *
8
 * For the full copyright and license information, please view the LICENSE
9
 * file that was distributed with this source code.
10
 */
11
12
declare(strict_types=1);
13
14
namespace PhpProfiler\Lib\Concurrency\Amphp\Task;
15
16
use Amp\Parallel\Sync\Channel;
0 ignored issues
show
Bug introduced by
The type Amp\Parallel\Sync\Channel was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
17
use Generator;
18
use PhpProfiler\Command\Inspector\GetTraceCommand;
19
20
final class PhpReaderTask
21
{
22
    private Channel $channel;
23
    private GetTraceCommand $get_trace_command;
24
25
    public function __construct(Channel $channel, GetTraceCommand $get_trace_command)
26
    {
27
        $this->channel = $channel;
28
        $this->get_trace_command = $get_trace_command;
29
    }
30
31
    public function run(): Generator
32
    {
33
        /** @var int $pid */
34
        $pid = yield $this->channel->receive();
35
        $process = proc_open(
36
            [
0 ignored issues
show
Bug introduced by
array(PhpProfiler\Lib\Co...tor:trace', '-p', $pid) of type array<integer,integer|string> is incompatible with the type string expected by parameter $cmd of proc_open(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

36
            /** @scrutinizer ignore-type */ [
Loading history...
37
                PHP_BINARY,
38
                __DIR__ . '/../../../../../php-profiler',
39
                'inspector:trace',
40
                '-p',
41
                $pid
42
            ],
43
            [
44
                0 => ['pipe', 'r'],
45
                1 => ['pipe', 'w'],
46
                2 => ['pipe', 'w'],
47
            ],
48
            $pipes
49
        );
50
        stream_set_blocking($pipes[1], false);
51
        for ($status = proc_get_status($process); $status['running']; $status = proc_get_status($process)) {
0 ignored issues
show
Bug introduced by
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? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

51
        for ($status = proc_get_status(/** @scrutinizer ignore-type */ $process); $status['running']; $status = proc_get_status($process)) {
Loading history...
52
            yield $this->channel->send(stream_get_contents($pipes[1]));
53
        }
54
    }
55
}
56