feat: resolve todos and create custom transformation (who cannot be used at the moment)

# Conflicts:
#	README.md

Author: Adrian Cerdeira

Classes/Migration/Transformation/PropertyValueToLowercase.php

@@ -4,39 +4,58 @@
 
 namespace Flowpack\SeoRouting\Migration\Transformation;
 
-use Neos\ContentRepository\Domain\Model\NodeData;
-use Neos\ContentRepository\Migration\Transformations\AbstractTransformation;
-
-// TODO 9.0 migration: You need to convert your AbstractTransformation to an implementation of Neos\ContentRepository\NodeMigration\Transformation\TransformationFactoryInterface
-class PropertyValueToLowercase
+use Neos\ContentRepository\Core\DimensionSpace\DimensionSpacePoint;
+use Neos\ContentRepository\Core\Projection\ContentGraph\Node;
+use Neos\ContentRepository\Core\SharedModel\Workspace\WorkspaceName;
+use Neos\ContentRepository\NodeMigration\Transformation\NodeBasedTransformationInterface;
+use Neos\ContentRepository\Core\ContentRepository;
+use Neos\ContentRepository\Core\Feature\NodeModification\Command\SetNodeProperties;
+use Neos\ContentRepository\Core\Feature\NodeModification\Dto\PropertyValuesToWrite;
+use Neos\ContentRepository\NodeMigration\Transformation\GlobalTransformationInterface;
+use Neos\ContentRepository\NodeMigration\Transformation\TransformationFactoryInterface;
+use Neos\ContentRepository\NodeMigration\Transformation\TransformationStep;
+
+/**
+ * Transforms a specified property value of a node to lowercase.
+ */
+class PropertyValueToLowercase implements TransformationFactoryInterface
 {
-    private string $propertyName;
-
-    public function setProperty(string $propertyName): void
-    {
-        $this->propertyName = $propertyName;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function isTransformable(NodeData $node)
-    {
-        return $node->hasProperty($this->propertyName);
-    }
-
     /**
-     * @inheritDoc
+     * @param array<string,string> $settings
      */
-    public function execute(NodeData $node)
+    public function build(array $settings, ContentRepository $contentRepository): GlobalTransformationInterface|NodeBasedTransformationInterface
     {
-        $currentPropertyValue = $node->getProperty($this->propertyName);
-        if (! is_string($currentPropertyValue)) {
-            return $node;
-        }
-        $newPropertyValue = strtolower($currentPropertyValue);
-        $node->setProperty($this->propertyName, $newPropertyValue);
-
-        return $node;
+        return new class(
+            $settings['property']
+        ) implements NodeBasedTransformationInterface {
+
+            private string $propertyName;
+
+            public function __construct(string $propertyName)
+            {
+                $this->propertyName = $propertyName;
+            }
+
+            public function execute(Node $node, DimensionSpacePoint $coveredDimensionSpacePoints, WorkspaceName $workspaceNameForWriting): TransformationStep
+            {
+                $currentProperty = $node->getProperty($this->propertyName);
+
+                if ($currentProperty !== null && is_string($currentProperty)) {
+                    $value = strtolower($currentProperty);
+                    return TransformationStep::fromCommand(
+                        SetNodeProperties::create(
+                            $workspaceNameForWriting,
+                            $node->aggregateId,
+                            $node->originDimensionSpacePoint,
+                            PropertyValuesToWrite::fromArray([
+                                $this->propertyName => $value,
+                            ])
+                        )
+                    );
+                }
+
+                return TransformationStep::createEmpty();
+            }
+        };
     }
 }

Migrations/ContentRepository/Version20250124153030.yaml

@@ -1,16 +1,13 @@
-up:
-  comments: 'Transforms all uriPathSegment values to lowercase'
-  warnings: 'As this migration removes the distinction between uppercase and lowercase it might not be cleanly undone by the down migration.'
-  migration:
-    - filters:
-        - type: 'NodeType'
-          settings:
-            nodeType: 'Neos.Neos:Document'
-            withSubTypes: TRUE
-      transformations:
-        - type: 'Flowpack\SeoRouting\Migration\Transformation\PropertyValueToLowercase'
-          settings:
-            property: 'uriPathSegment'
-
-down:
-  comments: 'No down migration available'
+comments: "Transforms all uriPathSegment values to lowercase"
+warnings: "As this migration removes the distinction between uppercase and lowercase it might not be cleanly undone by the down migration."
+migration:
+  - filters:
+      - type: "NodeType"
+        settings:
+          nodeType: "Neos.Neos:Document"
+          withSubTypes: TRUE
+    # Custom transformation not possible in Neos < 9.0: https://neos-project.slack.com/archives/C04V4C6B0/p1752678184783919
+    # transformations:
+    #   - type: 'Flowpack\SeoRouting\Migration\Transformation\PropertyValueToLowercase'
+    #     settings:
+    #       property: "uriPathSegment"

README.md

@@ -50,7 +50,7 @@ composer require flowpack/seo-routing
 If you want to use the *toLowerCase* feature you should execute the migration that comes with this package:
 
 ```
-./flow node:migrate 20250124153030 --confirmation true
+./flow nodemigration:execute 20250124153030 --force
 ```
 
 > [!WARNING]  

composer.json

@@ -13,7 +13,7 @@
   "require": {
     "guzzlehttp/psr7": "^2.0",
     "php": "^8.1",
-    "neos/neos": "^8.3|^9.0"
+    "neos/neos": "^9.0"
   },
   "require-dev": {
     "phpstan/phpstan": "^2.1",