Completed
Pull Request — master (#31)
by Théo
02:15
created

PhpScoper::__construct()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
c 0
b 0
f 0
dl 0
loc 4
rs 10
cc 1
eloc 2
nc 1
nop 2
1
<?php
2
3
declare(strict_types=1);
4
5
/*
6
 * This file is part of the box project.
7
 *
8
 * (c) Kevin Herrera <[email protected]>
9
 *     Théo Fidry <[email protected]>
10
 *
11
 * This source file is subject to the MIT license that is bundled
12
 * with this source code in the file LICENSE.
13
 */
14
15
namespace KevinGH\Box\Compactor;
16
17
use Humbug\PhpScoper\Console\Configuration as PhpScoperConfiguration;
18
use Humbug\PhpScoper\Scoper;
19
use KevinGH\Box\Compactor;
20
21
final class PhpScoper implements Compactor
22
{
23
    private $scoper;
24
    private $config;
25
26
    public function __construct(Scoper $scoper, PhpScoperConfiguration $config)
27
    {
28
        $this->scoper = $scoper;
29
        $this->config = $config;
30
    }
31
32
    /**
33
     * {@inheritdoc}
34
     */
35
    public function compact(string $file, string $contents): string
36
    {
37
        return $this->scoper->scope(
38
            $file,
39
            $contents,
40
            'TODOAllowNullPrefix',
41
            $this->config->getPatchers(),
42
            $this->config->getWhitelist(),
43
            $this->config->getGlobalNamespaceWhitelister()
0 ignored issues
show
Unused Code introduced by
The call to Humbug\PhpScoper\Scoper::scope() has too many arguments starting with $this->config->getGlobalNamespaceWhitelister(). ( Ignorable by Annotation )

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

43
        return $this->scoper->/** @scrutinizer ignore-call */ scope(

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
Bug introduced by
The method getGlobalNamespaceWhitelister() does not exist on Humbug\PhpScoper\Console\Configuration. ( Ignorable by Annotation )

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

43
            $this->config->/** @scrutinizer ignore-call */ 
44
                           getGlobalNamespaceWhitelister()

This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces.

This is most likely a typographical error or the method has been renamed.

Loading history...
44
        );
45
    }
46
47
    public function getConfiguration(): PhpScoperConfiguration
48
    {
49
        return $this->config;
50
    }
51
}
52