[FEATURE] Add fractor-fluid package

.github/workflows/lint_test_pull_requests.yaml

@@ -21,7 +21,7 @@ jobs:
             command: test:php
           - name: Rector
             command: 'rector --dry-run'
-        directory: [ 'extension-installer', 'fractor', 'fractor-xml', 'typo3-fractor', 'fractor-doc-generator', 'fractor-yaml']
+        directory: [ 'extension-installer', 'fractor', 'fractor-xml', 'typo3-fractor', 'fractor-doc-generator', 'fractor-yaml', 'fractor-fluid']
         exclude:
           - directory: extension-installer
             composer-command: {name: 'PHPUnit', command: 'test:php'}

fractor-fluid/.gitattributes

@@ -0,0 +1,7 @@
+ecs.php export-ignore
+phpstan.neon export-ignore
+README.md export-ignore
+rector.php export-ignore
+phpunit.xml export-ignore
+tests export-ignore
+.gitignore export-ignore

fractor-fluid/.gitignore

@@ -0,0 +1,3 @@
+/vendor/
+/composer.lock
+.phpunit.cache

fractor-fluid/composer.json

@@ -0,0 +1,57 @@
+{
+    "name": "a9f/fractor-yaml",
+    "description": "Fluid extension for the File Read-Analyse-Change TOol. Allows modifying Fluid files",
+    "license": "MIT",
+    "type": "fractor-extension",
+    "authors": [
+        {
+            "name": "Andreas Wolf",
+            "email": "[email protected]",
+            "role": "Lead Developer"
+        }
+    ],
+    "require": {
+        "php": "^8.2",
+        "a9f/fractor": "@dev",
+        "a9f/fractor-extension-installer": "@dev",
+        "nette/utils": "^4.0",
+        "webmozart/assert": "^1.11"
+    },
+    "require-dev": {
+        "ergebnis/composer-normalize": "^2.42",
+        "phpstan/phpstan": "^1.10",
+        "phpunit/phpunit": "^10.5",
+        "rector/rector": "^1.0",
+        "symplify/easy-coding-standard": "^12.1"
+    },
+    "repositories": {
+        "fractor": {
+            "type": "path",
+            "url": "../*"
+        }
+    },
+    "autoload": {
+        "psr-4": {
+            "a9f\\FractorFluid\\": "src/"
+        }
+    },
+    "autoload-dev": {
+        "psr-4": {
+            "a9f\\FractorFluid\\Tests\\": "tests/"
+        }
+    },
+    "config": {
+        "allow-plugins": {
+            "a9f/fractor-extension-installer": true,
+            "ergebnis/composer-normalize": true
+        },
+        "sort-packages": true
+    },
+    "scripts": {
+        "analyze:php": "phpstan analyze",
+        "rector": "rector",
+        "style:php:check": "ecs",
+        "style:php:fix": "ecs --fix",
+        "test:php": "phpunit"
+    }
+}

fractor-fluid/config/application.php

@@ -0,0 +1,20 @@
+<?php
+
+use a9f\FractorFluid\Contract\FluidFractorRule;
+use a9f\FractorFluid\FluidFileProcessor;
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+use function Symfony\Component\DependencyInjection\Loader\Configurator\tagged_iterator;
+
+return static function (ContainerConfigurator $containerConfigurator, ContainerBuilder $containerBuilder): void {
+    $services = $containerConfigurator->services();
+    $services->defaults()
+        ->autowire()
+        ->autoconfigure();
+
+    $services->load('a9f\\FractorFluid\\', __DIR__ . '/../src/');
+
+    $services->set(FluidFileProcessor::class)->arg('$rules', tagged_iterator('fractor.fluid_rule'));
+
+    $containerBuilder->registerForAutoconfiguration(FluidFractorRule::class)->addTag('fractor.fluid_rule');
+};

fractor-fluid/ecs.php

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+return (include __DIR__ . '/../.build/ecs.php')
+    ->withPaths([
+        __DIR__ . '/config',
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+     ;

fractor-fluid/phpstan.neon

@@ -0,0 +1,6 @@
+parameters:
+    level: 8
+
+    paths:
+        - src/
+        - tests/

fractor-fluid/phpunit.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" bootstrap="vendor/autoload.php" colors="true" cacheDirectory=".phpunit.cache">
+  <testsuites>
+    <testsuite name="fractor-fluid">
+      <directory>tests</directory>
+    </testsuite>
+  </testsuites>
+  <source>
+    <include>
+      <directory>./src</directory>
+    </include>
+  </source>
+</phpunit>

fractor-fluid/rector.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+return (include __DIR__ . '/../.build/rector.php')
+    ->withPaths([
+        __DIR__ . '/config',
+        __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ]);

fractor-fluid/src/Contract/FluidFractorRule.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace a9f\FractorFluid\Contract;
+
+use a9f\Fractor\Application\Contract\FractorRule;
+
+interface FluidFractorRule extends FractorRule
+{
+    public function refactor(string $fluid): string;
+}

fractor-fluid/src/FluidFileProcessor.php

@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+namespace a9f\FractorFluid;
+
+use a9f\Fractor\Application\Contract\FileProcessor;
+use a9f\Fractor\Application\ValueObject\AppliedRule;
+use a9f\Fractor\Application\ValueObject\File;
+use a9f\FractorFluid\Contract\FluidFractorRule;
+use Webmozart\Assert\Assert;
+
+final readonly class FluidFileProcessor implements FileProcessor
+{
+    /**
+     * @param iterable<FluidFractorRule> $rules
+     */
+    public function __construct(private iterable $rules)
+    {
+        Assert::allIsInstanceOf($this->rules, FluidFractorRule::class);
+    }
+
+    public function canHandle(File $file): bool
+    {
+        return in_array($file->getFileExtension(), $this->allowedFileExtensions(), true);
+    }
+
+    public function handle(File $file): void
+    {
+        foreach ($this->rules as $rule) {
+            $newContent = $rule->refactor($file->getContent());
+
+            if ($newContent !== $file->getContent()) {
+                $file->changeFileContent($newContent);
+                $file->addAppliedRule(AppliedRule::fromRule($rule));
+            }
+        }
+    }
+
+    public function allowedFileExtensions(): array
+    {
+        return ['html', 'xml', 'txt'];
+    }
+}

fractor-fluid/tests/Fixtures/DummyFluidFractorRule.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace a9f\FractorFluid\Tests\Fixtures;
+
+use a9f\FractorFluid\Contract\FluidFractorRule;
+use Nette\Utils\Strings;
+use Symfony\Component\DependencyInjection\Exception\BadMethodCallException;
+use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
+
+final class DummyFluidFractorRule implements FluidFractorRule
+{
+    public function getRuleDefinition(): RuleDefinition
+    {
+        throw new BadMethodCallException('Not implemented yet');
+    }
+
+    public function refactor(string $fluid): string
+    {
+        $fluid = Strings::replace($fluid, '# noCacheHash="(1|0|true|false)"#imsU', '');
+
+        return Strings::replace($fluid, '# useCacheHash="(1|0|true|false)"#imsU', '');
+    }
+}

fractor-fluid/tests/FluidFileProcessor/Fixtures/fluid.html

@@ -0,0 +1,13 @@
+<f:link.page addQueryString="1" noCacheHash="1">Link</f:link.page>
+<f:link.page addQueryString="1" noCacheHash="1">Link</f:link.page>
+<f:link.page noCacheHash="0">Link</f:link.page>
+<f:link.typolink useCacheHash="1">Link</f:link.typolink>
+<f:link.typolink useCacheHash="0">Link</f:link.typolink>
+<f:uri.typolink parameter="{link}" additionalParams="&u=b" useCacheHash="true" />
+-----
+<f:link.page addQueryString="1">Link</f:link.page>
+<f:link.page addQueryString="1">Link</f:link.page>
+<f:link.page>Link</f:link.page>
+<f:link.typolink>Link</f:link.typolink>
+<f:link.typolink>Link</f:link.typolink>
+<f:uri.typolink parameter="{link}" additionalParams="&u=b" />

fractor-fluid/tests/FluidFileProcessor/FluidFileProcessorTest.php

@@ -0,0 +1,37 @@
+<?php
+
+declare(strict_types=1);
+
+namespace a9f\FractorFluid\Tests\FluidFileProcessor;
+
+use a9f\Fractor\Application\ValueObject\AppliedRule;
+use a9f\Fractor\Application\ValueObject\File;
+use a9f\Fractor\Testing\PHPUnit\AbstractFractorTestCase;
+use a9f\FractorFluid\Tests\Fixtures\DummyFluidFractorRule;
+use Iterator;
+use PHPUnit\Framework\Attributes\DataProvider;
+
+final class FluidFileProcessorTest extends AbstractFractorTestCase
+{
+    #[DataProvider('provideData')]
+    public function test(string $filePath): void
+    {
+        $this->doTestFile($filePath);
+
+        $file = $this->fileCollector->getFileByPath($filePath);
+        self::assertInstanceOf(File::class, $file);
+        self::assertEquals([AppliedRule::fromClassString(DummyFluidFractorRule::class)], $file->getAppliedRules());
+    }
+
+    public static function provideData(): Iterator
+    {
+        return self::yieldFilesFromDirectory(__DIR__ . '/Fixtures', '*.html');
+    }
+
+    protected function additionalConfigurationFiles(): array
+    {
+        return [
+            __DIR__ . '/config/config.php',
+        ];
+    }
+}

fractor-fluid/tests/FluidFileProcessor/config/config.php

@@ -0,0 +1,15 @@
+<?php
+
+use a9f\FractorFluid\Tests\Fixtures\DummyFluidFractorRule;
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+
+return static function (ContainerConfigurator $containerConfigurator): void {
+    $containerConfigurator->import(__DIR__ . '/../../../config/application.php');
+
+    $services = $containerConfigurator->services();
+    $services->defaults()
+        ->autowire()
+        ->autoconfigure();
+
+    $services->set(DummyFluidFractorRule::class);
+};