Issues (30)

src/functions/common.php (2 issues)

1
<?php
2
3
if (!function_exists('app')) {
4
    /**
5
     * Get the available container instance.
6
     *
7
     * @param string $make
8
     * @param array $parameters
9
     * @return mixed
10
     */
11
    function app($make = null, $parameters = [])
12
    {
13
        return \Nip\Container\Utility\Container::get($make, $parameters);
0 ignored issues
show
It seems like $make can also be of type string; however, parameter $make of Nip\Container\Utility\Container::get() does only seem to accept null, 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

13
        return \Nip\Container\Utility\Container::get(/** @scrutinizer ignore-type */ $make, $parameters);
Loading history...
The call to Nip\Container\Utility\Container::get() has too many arguments starting with $parameters. ( Ignorable by Annotation )

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

13
        return \Nip\Container\Utility\Container::/** @scrutinizer ignore-call */ get($make, $parameters);

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...
14
    }
15
}
16