Resolver::autowireArguments() - Code Metrics - Inspection of "Fixed error with unable to pass specified argument..." - biurad/php-dependency-injection - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 059934...f33627 )

by Divine Niiquaye

created 2020-12-13 13:53 UTC

Resolver::autowireArguments() B

↳ Parent: Resolver

Complexity

Conditions	10
Paths	76

Size

Total Lines	38
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	10
eloc	22
c	1
b	0
f	0
nc	76
nop	3
dl	0
loc	38
rs	7.6666

How to fix Complexity

<?php

declare(strict_types=1);

/*
 * This file is part of Biurad opensource projects.
 *
 * PHP version 7.2 and above required
 *
 * @author    Divine Niiquaye Ibok <[email protected]>
 * @copyright 2019 Biurad Group (https://biurad.com/)
 * @license   https://opensource.org/licenses/BSD-3-Clause License
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Biurad\DependencyInjection;

use Nette\DI\Resolver as NetteResolver;
use Nette\Utils\Reflection;
use ReflectionFunctionAbstract;

/**
 * Services resolver
 *
 * @internal
 */
class Resolver extends NetteResolver
{
    /**
     * Add missing arguments using autowiring.
     *
     * @param ReflectionFunctionAbstract $method
     * @param mixed[] $arguments
     * @param  (callable(string $type, bool $single): object|object[]|null) $getter

     *
     * @return mixed[]
     */
    public static function autowireArguments(
        ReflectionFunctionAbstract $method,
        array $arguments,
        callable $getter
    ): array {
        $optCount = 0;
        $num = -1;
        $res = [];

        foreach ($method->getParameters() as $num => $param) {
            $paramName = $param->name;
            if (!$param->isVariadic() && array_key_exists($paramName, $arguments)) {
                $res[$num] = $arguments[$paramName];
                unset($arguments[$paramName], $arguments[$num]);
            } elseif (array_key_exists($num, $arguments)) {
                $res[$num] = $arguments[$num];
                unset($arguments[$num]);
            } else {
                $res[$num] = self::autowireArgument($param, $getter);
            }

            $optCount = $param->isOptional() && $res[$num] === ($param->isDefaultValueAvailable() ? Reflection::getParameterDefaultValue($param) : null)
                ? $optCount + 1
                : 0;
        }

        // extra parameters
        while (array_key_exists(++$num, $arguments)) {
            $res[$num] = $arguments[$num];
            unset($arguments[$num]);
            $optCount = 0;
        }

        if ($optCount) {
            $res = array_slice($res, 0, -$optCount);
        }

        return $res;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of Biurad opensource projects.
7			*
8			* PHP version 7.2 and above required
9			*
10			* @author Divine Niiquaye Ibok <[email protected]>
11			* @copyright 2019 Biurad Group (https://biurad.com/)
12			* @license https://opensource.org/licenses/BSD-3-Clause License
13			*
14			* For the full copyright and license information, please view the LICENSE
15			* file that was distributed with this source code.
16			*/
17
18			namespace Biurad\DependencyInjection;
19
20			use Nette\DI\Resolver as NetteResolver;
21			use Nette\Utils\Reflection;
22			use ReflectionFunctionAbstract;
23
24			/**
25			* Services resolver
26			*
27			* @internal
28			*/
29			class Resolver extends NetteResolver
30			{
31			/**
32			* Add missing arguments using autowiring.
33			*
34			* @param ReflectionFunctionAbstract $method
35			* @param mixed[] $arguments
36			* @param (callable(string $type, bool $single): object\|object[]\|null) $getter
			0 ignored issues – show Documentation Bug introduced 2020-12-13 13:55 UTC by Report Bug Copy Issue Report The doc comment `(callable(string` at position `1` could not be parsed: Expected ')' at position 1, but found 'callable'. Loading history...
37			*
38			* @return mixed[]
39			*/
40			public static function autowireArguments(
41			ReflectionFunctionAbstract $method,
42			array $arguments,
43			callable $getter
44			): array {
45			$optCount = 0;
46			$num = -1;
47			$res = [];
48
49			foreach ($method->getParameters() as $num => $param) {
50			$paramName = $param->name;
51			if (!$param->isVariadic() && array_key_exists($paramName, $arguments)) {
52			$res[$num] = $arguments[$paramName];
53			unset($arguments[$paramName], $arguments[$num]);
54			} elseif (array_key_exists($num, $arguments)) {
55			$res[$num] = $arguments[$num];
56			unset($arguments[$num]);
57			} else {
58			$res[$num] = self::autowireArgument($param, $getter);
59			}
60
61			$optCount = $param->isOptional() && $res[$num] === ($param->isDefaultValueAvailable() ? Reflection::getParameterDefaultValue($param) : null)
62			? $optCount + 1
63			: 0;
64			}
65
66			// extra parameters
67			while (array_key_exists(++$num, $arguments)) {
68			$res[$num] = $arguments[$num];
69			unset($arguments[$num]);
70			$optCount = 0;
71			}
72
73			if ($optCount) {
74			$res = array_slice($res, 0, -$optCount);
75			}
76
77			return $res;
78			}
79			}
80

biurad / php-dependency-injection

Push — master ( 059934...f33627 )

Resolver::autowireArguments() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like