Merge pull request #133 from goaop/feature/4.0-cleanup

New 4.0 version implementation for PHP>=8.2

Author: lisachenko

.github/workflows/phpunit.yml

@@ -17,6 +17,7 @@ jobs:
           - "highest"
         php-version:
           - "8.2"
+          - "8.3"
         operating-system:
           - "ubuntu-latest"
 

LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2015-2020 Alexander Lisachenko
+Copyright (c) 2015-2024 Alexander Lisachenko
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

composer.json

@@ -23,11 +23,13 @@
   },
   "require": {
     "php": ">=8.2",
-    "nikic/php-parser": "^4.0"
+    "nikic/php-parser": "^5.0"
   },
   "require-dev": {
     "phpunit/phpunit": "^10.5.8",
-    "tracy/tracy": "^2.10"
+    "tracy/tracy": "^2.10",
+    "rector/rector": "^1.0",
+    "rector/rector-php-parser": "^0.14.0"
   },
   "extra": {
     "branch-alias": {

phpunit.xml.dist

@@ -10,4 +10,7 @@
       <directory>./src/</directory>
     </include>
   </source>
+  <php>
+    <ini name="memory_limit" value="512M" />
+  </php>
 </phpunit>

rector.php

@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\DeadCode\Rector\ClassMethod\RemoveUselessParamTagRector;
+use Rector\DeadCode\Rector\ClassMethod\RemoveUselessReturnTagRector;
+use Rector\Php71\Rector\ClassConst\PublicConstantVisibilityRector;
+use Rector\Php74\Rector\Closure\ClosureToArrowFunctionRector;
+use Rector\PhpParser\Set\PhpParserSetList;
+use Rector\PHPUnit\Set\PHPUnitSetList;
+use Rector\TypeDeclaration\Rector\ClassMethod\AddVoidReturnTypeWhereNoReturnRector;
+
+return RectorConfig::configure()
+    ->withPaths([
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+    // uncomment to reach your current PHP version
+    // ->withPhpSets()
+    ->withRules([
+        AddVoidReturnTypeWhereNoReturnRector::class,
+        RemoveUselessParamTagRector::class,
+        RemoveUselessReturnTagRector::class,
+        PublicConstantVisibilityRector::class,
+        ClosureToArrowFunctionRector::class,
+    ])
+    ->withSets([
+        PHPUnitSetList::ANNOTATIONS_TO_ATTRIBUTES,
+        PHPUnitSetList::PHPUNIT_CODE_QUALITY,
+        PhpParserSetList::PHP_PARSER_50
+    ]);

src/Locator/CallableLocator.php

@@ -16,6 +16,7 @@
 
 /**
  * Locator, that can find a file for the given class name by asking composer
+ * @see \Go\ParserReflection\Locator\CallableLocatorTest
  */
 class CallableLocator implements LocatorInterface
 {
@@ -33,10 +34,8 @@ public function __construct(callable $callable)
      * Returns a path to the file for given class name
      *
      * @param string $className Name of the class
-     *
-     * @return string|false Path to the file with given class or false if not found
      */
-    public function locateClass(string $className)
+    public function locateClass(string $className): false|string
     {
         return call_user_func($this->callable, ltrim($className, '\\'));
     }

src/Locator/ComposerLocator.php

@@ -19,6 +19,7 @@
 
 /**
  * Locator, that can find a file for the given class name by asking composer
+ * @see \Go\ParserReflection\Locator\ComposerLocatorTest
  */
 class ComposerLocator implements LocatorInterface
 {
@@ -48,10 +49,8 @@ public function __construct(ClassLoader $composerLoader = null)
      * Returns a path to the file for given class name
      *
      * @param string $className Name of the class
-     *
-     * @return string|false Path to the file with given class or false if not found
-     */
-    public function locateClass(string $className)
+     **/
+    public function locateClass(string $className): false|string
     {
         $filePath = $this->loader->findFile(ltrim($className, '\\'));
         if (!empty($filePath)) {

src/LocatorInterface.php

@@ -21,8 +21,6 @@ interface LocatorInterface
      * Returns a path to the file for given class name
      *
      * @param string $className Name of the class (with or without leading '\' FQCN)
-     *
-     * @return string|false Path to the file with given class or false if not found
      */
-    public function locateClass(string $className);
+    public function locateClass(string $className): false|string;
 }

src/NodeVisitor/GeneratorDetector.php

@@ -13,15 +13,14 @@
 namespace Go\ParserReflection\NodeVisitor;
 
 use PhpParser\Node;
-use PhpParser\NodeTraverser;
 use PhpParser\NodeVisitorAbstract;
 
 /**
  * Visitor to check if the method body
  */
 class GeneratorDetector extends NodeVisitorAbstract
 {
-    private $isGenerator = false;
+    private bool $isGenerator = false;
 
     /**
      * {@inheritDoc}
@@ -30,7 +29,7 @@ public function enterNode(Node $node)
     {
         // There may be internal generators in closures, we do not need to look at them
         if ($node instanceof Node\Expr\Closure) {
-            return NodeTraverser::DONT_TRAVERSE_CHILDREN;
+            return self::DONT_TRAVERSE_CHILDREN;
         }
 
         if ($node instanceof Node\Expr\Yield_ || $node instanceof Node\Expr\YieldFrom) {

src/NodeVisitor/StaticVariablesCollector.php

@@ -12,9 +12,8 @@
 
 namespace Go\ParserReflection\NodeVisitor;
 
-use Go\ParserReflection\ValueResolver\NodeExpressionResolver;
+use Go\ParserReflection\Resolver\NodeExpressionResolver;
 use PhpParser\Node;
-use PhpParser\NodeTraverser;
 use PhpParser\NodeVisitorAbstract;
 
 /**
@@ -24,19 +23,17 @@ class StaticVariablesCollector extends NodeVisitorAbstract
 {
     /**
      * Reflection context, eg. ReflectionClass, ReflectionMethod, etc
-     *
-     * @var mixed
      */
-    private $context;
+    private mixed $context;
 
-    private $staticVariables = [];
+    private array $staticVariables = [];
 
     /**
      * Default constructor
      *
      * @param mixed $context Reflection context, eg. ReflectionClass, ReflectionMethod, etc
      */
-    public function __construct($context)
+    public function __construct(mixed $context)
     {
         $this->context = $context;
     }
@@ -48,7 +45,7 @@ public function enterNode(Node $node)
     {
         // There may be internal closures, we do not need to look at them
         if ($node instanceof Node\Expr\Closure) {
-            return NodeTraverser::DONT_TRAVERSE_CHILDREN;
+            return self::DONT_TRAVERSE_CHILDREN;
         }
 
         if ($node instanceof Node\Stmt\Static_) {