The call to App\Commands\BaseCommand::isSupported() has too many arguments starting with $docset.
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the ignore-call annotation
16
if ($this->/** @scrutinizer ignore-call */ isSupported($docset)) {
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...
17
$this->info(Str::ucfirst("$action started"));
18
(new DocsetBuilder(new $docset(), $this))->$action();
19
$this->info(Str::ucfirst("$action finished"));
20
21
return;
22
}
23
24
$this->warn('The doc requested does not seem to be supported.');
It seems like $this->argument('doc') can also be of type string[]; however, parameter $value of Illuminate\Support\Str::camel() does only seem to accept string, 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
godbout /
dash-docset-builder
Loading history...
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.