Merge pull request #9964 from ygottschalk/fix/phpparser-deprecation-n…

…ame-parts Fix/phpparser deprecation name parts
vimeo · Jul 3, 2023 · 3e54feb · 3e54feb
2 parents bc46938 + bfd3cb8
commit 3e54feb
Show file tree

Hide file tree

Showing 39 changed files with 196 additions and 187 deletions.
diff --git a/composer.json b/composer.json
@@ -23,17 +23,17 @@
         "ext-libxml": "*",
         "ext-mbstring": "*",
         "ext-tokenizer": "*",
+        "composer-runtime-api": "^2",
         "amphp/amp": "^2.4.2",
         "amphp/byte-stream": "^1.5",
-        "composer-runtime-api": "^2",
         "composer/semver": "^1.4 || ^2.0 || ^3.0",
         "composer/xdebug-handler": "^2.0 || ^3.0",
         "dnoegel/php-xdg-base-dir": "^0.1.1",
         "felixfbecker/advanced-json-rpc": "^3.1",
         "felixfbecker/language-server-protocol": "^1.5.2",
         "fidry/cpu-core-counter": "^0.4.1 || ^0.5.1",
         "netresearch/jsonmapper": "^1.0 || ^2.0 || ^3.0 || ^4.0",
-        "nikic/php-parser": "^4.14",
+        "nikic/php-parser": "^4.16",
         "sebastian/diff": "^4.0 || ^5.0",
         "spatie/array-to-xml": "^2.17.0 || ^3.0",
         "symfony/console": "^4.1.6 || ^5.0 || ^6.0",

diff --git a/examples/plugins/FunctionCasingChecker.php b/examples/plugins/FunctionCasingChecker.php
@@ -84,7 +84,7 @@ public static function afterFunctionCallAnalysis(AfterFunctionCallAnalysisEvent
 
             $function_name_parts = explode('\\', $function_storage->cased_name);
 
-            if (end($function_name_parts) !== end($expr->name->parts)) {
+            if (end($function_name_parts) !== $expr->name->getLast()) {
                 IssueBuffer::maybeAdd(
                     new IncorrectFunctionCasing(
                         'Function is incorrectly cased, expecting ' . $function_storage->cased_name,

diff --git a/examples/plugins/StringChecker.php b/examples/plugins/StringChecker.php
@@ -50,7 +50,7 @@ public static function afterExpressionAnalysis(AfterExpressionAnalysisEvent $eve
             && $expr->left->class instanceof PhpParser\Node\Name
             && $expr->left->name instanceof PhpParser\Node\Identifier
             && strtolower($expr->left->name->name) === 'class'
-            && !in_array(strtolower($expr->left->class->parts[0]), ['self', 'static', 'parent'])
+            && !in_array(strtolower($expr->left->class->getFirst()), ['self', 'static', 'parent'])
             && $expr->right instanceof PhpParser\Node\Scalar\String_
             && preg_match('/^::[A-Za-z0-9]+$/', $expr->right->value)
         ) {

diff --git a/psalm.xml.dist b/psalm.xml.dist
@@ -93,6 +93,7 @@
         <MissingConstructor>
             <errorLevel type="suppress">
                 <directory name="tests"/>
+                <file name="vendor/nikic/php-parser/lib/PhpParser/Node/Name.php" />
             </errorLevel>
         </MissingConstructor>
 

diff --git a/src/Psalm/Internal/Analyzer/CanAlias.php b/src/Psalm/Internal/Analyzer/CanAlias.php
@@ -8,7 +8,6 @@
 use Psalm\FileManipulation;
 use Psalm\Internal\FileManipulation\FileManipulationBuffer;
 
-use function implode;
 use function strtolower;
 
 /**
@@ -51,7 +50,7 @@ public function visitUse(PhpParser\Node\Stmt\Use_ $stmt): void
         $codebase = $this->getCodebase();
 
         foreach ($stmt->uses as $use) {
-            $use_path = implode('\\', $use->name->parts);
+            $use_path = $use->name->toString();
             $use_path_lc = strtolower($use_path);
             $use_alias = $use->alias->name ?? $use->name->getLast();
             $use_alias_lc = strtolower($use_alias);
@@ -106,12 +105,12 @@ public function visitUse(PhpParser\Node\Stmt\Use_ $stmt): void
 
     public function visitGroupUse(PhpParser\Node\Stmt\GroupUse $stmt): void
     {
-        $use_prefix = implode('\\', $stmt->prefix->parts);
+        $use_prefix = $stmt->prefix->toString();
 
         $codebase = $this->getCodebase();
 
         foreach ($stmt->uses as $use) {
-            $use_path = $use_prefix . '\\' . implode('\\', $use->name->parts);
+            $use_path = $use_prefix . '\\' . $use->name->toString();
             $use_alias = $use->alias->name ?? $use->name->getLast();
 
             switch ($use->type !== PhpParser\Node\Stmt\Use_::TYPE_UNKNOWN ? $use->type : $stmt->type) {

diff --git a/src/Psalm/Internal/Analyzer/ClassAnalyzer.php b/src/Psalm/Internal/Analyzer/ClassAnalyzer.php
@@ -2021,7 +2021,7 @@ private function checkImplementedInterfaces(
                         . ($interface_name instanceof PhpParser\Node\Name\FullyQualified
                             ? '\\'
                             : $this->getNamespace() . '-')
-                        . implode('\\', $interface_name->parts),
+                        . $interface_name->toString(),
             );
 
             $interface_location = new CodeLocation($this, $interface_name);
@@ -2437,7 +2437,7 @@ private function checkParentClass(
                             . ($extended_class instanceof PhpParser\Node\Name\FullyQualified
                                 ? '\\'
                                 : $this->getNamespace() . '-')
-                            . implode('\\', $extended_class->parts),
+                            . $extended_class->toString(),
                 );
             }
 

diff --git a/src/Psalm/Internal/Analyzer/ClassLikeAnalyzer.php b/src/Psalm/Internal/Analyzer/ClassLikeAnalyzer.php
@@ -40,7 +40,6 @@
 use function count;
 use function explode;
 use function gettype;
-use function implode;
 use function in_array;
 use function preg_match;
 use function preg_replace;
@@ -421,15 +420,15 @@ public static function getFQCLNFromNameObject(
         }
 
         if ($class_name instanceof PhpParser\Node\Name\FullyQualified) {
-            return implode('\\', $class_name->parts);
+            return $class_name->toString();
         }
 
-        if (in_array($class_name->parts[0], ['self', 'static', 'parent'], true)) {
-            return $class_name->parts[0];
+        if (in_array($class_name->getFirst(), ['self', 'static', 'parent'], true)) {
+            return $class_name->getFirst();
         }
 
         return Type::getFQCLNFromString(
-            implode('\\', $class_name->parts),
+            $class_name->toString(),
             $aliases,
         );
     }

diff --git a/src/Psalm/Internal/Analyzer/FileAnalyzer.php b/src/Psalm/Internal/Analyzer/FileAnalyzer.php
@@ -32,7 +32,6 @@
 use function array_diff_key;
 use function array_keys;
 use function count;
-use function implode;
 use function strpos;
 use function strtolower;
 
@@ -283,7 +282,7 @@ public function populateCheckers(array $stmts): array
             } elseif ($stmt instanceof PhpParser\Node\Stmt\ClassLike) {
                 $this->populateClassLikeAnalyzers($stmt);
             } elseif ($stmt instanceof PhpParser\Node\Stmt\Namespace_) {
-                $namespace_name = $stmt->name ? implode('\\', $stmt->name->parts) : '';
+                $namespace_name = $stmt->name ? $stmt->name->toString() : '';
 
                 $namespace_analyzer = new NamespaceAnalyzer($stmt, $this);
                 $namespace_analyzer->collectAnalyzableInformation();

diff --git a/src/Psalm/Internal/Analyzer/NamespaceAnalyzer.php b/src/Psalm/Internal/Analyzer/NamespaceAnalyzer.php
@@ -14,7 +14,6 @@
 
 use function assert;
 use function count;
-use function implode;
 use function is_string;
 use function preg_replace;
 use function strpos;
@@ -49,7 +48,7 @@ public function __construct(Namespace_ $namespace, FileAnalyzer $source)
     {
         $this->source = $source;
         $this->namespace = $namespace;
-        $this->namespace_name = $this->namespace->name ? implode('\\', $this->namespace->name->parts) : '';
+        $this->namespace_name = $this->namespace->name ? $this->namespace->name->toString() : '';
     }
 
     public function collectAnalyzableInformation(): void

diff --git a/src/Psalm/Internal/Analyzer/Statements/Block/IfConditionalAnalyzer.php b/src/Psalm/Internal/Analyzer/Statements/Block/IfConditionalAnalyzer.php
@@ -238,13 +238,13 @@ private static function getDefinitelyEvaluatedExpressionAfterIf(PhpParser\Node\E
             || $stmt instanceof PhpParser\Node\Expr\BinaryOp\Identical
         ) {
             if ($stmt->left instanceof PhpParser\Node\Expr\ConstFetch
-                && $stmt->left->name->parts === ['true']
+                && $stmt->left->name->getParts() === ['true']
             ) {
                 return self::getDefinitelyEvaluatedExpressionAfterIf($stmt->right);
             }
 
             if ($stmt->right instanceof PhpParser\Node\Expr\ConstFetch
-                && $stmt->right->name->parts === ['true']
+                && $stmt->right->name->getParts() === ['true']
             ) {
                 return self::getDefinitelyEvaluatedExpressionAfterIf($stmt->left);
             }
@@ -282,13 +282,13 @@ private static function getDefinitelyEvaluatedExpressionInsideIf(PhpParser\Node\
             || $stmt instanceof PhpParser\Node\Expr\BinaryOp\Identical
         ) {
             if ($stmt->left instanceof PhpParser\Node\Expr\ConstFetch
-                && $stmt->left->name->parts === ['true']
+                && $stmt->left->name->getParts() === ['true']
             ) {
                 return self::getDefinitelyEvaluatedExpressionInsideIf($stmt->right);
             }
 
             if ($stmt->right instanceof PhpParser\Node\Expr\ConstFetch
-                && $stmt->right->name->parts === ['true']
+                && $stmt->right->name->getParts() === ['true']
             ) {
                 return self::getDefinitelyEvaluatedExpressionInsideIf($stmt->left);
             }

diff --git a/src/Psalm/Internal/Analyzer/Statements/Block/SwitchCaseAnalyzer.php b/src/Psalm/Internal/Analyzer/Statements/Block/SwitchCaseAnalyzer.php
@@ -209,7 +209,7 @@ public static function analyze(
             }
 
             if ($switch_condition instanceof PhpParser\Node\Expr\ConstFetch
-                && $switch_condition->name->parts === ['true']
+                && $switch_condition->name->getParts() === ['true']
             ) {
                 $case_equality_expr = $case->cond;
             } elseif (($switch_condition_type = $statements_analyzer->node_data->getType($switch_condition))

diff --git a/src/Psalm/Internal/Analyzer/Statements/Block/WhileAnalyzer.php b/src/Psalm/Internal/Analyzer/Statements/Block/WhileAnalyzer.php
@@ -26,7 +26,8 @@ public static function analyze(
         PhpParser\Node\Stmt\While_ $stmt,
         Context $context
     ): ?bool {
-        $while_true = ($stmt->cond instanceof PhpParser\Node\Expr\ConstFetch && $stmt->cond->name->parts === ['true'])
+        $while_true = ($stmt->cond instanceof PhpParser\Node\Expr\ConstFetch
+                && $stmt->cond->name->getParts() === ['true'])
             || (($t = $statements_analyzer->node_data->getType($stmt->cond))
                 && $t->isAlwaysTruthy());