TASK: Extract type converters to services.yaml

config/v12/typo3-120.php

@@ -161,4 +161,5 @@
     $rectorConfig->rule(\Ssch\TYPO3Rector\Rector\v12\v0\typo3\UseConfigArrayForTSFEPropertiesRector::class);
     $rectorConfig->rule(\Ssch\TYPO3Rector\Rector\v12\v0\typo3\ReplacePageRepoOverlayFunctionRector::class);
     $rectorConfig->rule(\Ssch\TYPO3Rector\Rector\v12\v0\typo3\ImplementSiteLanguageAwareInterfaceRector::class);
+    $rectorConfig->rule(\Ssch\TYPO3Rector\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector::class);
 };

src/Rector/v12/v0/typo3/RegisterExtbaseTypeConvertersAsServicesRector.php

@@ -0,0 +1,234 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Ssch\TYPO3Rector\Rector\v12\v0\typo3;
+
+use PhpParser\Node;
+use PhpParser\Node\Expr\StaticCall;
+use PhpParser\Node\Stmt\ClassMethod;
+use PHPStan\Reflection\ReflectionProvider;
+use PHPStan\Type\ObjectType;
+use Rector\Core\Application\FileSystem\RemovedAndAddedFilesCollector;
+use Rector\Core\PhpParser\Parser\SimplePhpParser;
+use Rector\Core\Rector\AbstractRector;
+use Rector\FileSystemRector\ValueObject\AddedFileWithContent;
+use Ssch\TYPO3Rector\Helper\FilesFinder;
+use Symfony\Component\Yaml\Yaml;
+use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+use Symplify\SmartFileSystem\SmartFileInfo;
+
+/**
+ * @changelog https://docs.typo3.org/c/typo3/cms-core/master/en-us/Changelog/12.0/Breaking-94117-RegisterExtbaseTypeConvertersAsServices.html
+ * @see \Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\RegisterExtbaseTypeConvertersAsServicesRectorTest
+ */
+final class RegisterExtbaseTypeConvertersAsServicesRector extends AbstractRector
+{
+    /**
+     * @readonly
+     */
+    private ReflectionProvider $reflectionProvider;
+
+    /**
+     * @readonly
+     */
+    private SimplePhpParser $simplePhpParser;
+
+    /**
+     * @readonly
+     */
+    private FilesFinder $filesFinder;
+
+    private RemovedAndAddedFilesCollector $removedAndAddedFilesCollector;
+
+    public function __construct(
+        ReflectionProvider $reflectionProvider,
+        SimplePhpParser $simplePhpParser,
+        FilesFinder $filesFinder,
+        RemovedAndAddedFilesCollector $removedAndAddedFilesCollector
+    ) {
+        $this->reflectionProvider = $reflectionProvider;
+        $this->simplePhpParser = $simplePhpParser;
+        $this->filesFinder = $filesFinder;
+        $this->removedAndAddedFilesCollector = $removedAndAddedFilesCollector;
+    }
+
+    /**
+     * @return array<class-string<Node>>
+     */
+    public function getNodeTypes(): array
+    {
+        return [StaticCall::class];
+    }
+
+    /**
+     * @param StaticCall $node
+     */
+    public function refactor(Node $node): ?Node
+    {
+        if ($this->shouldSkip($node)) {
+            return null;
+        }
+
+        $className = $this->valueResolver->getValue($node->args[0]->value);
+
+        if (! $this->reflectionProvider->hasClass($className)) {
+            return null;
+        }
+
+        $classReflection = $this->reflectionProvider->getClass($className);
+
+        $fileName = $classReflection->getFileName();
+
+        if (! is_string($fileName) || ! file_exists($fileName)) {
+            return null;
+        }
+
+        $extEmConf = $this->filesFinder->findExtEmConfRelativeFromGivenFileInfo(
+            new SmartFileInfo($this->file->getFilePath())
+        );
+
+        if (null === $extEmConf) {
+            return null;
+        }
+
+        $classStatements = $this->simplePhpParser->parseFile($fileName);
+
+        $this->nodeRemover->removeNode($node);
+
+        $collectServiceTags = $this->collectServiceTags($classStatements);
+
+        $existingServicesYamlFilePath = $extEmConf->getRealPathDirectory() . '/Configuration/Services.yaml';
+
+        $yamlConfiguration = $this->getYamlConfiguration($existingServicesYamlFilePath);
+
+        if (! isset($yamlConfiguration['services'])) {
+            $yamlConfiguration['services'] = [];
+        }
+
+        if (! isset($yamlConfiguration['services'][$className])) {
+            $yamlConfiguration['services'][$className]['tags'] = $collectServiceTags;
+        }
+
+        $yamlConfigurationAsYaml = Yaml::dump($yamlConfiguration, 99);
+
+        $servicesYaml = new AddedFileWithContent($existingServicesYamlFilePath, $yamlConfigurationAsYaml);
+        $this->removedAndAddedFilesCollector->addAddedFile($servicesYaml);
+
+        return null;
+    }
+
+    /**
+     * @codeCoverageIgnore
+     */
+    public function getRuleDefinition(): RuleDefinition
+    {
+        return new RuleDefinition('Register extbase type converters as services', [
+            new CodeSample(
+                <<<'CODE_SAMPLE'
+\TYPO3\CMS\Extbase\Utility\ExtensionUtility::registerTypeConverter(
+    MySpecialTypeConverter::class
+);
+CODE_SAMPLE
+                ,
+                <<<'CODE_SAMPLE'
+// Remove node and add or modify existing Services.yaml in Configuration/Services.yaml
+CODE_SAMPLE
+            ),
+        ]);
+    }
+
+    /**
+     * @param Node[] $classStatements
+     *
+     * @return array<string, string|int>
+     */
+    protected function collectServiceTags(array $classStatements): array
+    {
+        $collectServiceTags = [
+            'name' => 'extbase.type_converter',
+        ];
+
+        $this->traverseNodesWithCallable($classStatements, function (Node $node) use (&$collectServiceTags) {
+            if (! $node instanceof ClassMethod) {
+                return null;
+            }
+
+            if (! $this->nodeNameResolver->isNames(
+                $node->name,
+                ['getSupportedSourceTypes', 'getSupportedTargetType', 'getPriority']
+            )) {
+                return null;
+            }
+
+            if (null === $node->stmts) {
+                return null;
+            }
+
+            /** @var Node\Stmt\Return_[] $returns */
+            $returns = $this->betterNodeFinder->findInstanceOf($node->stmts, Node\Stmt\Return_::class);
+
+            $value = null;
+            foreach ($returns as $return) {
+                if (null === $return->expr) {
+                    continue;
+                }
+
+                $value = $this->valueResolver->getValue($return->expr);
+            }
+
+            if ($this->isName($node->name, 'getPriority')) {
+                $collectServiceTags['priority'] = $value;
+            }
+
+            if ($this->isName($node->name, 'getSupportedSourceTypes')) {
+                $collectServiceTags['sources'] = implode(',', $value);
+            }
+
+            if ($this->isName($node->name, 'getSupportedTargetType')) {
+                $collectServiceTags['target'] = $value;
+            }
+        });
+
+        return $collectServiceTags;
+    }
+
+    private function shouldSkip(StaticCall $node): bool
+    {
+        if (! $this->nodeTypeResolver->isMethodStaticCallOrClassMethodObjectType(
+            $node,
+            new ObjectType('TYPO3\\CMS\\Extbase\\Utility\\ExtensionUtility')
+        )) {
+            return true;
+        }
+
+        if (! $this->nodeNameResolver->isName($node->name, 'registerTypeConverter')) {
+            return true;
+        }
+
+        return ! isset($node->args[0]);
+    }
+
+    /**
+     * @return array<mixed>
+     */
+    private function getYamlConfiguration(string $existingServicesYamlFilePath): array
+    {
+        $yamlConfiguration = [];
+
+        if (file_exists($existingServicesYamlFilePath)) {
+            $yamlConfiguration = Yaml::parse((string) file_get_contents($existingServicesYamlFilePath));
+        } else {
+            $addedFilesWithContent = $this->removedAndAddedFilesCollector->getAddedFilesWithContent();
+            foreach ($addedFilesWithContent as $addedFileWithContent) {
+                if ($addedFileWithContent->getFilePath() === $existingServicesYamlFilePath) {
+                    $yamlConfiguration = Yaml::parse($addedFileWithContent->getFileContent());
+                    break;
+                }
+            }
+        }
+
+        return $yamlConfiguration;
+    }
+}

stubs/TYPO3/CMS/Extbase/Property/TypeConverterInterface.php

@@ -8,5 +8,16 @@
 
 interface TypeConverterInterface
 {
+    public function getSupportedSourceTypes(): array;
 
+    public function getSupportedTargetType(): string;
+
+    /**
+     * Return the priority of this TypeConverter. TypeConverters with a high priority are chosen before low priority.
+     *
+     * @return int
+     */
+    public function getPriority(): int;
+
+    public function canConvertFrom($source, string $targetType): bool;
 }

stubs/TYPO3/CMS/Extbase/Utility/ExtensionUtility.php

@@ -46,4 +46,9 @@ public static function registerModule($extensionName, $mainModuleName = '', $sub
     public static function configurePlugin($extensionName, $pluginName, array $controllerActions, array $nonCacheableControllerActions = [], $pluginType = self::PLUGIN_TYPE_PLUGIN)
     {
     }
+
+    public static function registerTypeConverter($typeConverterClassName): void
+    {
+
+    }
 }

tests/FileProcessor/Resources/Icons/Rector/v8/v3/IconsRector/IconsRectorTest.php

@@ -14,7 +14,6 @@ final class IconsRectorTest extends AbstractRectorTestCase
      */
     public function test(string $filePath): void
     {
-        #$this->markTestIncomplete('Could not find a way to make this work again for now');
         $this->doTestFile($filePath);
         $this->assertSame(1, $this->removedAndAddedFilesCollector->getAddedFileCount());
     }

tests/Rector/v12/v0/typo3/RegisterExtbaseTypeConvertersAsServicesRector/Fixture/ExpectedServices.yaml

@@ -0,0 +1,19 @@
+services:
+    _defaults:
+        autowire: true
+        autoconfigure: true
+        public: false
+    TYPO3\CMS\Extbase\:
+        resource: '../Classes/*'
+    Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySpecialTypeConverter:
+        tags:
+            name: extbase.type_converter
+            sources: 'int,string'
+            target: string
+            priority: 70
+    Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySecondSpecialTypeConverter:
+        tags:
+            name: extbase.type_converter
+            sources: array
+            target: int
+            priority: 10

tests/Rector/v12/v0/typo3/RegisterExtbaseTypeConvertersAsServicesRector/Fixture/Services.yaml

@@ -0,0 +1,13 @@
+services:
+    _defaults:
+        autowire: true
+        autoconfigure: true
+        public: false
+    TYPO3\CMS\Extbase\:
+        resource: '../Classes/*'
+    Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySpecialTypeConverter:
+        tags:
+            name: extbase.type_converter
+            sources: 'int,string'
+            target: string
+            priority: 70

tests/Rector/v12/v0/typo3/RegisterExtbaseTypeConvertersAsServicesRector/Fixture/ext_localconf.php.inc

@@ -0,0 +1,24 @@
+<?php
+
+namespace Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Fixture;
+
+use Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySpecialTypeConverter;
+use Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySecondSpecialTypeConverter;
+
+\TYPO3\CMS\Extbase\Utility\ExtensionUtility::registerTypeConverter(
+    MySpecialTypeConverter::class
+);
+\TYPO3\CMS\Extbase\Utility\ExtensionUtility::registerTypeConverter(
+    MySecondSpecialTypeConverter::class
+);
+
+?>
+-----
+<?php
+
+namespace Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Fixture;
+
+use Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySpecialTypeConverter;
+use Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector\Source\MySecondSpecialTypeConverter;
+
+?>

tests/Rector/v12/v0/typo3/RegisterExtbaseTypeConvertersAsServicesRector/RegisterExtbaseTypeConvertersAsServicesRectorTest.php

@@ -0,0 +1,50 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Ssch\TYPO3Rector\Tests\Rector\v12\v0\typo3\RegisterExtbaseTypeConvertersAsServicesRector;
+
+use Iterator;
+use Rector\Testing\PHPUnit\AbstractRectorTestCase;
+
+final class RegisterExtbaseTypeConvertersAsServicesRectorTest extends AbstractRectorTestCase
+{
+    /**
+     * @dataProvider provideData()
+     */
+    public function test(string $filePath): void
+    {
+        $this->copyAdditionalFixturesToTemporaryDirectory();
+
+        $this->doTestFile($filePath);
+
+        $addedFilesWithContent = $this->removedAndAddedFilesCollector->getAddedFilesWithContent();
+        self::assertStringEqualsFile(
+            __DIR__ . '/Fixture/ExpectedServices.yaml',
+            $addedFilesWithContent[1]->getFileContent()
+        );
+    }
+
+    /**
+     * @return Iterator<array<string>>
+     */
+    public function provideData(): Iterator
+    {
+        return $this->yieldFilesFromDirectory(__DIR__ . '/Fixture');
+    }
+
+    public function provideConfigFilePath(): string
+    {
+        return __DIR__ . '/config/configured_rule.php';
+    }
+
+    private function copyAdditionalFixturesToTemporaryDirectory(): void
+    {
+        $configurationDirectory = self::getFixtureTempDirectory() . '/Configuration/';
+        if (! file_exists($configurationDirectory)) {
+            mkdir(self::getFixtureTempDirectory() . '/Configuration/');
+        }
+
+        copy(__DIR__ . '/Fixture/Services.yaml', self::getFixtureTempDirectory() . '/Configuration/Services.yaml');
+    }
+}