Updated Rector to commit 9ba9187fb327746383199a5924497539bced744d

TomasVotruba · TomasVotruba · commit 6e231c66faeb · 2026-06-27T19:46:41.000Z
rectorphp/rector-src@9ba9187 [CodingStyle] Deprecate StaticClosureRector and StaticArrowFunctionRector (#8092)
diff --git a/rules/CodingStyle/Guard/StaticGuard.php b/rules/CodingStyle/Guard/StaticGuard.php
diff --git a/rules/CodingStyle/Rector/ArrowFunction/StaticArrowFunctionRector.php b/rules/CodingStyle/Rector/ArrowFunction/StaticArrowFunctionRector.php
@@ -5,23 +5,16 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Expr\ArrowFunction;
-use Rector\CodingStyle\Guard\StaticGuard;
+use Rector\Configuration\Deprecation\Contract\DeprecatedInterface;
+use Rector\Exception\ShouldNotHappenException;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 /**
- * @see \Rector\Tests\CodingStyle\Rector\ArrowFunction\StaticArrowFunctionRector\StaticArrowFunctionRectorTest
+ * @deprecated as noisy change with little value. Use manually or custom rule where needed instead.
  */
-final class StaticArrowFunctionRector extends AbstractRector
+final class StaticArrowFunctionRector extends AbstractRector implements DeprecatedInterface
 {
-    /**
-     * @readonly
-     */
-    private StaticGuard $staticGuard;
-    public function __construct(StaticGuard $staticGuard)
-    {
-        $this->staticGuard = $staticGuard;
-    }
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition('Changes ArrowFunction to be static when possible', [new CodeSample(<<<'CODE_SAMPLE'
@@ -44,10 +37,6 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        if (!$this->staticGuard->isLegal($node)) {
-            return null;
-        }
-        $node->static = \true;
-        return $node;
+        throw new ShouldNotHappenException(sprintf('"%s" is deprecated as noisy change with little value. Use manually or custom rule where needed instead', self::class));
     }
 }
diff --git a/rules/CodingStyle/Rector/Closure/StaticClosureRector.php b/rules/CodingStyle/Rector/Closure/StaticClosureRector.php
@@ -5,24 +5,16 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Expr\Closure;
-use Rector\CodingStyle\Guard\StaticGuard;
-use Rector\NodeTypeResolver\Node\AttributeKey;
+use Rector\Configuration\Deprecation\Contract\DeprecatedInterface;
+use Rector\Exception\ShouldNotHappenException;
 use Rector\Rector\AbstractRector;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
 use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
 /**
- * @see \Rector\Tests\CodingStyle\Rector\Closure\StaticClosureRector\StaticClosureRectorTest
+ * @deprecated as noisy change with little value. Use manually or custom rule where needed instead.
  */
-final class StaticClosureRector extends AbstractRector
+final class StaticClosureRector extends AbstractRector implements DeprecatedInterface
 {
-    /**
-     * @readonly
-     */
-    private StaticGuard $staticGuard;
-    public function __construct(StaticGuard $staticGuard)
-    {
-        $this->staticGuard = $staticGuard;
-    }
     public function getRuleDefinition(): RuleDefinition
     {
         return new RuleDefinition('Changes Closure to be static when possible', [new CodeSample(<<<'CODE_SAMPLE'
@@ -57,13 +49,6 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        if ($node->hasAttribute(AttributeKey::IS_CLOSURE_USES_THIS)) {
-            return null;
-        }
-        if (!$this->staticGuard->isLegal($node)) {
-            return null;
-        }
-        $node->static = \true;
-        return $node;
+        throw new ShouldNotHappenException(sprintf('"%s" is deprecated as noisy change with little value. Use manually or custom rule where needed instead', self::class));
     }
 }
diff --git a/rules/Php81/Rector/ClassMethod/NewInInitializerRector.php b/rules/Php81/Rector/ClassMethod/NewInInitializerRector.php
@@ -51,7 +51,7 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        throw new ShouldNotHappenException(sprintf('"%s" is deprecated as depends on context. Cannot be automated. Use manually where needed instead', self::class));
+        throw new ShouldNotHappenException(sprintf('"%s" is deprecated as depends on context. Cannot be automated. Use manually or custom rule where needed instead', self::class));
     }
     public function provideMinPhpVersion(): int
     {
diff --git a/src/Application/VersionResolver.php b/src/Application/VersionResolver.php
@@ -19,12 +19,12 @@ final class VersionResolver
      * @api
      * @var string
      */
-    public const PACKAGE_VERSION = '6dcc12669e87748c5c11607bb41580c61477fd35';
+    public const PACKAGE_VERSION = '9ba9187fb327746383199a5924497539bced744d';
     /**
      * @api
      * @var string
      */
-    public const RELEASE_DATE = '2026-06-27 17:14:24';
+    public const RELEASE_DATE = '2026-06-27 21:43:46';
     /**
      * @var int
      */
diff --git a/vendor/composer/autoload_classmap.php b/vendor/composer/autoload_classmap.php
@@ -1301,7 +1301,6 @@
     'Rector\\CodingStyle\\ClassNameImport\\ValueObject\\UsedImports' => $baseDir . '/rules/CodingStyle/ClassNameImport/ValueObject/UsedImports.php',
     'Rector\\CodingStyle\\Contract\\ClassNameImport\\ClassNameImportSkipVoterInterface' => $baseDir . '/rules/CodingStyle/Contract/ClassNameImport/ClassNameImportSkipVoterInterface.php',
     'Rector\\CodingStyle\\Guard\\ArrowFunctionAndClosureFirstClassCallableGuard' => $baseDir . '/rules/CodingStyle/Guard/ArrowFunctionAndClosureFirstClassCallableGuard.php',
-    'Rector\\CodingStyle\\Guard\\StaticGuard' => $baseDir . '/rules/CodingStyle/Guard/StaticGuard.php',
     'Rector\\CodingStyle\\Naming\\ClassNaming' => $baseDir . '/rules/CodingStyle/Naming/ClassNaming.php',
     'Rector\\CodingStyle\\NodeAnalyzer\\UseImportNameMatcher' => $baseDir . '/rules/CodingStyle/NodeAnalyzer/UseImportNameMatcher.php',
     'Rector\\CodingStyle\\NodeFactory\\ArrayCallableToMethodCallFactory' => $baseDir . '/rules/CodingStyle/NodeFactory/ArrayCallableToMethodCallFactory.php',
diff --git a/vendor/composer/autoload_static.php b/vendor/composer/autoload_static.php
@@ -1566,7 +1566,6 @@ class ComposerStaticInite7732782e781a0db6ef2f014a417d492
         'Rector\\CodingStyle\\ClassNameImport\\ValueObject\\UsedImports' => __DIR__ . '/../..' . '/rules/CodingStyle/ClassNameImport/ValueObject/UsedImports.php',
         'Rector\\CodingStyle\\Contract\\ClassNameImport\\ClassNameImportSkipVoterInterface' => __DIR__ . '/../..' . '/rules/CodingStyle/Contract/ClassNameImport/ClassNameImportSkipVoterInterface.php',
         'Rector\\CodingStyle\\Guard\\ArrowFunctionAndClosureFirstClassCallableGuard' => __DIR__ . '/../..' . '/rules/CodingStyle/Guard/ArrowFunctionAndClosureFirstClassCallableGuard.php',
-        'Rector\\CodingStyle\\Guard\\StaticGuard' => __DIR__ . '/../..' . '/rules/CodingStyle/Guard/StaticGuard.php',
         'Rector\\CodingStyle\\Naming\\ClassNaming' => __DIR__ . '/../..' . '/rules/CodingStyle/Naming/ClassNaming.php',
         'Rector\\CodingStyle\\NodeAnalyzer\\UseImportNameMatcher' => __DIR__ . '/../..' . '/rules/CodingStyle/NodeAnalyzer/UseImportNameMatcher.php',
         'Rector\\CodingStyle\\NodeFactory\\ArrayCallableToMethodCallFactory' => __DIR__ . '/../..' . '/rules/CodingStyle/NodeFactory/ArrayCallableToMethodCallFactory.php',

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public function getNodeTypes(): array`
`51`	`51`	`*/`
`52`	`52`	`public function refactor(Node $node): ?Node`
`53`	`53`	`{`
`54`		`- throw new ShouldNotHappenException(sprintf('"%s" is deprecated as depends on context. Cannot be automated. Use manually where needed instead', self::class));`
	`54`	`+ throw new ShouldNotHappenException(sprintf('"%s" is deprecated as depends on context. Cannot be automated. Use manually or custom rule where needed instead', self::class));`
`55`	`55`	`}`
`56`	`56`	`public function provideMinPhpVersion(): int`
`57`	`57`	`{`