Passed
Push — master ( b2d632...002539 )
by Darko
10:06
created

MoviesProcessor::process()   A

Complexity

Conditions 3
Paths 4

Size

Total Lines 5
Code Lines 3

Duplication

Lines 0
Ratio 0 %

Importance

Changes 2
Bugs 1 Features 0
Metric Value
eloc 3
c 2
b 1
f 0
dl 0
loc 5
rs 10
cc 3
nc 4
nop 3
1
<?php
2
3
namespace App\Services;
4
5
use App\Models\Settings;
6
use Blacklight\Movie;
7
use GuzzleHttp\Exception\GuzzleException;
8
9
class MoviesProcessor
10
{
11
    private bool $echooutput;
12
13
    public function __construct(bool $echooutput)
14
    {
15
        $this->echooutput = $echooutput;
16
    }
17
18
    /**
19
     * @param  int|string|null  $processMovies  0/1/2 or '' to read from settings
20
     *
21
     * @throws GuzzleException
22
     */
23
    public function process(string $groupID = '', string $guidChar = '', int|string|null $processMovies = ''): void
24
    {
25
        $processMovies = (is_numeric($processMovies) ? $processMovies : Settings::settingValue('lookupimdb'));
26
        if ($processMovies > 0) {
27
            (new Movie)->processMovieReleases($groupID, $guidChar, $processMovies);
0 ignored issues
show
Bug introduced by
It seems like $processMovies can also be of type string; however, parameter $lookupIMDB of Blacklight\Movie::processMovieReleases() does only seem to accept integer, 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

27
            (new Movie)->processMovieReleases($groupID, $guidChar, /** @scrutinizer ignore-type */ $processMovies);
Loading history...
28
        }
29
    }
30
}
31