App\Services\MoviesProcessor

Complexity

Total Complexity

4

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	0	5	3
A	__construct()	0	3	1

<?php

namespace App\Services;

use App\Models\Settings;
use Blacklight\Movie;
use GuzzleHttp\Exception\GuzzleException;

class MoviesProcessor
{
    private bool $echooutput;

    public function __construct(bool $echooutput)
    {
        $this->echooutput = $echooutput;
    }

    /**
     * @param  int|string|null  $processMovies  0/1/2 or '' to read from settings
     *
     * @throws GuzzleException
     */
    public function process(string $groupID = '', string $guidChar = '', int|string|null $processMovies = ''): void
    {
        $processMovies = (is_numeric($processMovies) ? $processMovies : Settings::settingValue('lookupimdb'));
        if ($processMovies > 0) {
            (new Movie)->processMovieReleases($groupID, $guidChar, $processMovies);

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?

        }
    }
}