release PHP 7.2 downgraded 0.17.0

driftingly · Mar 17, 2023 · 5eb5c21 · 5eb5c21
1 parent 870ca1f
commit 5eb5c21
Show file tree

Hide file tree

Showing 220 changed files with 518 additions and 4,614 deletions.
diff --git a/.github/workflows/code_analysis.yaml b/.github/workflows/code_analysis.yaml
diff --git a/.github/workflows/downgraded_release.yaml b/.github/workflows/downgraded_release.yaml
diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
diff --git a/build/composer-php-72.json b/build/composer-php-72.json
diff --git a/build/rector-downgrade-php-72.php b/build/rector-downgrade-php-72.php
diff --git a/composer.json b/composer.json
@@ -4,50 +4,16 @@
     "license": "MIT",
     "description": "Rector upgrades rules for Laravel Framework",
     "require": {
-        "php": ">=8.1",
-        "rector/rector": "^0.15.12"
-    },
-    "require-dev": {
-        "phpunit/phpunit": "^10.0",
-        "phpstan/phpstan": "^1.8.2",
-        "symplify/phpstan-rules": "^11.0",
-        "symplify/phpstan-extensions": "^11.0",
-        "symplify/easy-coding-standard": "^11.0",
-        "symplify/rule-doc-generator": "^11.0",
-        "rector/phpstan-rules": "^0.6",
-        "phpstan/extension-installer": "^1.1",
-        "phpstan/phpstan-webmozart-assert": "^1.1",
-        "phpstan/phpstan-strict-rules": "^1.2",
-        "symplify/vendor-patches": "^11.0",
-        "rector/rector-debugging": "dev-main"
+        "php": "^7.2 || ^8.0"
     },
     "autoload": {
         "psr-4": {
             "RectorLaravel\\": "src"
         }
     },
     "autoload-dev": {
-        "psr-4": {
-            "RectorLaravel\\Tests\\": "tests"
-        },
         "classmap": ["stubs"]
     },
-    "scripts": {
-        "phpstan": "vendor/bin/phpstan analyse --ansi --error-format symplify",
-        "check-cs": "vendor/bin/ecs check --ansi",
-        "fix-cs": "vendor/bin/ecs check --fix --ansi",
-        "docs": [
-            "vendor/bin/rule-doc-generator generate src --output-file docs/rector_rules_overview.md --ansi",
-            "vendor/bin/ecs check-markdown docs/rector_rules_overview.md --ansi --fix"
-        ]
-    },
     "minimum-stability": "dev",
-    "prefer-stable": true,
-    "config": {
-        "allow-plugins": {
-            "cweagans/composer-patches": true,
-            "rector/extension-installer": true,
-            "phpstan/extension-installer": true
-        }
-    }
+    "prefer-stable": true
 }
diff --git a/config/sets/laravel90.php b/config/sets/laravel90.php
@@ -40,34 +40,28 @@
 
     // https://github.com/laravel/framework/commit/e095ac0e928b5620f33c9b60816fde5ece867d32
     $rectorConfig
-        ->ruleWithConfiguration(ArgumentAdderRector::class, [new ArgumentAdder(
-            'Illuminate\Database\Eloquent\Model',
-            'touch',
-            0,
-            'attribute',
-        ),
-        ]);
+        ->ruleWithConfiguration(
+            ArgumentAdderRector::class,
+            [new ArgumentAdder('Illuminate\Database\Eloquent\Model', 'touch', 0, 'attribute'),
+        ]
+        );
 
     // https://github.com/laravel/framework/commit/6daecf43dd931dc503e410645ff4a7d611e3371f
     $rectorConfig
-        ->ruleWithConfiguration(ArgumentAdderRector::class, [new ArgumentAdder(
-            'Illuminate\Queue\Failed\FailedJobProviderInterface',
-            'flush',
-            0,
-            'hours',
-        ),
-        ]);
+        ->ruleWithConfiguration(
+            ArgumentAdderRector::class,
+            [new ArgumentAdder('Illuminate\Queue\Failed\FailedJobProviderInterface', 'flush', 0, 'hours'),
+        ]
+        );
 
     // https://github.com/laravel/framework/commit/8b40e8b7cba2fbf8337dfc05e3c6a62ae457e889
     $rectorConfig
-        ->ruleWithConfiguration(ArgumentAdderRector::class, [new ArgumentAdder(
-            'Illuminate\Foundation\Http\FormRequest',
-            'validated',
-            0,
-            'key',
-        ),
+        ->ruleWithConfiguration(
+            ArgumentAdderRector::class,
+            [new ArgumentAdder('Illuminate\Foundation\Http\FormRequest', 'validated', 0, 'key'),
             new ArgumentAdder('Illuminate\Foundation\Http\FormRequest', 'validated', 1, 'default'),
-        ]);
+        ]
+        );
 
     // https://github.com/laravel/framework/commit/84c78b9f5f3dad58f92161069e6482f7267ffdb6
     $rectorConfig

diff --git a/easy-ci.php b/easy-ci.php
diff --git a/ecs.php b/ecs.php
diff --git a/phpstan.neon b/phpstan.neon
diff --git a/phpunit.xml b/phpunit.xml
diff --git a/src/NodeAnalyzer/LumenRouteRegisteringMethodAnalyzer.php b/src/NodeAnalyzer/LumenRouteRegisteringMethodAnalyzer.php
@@ -4,32 +4,48 @@
 
 namespace RectorLaravel\NodeAnalyzer;
 
-use PhpParser\Node\Expr;
 use PhpParser\Node\Expr\MethodCall;
-use PhpParser\Node\Identifier;
 use PHPStan\Type\ObjectType;
 use Rector\NodeNameResolver\NodeNameResolver;
 use Rector\NodeTypeResolver\NodeTypeResolver;
 
 final class LumenRouteRegisteringMethodAnalyzer
 {
-    public function __construct(
-        private readonly NodeTypeResolver $nodeTypeResolver,
-        private readonly NodeNameResolver $nodeNameResolver
-    ) {
+    /**
+     * @readonly
+     * @var \Rector\NodeTypeResolver\NodeTypeResolver
+     */
+    private $nodeTypeResolver;
+
+    /**
+     * @readonly
+     * @var \Rector\NodeNameResolver\NodeNameResolver
+     */
+    private $nodeNameResolver;
+
+    public function __construct(NodeTypeResolver $nodeTypeResolver, NodeNameResolver $nodeNameResolver)
+    {
+        $this->nodeTypeResolver = $nodeTypeResolver;
+        $this->nodeNameResolver = $nodeNameResolver;
     }
 
     public function isLumenRoutingClass(MethodCall $methodCall): bool
     {
         return $this->nodeTypeResolver->isObjectType($methodCall->var, new ObjectType('Laravel\Lumen\Routing\Router'));
     }
 
-    public function isRoutesRegisterGroup(Identifier|Expr $name): bool
+    /**
+     * @param \PhpParser\Node\Identifier|\PhpParser\Node\Expr $name
+     */
+    public function isRoutesRegisterGroup($name): bool
     {
         return $this->nodeNameResolver->isName($name, 'group');
     }
 
-    public function isRoutesRegisterRoute(Identifier|Expr $name): bool
+    /**
+     * @param \PhpParser\Node\Identifier|\PhpParser\Node\Expr $name
+     */
+    public function isRoutesRegisterRoute($name): bool
     {
         return $this->nodeNameResolver->isNames($name, ['delete', 'get', 'options', 'patch', 'post', 'put']);
     }

diff --git a/src/NodeAnalyzer/StaticCallAnalyzer.php b/src/NodeAnalyzer/StaticCallAnalyzer.php
@@ -11,9 +11,15 @@
 
 final class StaticCallAnalyzer
 {
-    public function __construct(
-        private readonly NodeNameResolver $nodeNameResolver
-    ) {
+    /**
+     * @readonly
+     * @var \Rector\NodeNameResolver\NodeNameResolver
+     */
+    private $nodeNameResolver;
+
+    public function __construct(NodeNameResolver $nodeNameResolver)
+    {
+        $this->nodeNameResolver = $nodeNameResolver;
     }
 
     public function isParentCallNamed(Node $node, string $desiredMethodName): bool