Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Passed
Push — fix-save-actions ( 1e2da8 )
by Pedro
12:25
created

PublishAssets::handle()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 1
nc 1
nop 0
dl 0
loc 3
rs 10
c 0
b 0
f 0
1
<?php
2
3
namespace Backpack\CRUD\app\Console\Commands;
4
5
use Illuminate\Console\Command;
6
use Symfony\Component\Process\Exception\ProcessFailedException;
7
use Symfony\Component\Process\Process;
8
9
class PublishAssets extends Command
10
{
11
    /**
12
     * The name and signature of the console command.
13
     *
14
     * @var string
15
     */
16
    protected $signature = 'backpack:publish-assets';
17
18
    /**
19
     * The console command description.
20
     *
21
     * @var string
22
     */
23
    protected $description = 'Publish new CSS and JS assets (will override existing ones).';
24
25
    /**
26
     * Execute the console command.
27
     *
28
     * @return mixed
29
     */
30
    public function handle()
31
    {
32
        $this->runConsoleCommand(['php', 'artisan', 'vendor:publish', '--provider=Backpack\CRUD\BackpackServiceProvider', '--tag=public', '--force']);
0 ignored issues
show
Bug introduced by
array('php', 'artisan', ...tag=public', '--force') of type array<integer,string> is incompatible with the type string expected by parameter $command of Backpack\CRUD\app\Consol...ts::runConsoleCommand(). ( Ignorable by Annotation )

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

32
        $this->runConsoleCommand(/** @scrutinizer ignore-type */ ['php', 'artisan', 'vendor:publish', '--provider=Backpack\CRUD\BackpackServiceProvider', '--tag=public', '--force']);
Loading history...
33
    }
34
35
    /**
36
     * Run a shell command in a separate process.
37
     *
38
     * @param  string  $command  Text to be executed.
39
     * @return void
40
     */
41
    private function runConsoleCommand($command)
42
    {
43
        $process = new Process($command, null, null, null, 60, null);
0 ignored issues
show
Bug introduced by
$command of type string is incompatible with the type array expected by parameter $command of Symfony\Component\Process\Process::__construct(). ( Ignorable by Annotation )

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

43
        $process = new Process(/** @scrutinizer ignore-type */ $command, null, null, null, 60, null);
Loading history...
Unused Code introduced by
The call to Symfony\Component\Process\Process::__construct() has too many arguments starting with null. ( Ignorable by Annotation )

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

43
        $process = /** @scrutinizer ignore-call */ new Process($command, null, null, null, 60, null);

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...
44
        $process->run(function ($type, $buffer) {
45
            $this->line($buffer);
46
        });
47
48
        // executes after the command finishes
49
        if (! $process->isSuccessful()) {
50
            throw new ProcessFailedException($process);
51
        }
52
    }
53
}
54