Extract __callbacks__ to public constant RenameClassRector::CALLBACKS

rectorphp · Dec 5, 2022 · 9b56cb1 · 9b56cb1
1 parent b83ac0b
commit 9b56cb1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/rules-tests/Renaming/Rector/Name/RenameClassRector/config/callback.php b/rules-tests/Renaming/Rector/Name/RenameClassRector/config/callback.php
@@ -10,6 +10,6 @@
 return static function (RectorConfig $rectorConfig): void {
     $rectorConfig
         ->ruleWithConfiguration(RenameClassRector::class, [
-            '__callbacks__' => [new EnforceExceptionSuffixCallback(), new EnforceInterfaceSuffixCallback()],
+            RenameClassRector::CALLBACKS => [new EnforceExceptionSuffixCallback(), new EnforceInterfaceSuffixCallback()],
         ]);
 };
diff --git a/rules/Renaming/Rector/Name/RenameClassRector.php b/rules/Renaming/Rector/Name/RenameClassRector.php
@@ -30,6 +30,11 @@
  */
 final class RenameClassRector extends AbstractRector implements ConfigurableRectorInterface
 {
+    /**
+     * @var string
+     */
+    public const CALLBACKS = '__callbacks__';
+
     public function __construct(
         private readonly RenamedClassesDataCollector $renamedClassesDataCollector,
         private readonly ClassRenamer $classRenamer,
@@ -118,13 +123,13 @@ public function refactor(Node $node): ?Node
      */
     public function configure(array $configuration): void
     {
-        $oldToNewClassCallbacks = $configuration['__callbacks__'] ?? [];
+        $oldToNewClassCallbacks = $configuration[self::CALLBACKS] ?? [];
         Assert::isArray($oldToNewClassCallbacks);
         if ($oldToNewClassCallbacks !== []) {
             Assert::allIsCallable($oldToNewClassCallbacks);
             /** @var array<callable(ClassLike, NodeNameResolver): ?string> $oldToNewClassCallbacks */
             $this->renameClassCallbackHandler->addOldToNewClassCallbacks($oldToNewClassCallbacks);
-            unset($configuration['__callbacks__']);
+            unset($configuration[self::CALLBACKS]);
         }
 
         Assert::allString($configuration);