AddNamedArgumentsRector::getMethodArgs() - Code Metrics - Inspection of "fix cs" - savinmikhail/AddNamedArgumentsRector - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 96ec27...429a5a )

by mikhail

created 2025-02-16 12:45 UTC

AddNamedArgumentsRector::getMethodArgs() A

↳ Parent: AddNamedArgumentsRector

Complexity

Conditions	4
Paths	4

Size

Total Lines	25
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	4.7031

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	17
c	1
b	0
f	0
nc	4
nop	1
dl	0
loc	25
ccs	11
cts	17
cp	0.6471
crap	4.7031
rs	9.7

<?php

declare(strict_types=1);

namespace SavinMikhail\AddNamedArgumentsRector;

use InvalidArgumentException;
use PhpParser\Node;
use PhpParser\Node\Expr\FuncCall;
use PhpParser\Node\Expr\MethodCall;
use PhpParser\Node\Expr\New_;
use PhpParser\Node\Expr\StaticCall;
use PhpParser\Node\Identifier;
use PhpParser\Node\Name;
use PHPStan\Reflection\ClassMemberAccessAnswerer;
filter:
    dependency_paths: ["lib/*"]
use PHPStan\Reflection\ExtendedParameterReflection;
filter:
    dependency_paths: ["lib/*"]
use PHPStan\Reflection\ReflectionProvider;
filter:
    dependency_paths: ["lib/*"]
use PHPStan\ShouldNotHappenException;
filter:
    dependency_paths: ["lib/*"]
use Rector\Contract\Rector\ConfigurableRectorInterface;
use Rector\NodeTypeResolver\Node\AttributeKey;
use Rector\Rector\AbstractRector;
use Rector\ValueObject\PhpVersion;
use Rector\VersionBonding\Contract\MinPhpVersionInterface;
use SavinMikhail\AddNamedArgumentsRector\Config\ConfigStrategy;
use SavinMikhail\AddNamedArgumentsRector\Config\DefaultStrategy;
filter:
    dependency_paths: ["lib/*"]
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
use Webmozart\Assert\Assert;

use function count;

/**
 * @see AddNamedArgumentsRectorTest
 */
final class AddNamedArgumentsRector extends AbstractRector implements MinPhpVersionInterface, ConfigurableRectorInterface
{
    private string $configStrategy = DefaultStrategy::class;

    public function __construct(
        private readonly ReflectionProvider $reflectionProvider,
    ) {}

    public function getRuleDefinition(): RuleDefinition
    {
        return new RuleDefinition('Convert all arguments to named arguments', codeSamples: [
            new CodeSample(
                badCode: '$user->setPassword("123456");',
                goodCode: '$user->changePassword(password: "123456");',
            ),
        ]);
    }

    public function getNodeTypes(): array
    {
        return [FuncCall::class, StaticCall::class, MethodCall::class, New_::class];
    }

    public function refactor(Node $node): ?Node
    {
        $parameters = $this->getParameters($node);

        if (!$this->configStrategy::shouldApply($node, $parameters)) {
            return null;
        }

        /** @var FuncCall|StaticCall|MethodCall|New_ $node */
        $hasChanged = $this->addNamesToArgs($node, $parameters);

        return $hasChanged ? $node : null;
    }

    /**
     * @return ExtendedParameterReflection[]
     */
    private function getParameters(Node $node): array
    {
        $parameters = [];

        if ($node instanceof New_) {
            $parameters = $this->getConstructorArgs($node);
        } elseif ($node instanceof MethodCall) {
            $parameters = $this->getMethodArgs($node);
        } elseif ($node instanceof StaticCall) {
            $parameters = $this->getStaticMethodArgs($node);
        } elseif ($node instanceof FuncCall) {
            $parameters = $this->getFuncArgs($node);
        }

        return $parameters;
    }

    /**
     * @return ExtendedParameterReflection[]
     */
    private function getStaticMethodArgs(StaticCall $node): array
    {
        if (! $node->class instanceof Name) {
            return [];
        }

        $className = $this->getName($node->class);
        if (! $this->reflectionProvider->hasClass($className)) {
            return [];
        }

        $classReflection = $this->reflectionProvider->getClass($className);

        if ($node->name instanceof Identifier) {
            $methodName = $node->name->name;
        } elseif ($node->name instanceof Name) {

            $methodName = (string) $node->name;
        } else {
            return [];
        }

        if (! $classReflection->hasMethod($methodName)) {
            return [];
        }

        /** @var ClassMemberAccessAnswerer $scope */
        $scope = $node->getAttribute(AttributeKey::SCOPE);
        $reflection = $classReflection->getMethod($methodName, $scope);

        try {
            return $reflection
                ->getOnlyVariant()
                ->getParameters();
        } catch (ShouldNotHappenException) {
            return $reflection
                ->getVariants()[0]
                ->getParameters();
        }
    }

    /**
     * @return ExtendedParameterReflection[]
     */
    private function getMethodArgs(MethodCall $node): array
    {
        $callerType = $this->nodeTypeResolver->getType($node->var);
        $name = $node->name;
        if ($name instanceof Node\Expr) {
            return [];
        }
        $methodName = $name->name;

        if (! $callerType->hasMethod($methodName)->yes()) {
            return [];
        }

        /** @var ClassMemberAccessAnswerer $scope */
        $scope = $node->getAttribute(AttributeKey::SCOPE);
        $reflection = $callerType->getMethod($methodName, $scope);

        try {
            return $reflection
                ->getOnlyVariant()
                ->getParameters();
        } catch (ShouldNotHappenException) {
            return $reflection
                ->getVariants()[0]
                ->getParameters();
        }
    }

    private function resolveCalledName(Node $node): ?string
    {
        if ($node instanceof FuncCall && $node->name instanceof Name) {
            return (string) $node->name;
        }

        if ($node instanceof MethodCall && $node->name instanceof Identifier) {
            return (string) $node->name;
        }

        if ($node instanceof StaticCall && $node->name instanceof Identifier) {
            return (string) $node->name;
        }

        if ($node instanceof New_ && $node->class instanceof Name) {
            return (string) $node->class;
        }

        return null;
    }

    /**
     * @return ExtendedParameterReflection[]
     */
    private function getConstructorArgs(New_ $node): array
    {
        $calledName = $this->resolveCalledName($node);
        if ($calledName === null) {
            return [];
        }

        if (! $this->reflectionProvider->hasClass($calledName)) {
            return [];
        }
        $classReflection = $this->reflectionProvider->getClass($calledName);

        if (! $classReflection->hasConstructor()) {
            return [];
        }

        $reflection = $classReflection->getConstructor();

        try {
            return $reflection
                ->getOnlyVariant()
                ->getParameters();
        } catch (ShouldNotHappenException) {
            return $reflection
                ->getVariants()[0]
                ->getParameters();
        }
    }

    /**
     * @return ExtendedParameterReflection[]
     */
    private function getFuncArgs(FuncCall $node): array
    {
        $calledName = $this->resolveCalledName($node);
        if ($calledName === null) {
            return [];
        }

        $scope = $node->getAttribute(AttributeKey::SCOPE);

        if (! $this->reflectionProvider->hasFunction(new Name($calledName), $scope)) {
            return [];
        }
        $reflection = $this->reflectionProvider->getFunction(new Name($calledName), $scope);

        try {
            return $reflection
                ->getOnlyVariant()
                ->getParameters();
        } catch (ShouldNotHappenException) {
            return $reflection
                ->getVariants()[0]
                ->getParameters();
        }
    }

    /**
     * @param ExtendedParameterReflection[] $parameters
     */
    private function addNamesToArgs(
        FuncCall|StaticCall|MethodCall|New_ $node,
        array $parameters,
    ): bool {
        $argNames = [];
        foreach ($node->args as $index => $arg) {
            if (! isset($parameters[$index])) {
                return false;
            }

            // Skip variadic parameters (...$param)
            if ($parameters[$index]->isVariadic()) {
                return false;
            }

            // Skip unpacking arguments (...$var)
            if ($arg instanceof Node\Arg && $arg->unpack) {
                return false;
            }

            if ($arg instanceof Node\VariadicPlaceholder) {
                return false;
            }

            $argNames[$index] = new Identifier($parameters[$index]->getName());
        }

        foreach ($node->args as $index => $arg) {
            $arg->name = $argNames[$index];

        }

        return true;
    }

    public function provideMinPhpVersion(): int
    {
        return PhpVersion::PHP_80;
    }

    public function configure(array $configuration): void
    {
        Assert::lessThan(count($configuration), 2, 'You can pass only 1 strategy');
        if ($configuration === []) {
            return;
        }
        $strategyClass = $configuration[0];

        if (!class_exists($strategyClass)) {
            throw new InvalidArgumentException("Class {$strategyClass} does not exist.");
        }

        $strategy = new $strategyClass();

        Assert::isInstanceOf($strategy, ConfigStrategy::class, 'Your strategy must implement ConfigStrategy interface');

        $this->configStrategy = $strategyClass;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace SavinMikhail\AddNamedArgumentsRector;
6
7		use InvalidArgumentException;
8		use PhpParser\Node;
9		use PhpParser\Node\Expr\FuncCall;
10		use PhpParser\Node\Expr\MethodCall;
11		use PhpParser\Node\Expr\New_;
12		use PhpParser\Node\Expr\StaticCall;
13		use PhpParser\Node\Identifier;
14		use PhpParser\Node\Name;
15		use PHPStan\Reflection\ClassMemberAccessAnswerer;
		0 ignored issues – show Bug introduced 2025-01-21 03:50 UTC by Report Bug Copy Issue Report The type `PHPStan\Reflection\ClassMemberAccessAnswerer` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
16		use PHPStan\Reflection\ExtendedParameterReflection;
		0 ignored issues – show Bug introduced 2025-01-21 03:50 UTC by Report Bug Copy Issue Report The type `PHPStan\Reflection\ExtendedParameterReflection` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
17		use PHPStan\Reflection\ReflectionProvider;
		0 ignored issues – show Bug introduced 2025-01-21 03:50 UTC by Report Bug Copy Issue Report The type `PHPStan\Reflection\ReflectionProvider` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
18		use PHPStan\ShouldNotHappenException;
		0 ignored issues – show Bug introduced 2025-02-16 12:48 UTC by Report Bug Copy Issue Report The type `PHPStan\ShouldNotHappenException` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
19		use Rector\Contract\Rector\ConfigurableRectorInterface;
20		use Rector\NodeTypeResolver\Node\AttributeKey;
21		use Rector\Rector\AbstractRector;
22		use Rector\ValueObject\PhpVersion;
23		use Rector\VersionBonding\Contract\MinPhpVersionInterface;
24		use SavinMikhail\AddNamedArgumentsRector\Config\ConfigStrategy;
25		use SavinMikhail\AddNamedArgumentsRector\Config\DefaultStrategy;
		0 ignored issues – show Bug introduced 2025-01-21 03:50 UTC by Report Bug Copy Issue Report The type `SavinMikhail\AddNamedArg...\Config\DefaultStrategy` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
26		use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
27		use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
28		use Webmozart\Assert\Assert;
29
30		use function count;
31
32		/**
33		* @see AddNamedArgumentsRectorTest
34		*/
35		final class AddNamedArgumentsRector extends AbstractRector implements MinPhpVersionInterface, ConfigurableRectorInterface
36		{
37		private string $configStrategy = DefaultStrategy::class;
38
39	2	public function __construct(
40		private readonly ReflectionProvider $reflectionProvider,
41	2	) {}
42
43		public function getRuleDefinition(): RuleDefinition
44		{
45		return new RuleDefinition('Convert all arguments to named arguments', codeSamples: [
46		new CodeSample(
47		badCode: '$user->setPassword("123456");',
48		goodCode: '$user->changePassword(password: "123456");',
49		),
50		]);
51		}
52
53	9	public function getNodeTypes(): array
54		{
55	9	return [FuncCall::class, StaticCall::class, MethodCall::class, New_::class];
56		}
57
58	9	public function refactor(Node $node): ?Node
59		{
60	9	$parameters = $this->getParameters($node);
61
62	9	if (!$this->configStrategy::shouldApply($node, $parameters)) {
63	1	return null;
64		}
65
66		/** @var FuncCall\|StaticCall\|MethodCall\|New_ $node */
67	9	$hasChanged = $this->addNamesToArgs($node, $parameters);
68
69	9	return $hasChanged ? $node : null;
70		}
71
72		/**
73		* @return ExtendedParameterReflection[]
74		*/
75	9	private function getParameters(Node $node): array
76		{
77	9	$parameters = [];
78
79	9	if ($node instanceof New_) {
80	3	$parameters = $this->getConstructorArgs($node);
81	7	} elseif ($node instanceof MethodCall) {
82	1	$parameters = $this->getMethodArgs($node);
83	6	} elseif ($node instanceof StaticCall) {
84	1	$parameters = $this->getStaticMethodArgs($node);
85	5	} elseif ($node instanceof FuncCall) {
86	5	$parameters = $this->getFuncArgs($node);
87		}
88
89	9	return $parameters;
90		}
91
92		/**
93		* @return ExtendedParameterReflection[]
94		*/
95	1	private function getStaticMethodArgs(StaticCall $node): array
96		{
97	1	if (! $node->class instanceof Name) {
98		return [];
99		}
100
101	1	$className = $this->getName($node->class);
102	1	if (! $this->reflectionProvider->hasClass($className)) {
103		return [];
104		}
105
106	1	$classReflection = $this->reflectionProvider->getClass($className);
107
108	1	if ($node->name instanceof Identifier) {
109	1	$methodName = $node->name->name;
110		} elseif ($node->name instanceof Name) {
		0 ignored issues – show introduced 2025-01-21 03:50 UTC by Report Bug Copy Issue Report `$node->name` is never a sub-type of `PhpParser\Node\Name`. Loading history...
111		$methodName = (string) $node->name;
112		} else {
113		return [];
114		}
115
116	1	if (! $classReflection->hasMethod($methodName)) {
117		return [];
118		}
119
120		/** @var ClassMemberAccessAnswerer $scope */
121	1	$scope = $node->getAttribute(AttributeKey::SCOPE);
122	1	$reflection = $classReflection->getMethod($methodName, $scope);
123
124		try {
125	1	return $reflection
126	1	->getOnlyVariant()
127	1	->getParameters();
128		} catch (ShouldNotHappenException) {
129		return $reflection
130		->getVariants()[0]
131		->getParameters();
132		}
133		}
134
135		/**
136		* @return ExtendedParameterReflection[]
137		*/
138	1	private function getMethodArgs(MethodCall $node): array
139		{
140	1	$callerType = $this->nodeTypeResolver->getType($node->var);
141	1	$name = $node->name;
142	1	if ($name instanceof Node\Expr) {
143		return [];
144		}
145	1	$methodName = $name->name;
146
147	1	if (! $callerType->hasMethod($methodName)->yes()) {
148		return [];
149		}
150
151		/** @var ClassMemberAccessAnswerer $scope */
152	1	$scope = $node->getAttribute(AttributeKey::SCOPE);
153	1	$reflection = $callerType->getMethod($methodName, $scope);
154
155		try {
156	1	return $reflection
157	1	->getOnlyVariant()
158	1	->getParameters();
159		} catch (ShouldNotHappenException) {
160		return $reflection
161		->getVariants()[0]
162		->getParameters();
163		}
164		}
165
166	8	private function resolveCalledName(Node $node): ?string
167		{
168	8	if ($node instanceof FuncCall && $node->name instanceof Name) {
169	5	return (string) $node->name;
170		}
171
172	3	if ($node instanceof MethodCall && $node->name instanceof Identifier) {
173		return (string) $node->name;
174		}
175
176	3	if ($node instanceof StaticCall && $node->name instanceof Identifier) {
177		return (string) $node->name;
178		}
179
180	3	if ($node instanceof New_ && $node->class instanceof Name) {
181	3	return (string) $node->class;
182		}
183
184		return null;
185		}
186
187		/**
188		* @return ExtendedParameterReflection[]
189		*/
190	3	private function getConstructorArgs(New_ $node): array
191		{
192	3	$calledName = $this->resolveCalledName($node);
193	3	if ($calledName === null) {
194		return [];
195		}
196
197	3	if (! $this->reflectionProvider->hasClass($calledName)) {
198		return [];
199		}
200	3	$classReflection = $this->reflectionProvider->getClass($calledName);
201
202	3	if (! $classReflection->hasConstructor()) {
203		return [];
204		}
205
206	3	$reflection = $classReflection->getConstructor();
207
208		try {
209	3	return $reflection
210	3	->getOnlyVariant()
211	3	->getParameters();
212	1	} catch (ShouldNotHappenException) {
213	1	return $reflection
214	1	->getVariants()[0]
215	1	->getParameters();
216		}
217		}
218
219		/**
220		* @return ExtendedParameterReflection[]
221		*/
222	5	private function getFuncArgs(FuncCall $node): array
223		{
224	5	$calledName = $this->resolveCalledName($node);
225	5	if ($calledName === null) {
226		return [];
227		}
228
229	5	$scope = $node->getAttribute(AttributeKey::SCOPE);
230
231	5	if (! $this->reflectionProvider->hasFunction(new Name($calledName), $scope)) {
232		return [];
233		}
234	5	$reflection = $this->reflectionProvider->getFunction(new Name($calledName), $scope);
235
236		try {
237	5	return $reflection
238	5	->getOnlyVariant()
239	5	->getParameters();
240		} catch (ShouldNotHappenException) {
241		return $reflection
242		->getVariants()[0]
243		->getParameters();
244		}
245		}
246
247		/**
248		* @param ExtendedParameterReflection[] $parameters
249		*/
250	9	private function addNamesToArgs(
251		FuncCall\|StaticCall\|MethodCall\|New_ $node,
252		array $parameters,
253		): bool {
254	9	$argNames = [];
255	9	foreach ($node->args as $index => $arg) {
256	9	if (! isset($parameters[$index])) {
257		return false;
258		}
259
260		// Skip variadic parameters (...$param)
261	9	if ($parameters[$index]->isVariadic()) {
262	1	return false;
263		}
264
265		// Skip unpacking arguments (...$var)
266	9	if ($arg instanceof Node\Arg && $arg->unpack) {
267	1	return false;
268		}
269
270	8	if ($arg instanceof Node\VariadicPlaceholder) {
271		return false;
272		}
273
274	8	$argNames[$index] = new Identifier($parameters[$index]->getName());
275		}
276
277	7	foreach ($node->args as $index => $arg) {
278	7	$arg->name = $argNames[$index];
		0 ignored issues – show Bug introduced 2025-02-16 05:21 UTC by Report Bug Copy Issue Report The property `name` does not seem to exist on `PhpParser\Node\VariadicPlaceholder`. Loading history...
279		}
280
281	7	return true;
282		}
283
284	2	public function provideMinPhpVersion(): int
285		{
286	2	return PhpVersion::PHP_80;
287		}
288
289	1	public function configure(array $configuration): void
290		{
291	1	Assert::lessThan(count($configuration), 2, 'You can pass only 1 strategy');
292	1	if ($configuration === []) {
293		return;
294		}
295	1	$strategyClass = $configuration[0];
296
297	1	if (!class_exists($strategyClass)) {
298		throw new InvalidArgumentException("Class {$strategyClass} does not exist.");
299		}
300
301	1	$strategy = new $strategyClass();
302
303	1	Assert::isInstanceOf($strategy, ConfigStrategy::class, 'Your strategy must implement ConfigStrategy interface');
304
305	1	$this->configStrategy = $strategyClass;
306		}
307		}
308

savinmikhail / AddNamedArgumentsRector

Push — main ( 96ec27...429a5a )

AddNamedArgumentsRector::getMethodArgs() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like