The call to BeyondCode\LaravelFavico...shouldGenerateFavicon() has too many arguments starting with app()->environment().
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the ignore-call annotation
8
if (app(FaviconGenerator::class)->/** @scrutinizer ignore-call */ shouldGenerateFavicon(app()->environment())) {
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.
The method environment() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the ignore-call annotation
8
if (app(FaviconGenerator::class)->shouldGenerateFavicon(app()->/** @scrutinizer ignore-call */ environment())) {
beyondcode /
laravel-favicon
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.