[PHPStanRules] Make NoNestedFuncCallRule work for nested expressions (#3145)

kaizen-ci · kaizen-ci · commit c449a9d4d25b · 2021-04-26T20:55:18.000Z
diff --git a/src/Console/Reporter/CheckerListReporter.php b/src/Console/Reporter/CheckerListReporter.php
@@ -21,16 +21,16 @@ public function __construct(SymfonyStyle $symfonyStyle)
     }
 
     /**
-     * @param FixerInterface[]|Sniff[]|string[] $checkers
+     * @param FixerInterface[]|Sniff[] $checkers
      */
     public function report(array $checkers, string $type): void
     {
         if ($checkers === []) {
             return;
         }
 
-        $checkerNames = array_map(function ($fixer): string {
-            return is_string($fixer) ? $fixer : get_class($fixer);
+        $checkerNames = array_map(function ($checker): string {
+            return get_class($checker);
         }, $checkers);
 
         $sectionMessage = sprintf('%d checker%s from %s:', count($checkers), count($checkers) === 1 ? '' : 's', $type);

Original file line number	Diff line number	Diff line change
`@@ -21,16 +21,16 @@ public function __construct(SymfonyStyle $symfonyStyle)`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`/**`
`24`		`- * @param FixerInterface[]\|Sniff[]\|string[] $checkers`
	`24`	`+ * @param FixerInterface[]\|Sniff[] $checkers`
`25`	`25`	`*/`
`26`	`26`	`public function report(array $checkers, string $type): void`
`27`	`27`	`{`
`28`	`28`	`if ($checkers === []) {`
`29`	`29`	`return;`
`30`	`30`	`}`
`31`	`31`
`32`		`- $checkerNames = array_map(function ($fixer): string {`
`33`		`- return is_string($fixer) ? $fixer : get_class($fixer);`
	`32`	`+ $checkerNames = array_map(function ($checker): string {`
	`33`	`+ return get_class($checker);`
`34`	`34`	`}, $checkers);`
`35`	`35`
`36`	`36`	`$sectionMessage = sprintf('%d checker%s from %s:', count($checkers), count($checkers) === 1 ? '' : 's', $type);`