‪TYPO3CMS  10.4
MethodArgumentUnusedMatcher.php
Go to the documentation of this file.
1 <?php
2 
3 declare(strict_types=1);
4 
5 /*
6  * This file is part of the TYPO3 CMS project.
7  *
8  * It is free software; you can redistribute it and/or modify it under
9  * the terms of the GNU General Public License, either version 2
10  * of the License, or any later version.
11  *
12  * For the full copyright and license information, please read the
13  * LICENSE.txt file that was distributed with this source code.
14  *
15  * The TYPO3 project - inspiring people to share!
16  */
17 
19 
20 use PhpParser\Node;
21 use PhpParser\Node\Expr\ConstFetch;
22 use PhpParser\Node\Expr\MethodCall;
23 
33 {
39  public function ‪__construct(array ‪$matcherDefinitions)
40  {
41  $this->matcherDefinitions = ‪$matcherDefinitions;
42  $this->‪validateMatcherDefinitions(['unusedArgumentNumbers']);
44  }
45 
51  public function ‪enterNode(Node $node)
52  {
53  // Match method call (not static)
54  if (!$this->‪isFileIgnored($node)
55  && !$this->‪isLineIgnored($node)
56  && $node instanceof MethodCall
57  && array_key_exists($node->name->name, $this->flatMatcherDefinitions)
58  ) {
59  $match = [
60  'restFiles' => [],
61  'line' => $node->getAttribute('startLine'),
62  'indicator' => 'weak',
63  ];
64 
65  $isArgumentUnpackingUsed = $this->‪isArgumentUnpackingUsed($node->args);
66 
67  $numberOfArguments = count($node->args);
68  $isPossibleMatch = false;
69  foreach ($this->flatMatcherDefinitions[$node->name->name]['candidates'] as $candidate) {
70  foreach ($candidate['unusedArgumentNumbers'] as $droppedArgumentNumber) {
71  // A method call is considered a match if name matches, unpacking is not used
72  // and the registered argument is not given as null.
73  if (!$isArgumentUnpackingUsed
74  && $numberOfArguments >= $droppedArgumentNumber
75  && !($node->args[$droppedArgumentNumber - 1]->value instanceof ConstFetch)
76  && (!isset($node->args[$droppedArgumentNumber - 1]->value->name->name->parts[0])
77  || $node->args[$droppedArgumentNumber - 1]->value->name->name->parts[0] !== null)
78  ) {
79  $isPossibleMatch = true;
80  $match['message'] = 'Call to method "' . $node->name->name . '()" with'
81  . ' argument ' . $droppedArgumentNumber . ' not given as null.';
82  $match['restFiles'] = array_unique(array_merge($match['restFiles'], $candidate['restFiles']));
83  }
84  }
85  }
86  if ($isPossibleMatch) {
87  $this->matches[] = $match;
88  }
89  }
90  }
91 }
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher\isArgumentUnpackingUsed
‪bool isArgumentUnpackingUsed(array $arguments=[])
Definition: AbstractCoreMatcher.php:164
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\MethodArgumentUnusedMatcher\enterNode
‪enterNode(Node $node)
Definition: MethodArgumentUnusedMatcher.php:51
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher
Definition: AbstractCoreMatcher.php:18
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\MethodArgumentUnusedMatcher\__construct
‪__construct(array $matcherDefinitions)
Definition: MethodArgumentUnusedMatcher.php:39
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher\isLineIgnored
‪bool isLineIgnored(Node $node)
Definition: AbstractCoreMatcher.php:181
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher
Definition: AbstractCoreMatcher.php:34
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher\validateMatcherDefinitions
‪validateMatcherDefinitions(array $requiredArrayKeys=[])
Definition: AbstractCoreMatcher.php:88
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher\isFileIgnored
‪bool isFileIgnored(Node $node)
Definition: AbstractCoreMatcher.php:214
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher\$matcherDefinitions
‪array $matcherDefinitions
Definition: AbstractCoreMatcher.php:41
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\MethodArgumentUnusedMatcher
Definition: MethodArgumentUnusedMatcher.php:33
‪TYPO3\CMS\Install\ExtensionScanner\Php\Matcher\AbstractCoreMatcher\initializeFlatMatcherDefinitions
‪initializeFlatMatcherDefinitions()
Definition: AbstractCoreMatcher.php:134