Fix YamlToAttributeDoctrineMappingRector (#377)

carlos-granados · web-flow · commit c9233892e967 · 2025-04-06T08:47:44.000+07:00
diff --git a/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/EmbeddableClassAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/EmbeddableClassAttributeTransformer.php
@@ -12,16 +12,17 @@
 
 final class EmbeddableClassAttributeTransformer implements ClassAttributeTransformerInterface
 {
-    public function transform(EntityMapping $entityMapping, Class_ $class): void
+    public function transform(EntityMapping $entityMapping, Class_ $class): bool
     {
         $classMapping = $entityMapping->getClassMapping();
 
         $type = $classMapping['type'] ?? null;
         if ($type !== 'embeddable') {
-            return;
+            return false;
         }
 
         $class->attrGroups[] = AttributeFactory::createGroup($this->getClassName());
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/EntityClassAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/EntityClassAttributeTransformer.php
@@ -19,13 +19,13 @@ final class EntityClassAttributeTransformer implements ClassAttributeTransformer
      */
     private const REPOSITORY_CLASS_KEY = 'repositoryClass';
 
-    public function transform(EntityMapping $entityMapping, Class_ $class): void
+    public function transform(EntityMapping $entityMapping, Class_ $class): bool
     {
         $classMapping = $entityMapping->getClassMapping();
 
         $type = $classMapping['type'] ?? null;
         if ($type !== 'entity') {
-            return;
+            return false;
         }
 
         $args = [];
@@ -37,6 +37,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void
         }
 
         $class->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/InheritanceClassAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/InheritanceClassAttributeTransformer.php
@@ -24,13 +24,13 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Class_ $class): void
+    public function transform(EntityMapping $entityMapping, Class_ $class): bool
     {
         $classMapping = $entityMapping->getClassMapping();
 
         $inheritanceType = $classMapping['inheritanceType'] ?? null;
         if ($inheritanceType === null) {
-            return;
+            return false;
         }
 
         $class->attrGroups[] = AttributeFactory::createGroup(MappingClass::INHERITANCE_TYPE, [$inheritanceType]);
@@ -45,6 +45,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void
         if (isset($classMapping['discriminatorMap'])) {
             $this->addDiscriminatorMap($classMapping['discriminatorMap'], $class);
         }
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/SoftDeletableClassAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/SoftDeletableClassAttributeTransformer.php
@@ -20,13 +20,13 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Class_ $class): void
+    public function transform(EntityMapping $entityMapping, Class_ $class): bool
     {
         $classMapping = $entityMapping->getClassMapping();
 
         $softDeletableMapping = $classMapping['gedmo']['soft_deleteable'] ?? null;
         if (! is_array($softDeletableMapping)) {
-            return;
+            return false;
         }
 
         $args = $this->nodeFactory->createArgs($softDeletableMapping);
@@ -40,6 +40,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void
         }
 
         $class->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/TableClassAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/ClassAttributeTransformer/TableClassAttributeTransformer.php
@@ -24,13 +24,13 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Class_ $class): void
+    public function transform(EntityMapping $entityMapping, Class_ $class): bool
     {
         $classMapping = $entityMapping->getClassMapping();
 
         $table = $classMapping[self::TABLE_KEY] ?? null;
         if (isset($classMapping['type']) && $classMapping['type'] !== 'entity') {
-            return;
+            return false;
         }
 
         $args = [];
@@ -42,6 +42,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void
 
         $this->addIndexes($classMapping['indexes'] ?? [], $class, MappingClass::INDEX);
         $this->addIndexes($classMapping['uniqueConstraints'] ?? [], $class, MappingClass::UNIQUE_CONSTRAINT);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/ColumnAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/ColumnAttributeTransformer.php
@@ -20,11 +20,11 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $propertyMapping = $entityMapping->matchFieldPropertyMapping($property);
         if ($propertyMapping === null) {
-            return;
+            return false;
         }
 
         // handled in another mapper
@@ -42,6 +42,7 @@ public function transform(EntityMapping $entityMapping, Property|Param $property
 
         $args = array_merge($args, $this->nodeFactory->createArgs($propertyMapping));
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/EmbeddedPropertyAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/EmbeddedPropertyAttributeTransformer.php
@@ -20,11 +20,11 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $propertyMapping = $entityMapping->matchEmbeddedPropertyMapping($property);
         if ($propertyMapping === null) {
-            return;
+            return false;
         }
 
         // handled in another attribute
@@ -34,6 +34,7 @@ public function transform(EntityMapping $entityMapping, Property|Param $property
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
 
         NodeValueNormalizer::ensureKeyIsClassConstFetch($args, 'class');
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/GedmoTimestampableAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/GedmoTimestampableAttributeTransformer.php
@@ -19,17 +19,18 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $fieldPropertyMapping = $entityMapping->matchFieldPropertyMapping($property);
 
         $timestampableMapping = $fieldPropertyMapping['gedmo']['timestampable'] ?? null;
         if (! is_array($timestampableMapping)) {
-            return;
+            return false;
         }
 
         $args = $this->nodeFactory->createArgs($timestampableMapping);
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/IdAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/IdAttributeTransformer.php
@@ -13,14 +13,15 @@
 
 final class IdAttributeTransformer implements PropertyAttributeTransformerInterface
 {
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $idMapping = $entityMapping->matchIdPropertyMapping($property);
         if (! is_array($idMapping)) {
-            return;
+            return false;
         }
 
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName());
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/IdColumnAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/IdColumnAttributeTransformer.php
@@ -13,11 +13,11 @@
 
 final class IdColumnAttributeTransformer implements PropertyAttributeTransformerInterface
 {
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $idMapping = $entityMapping->matchIdPropertyMapping($property);
         if (! is_array($idMapping)) {
-            return;
+            return false;
         }
 
         $args = [];
@@ -28,6 +28,7 @@ public function transform(EntityMapping $entityMapping, Property|Param $property
         }
 
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/IdGeneratorAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/IdGeneratorAttributeTransformer.php
@@ -27,23 +27,24 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $idMapping = $entityMapping->matchIdPropertyMapping($property);
         if (! is_array($idMapping)) {
-            return;
+            return false;
         }
 
         $generator = $idMapping[EntityMappingKey::GENERATOR] ?? null;
         if (! is_array($generator)) {
-            return;
+            return false;
         }
 
         // make sure strategy is uppercase as constant value
         $generator = $this->normalizeStrategy($generator);
 
         $args = $this->nodeFactory->createArgs($generator);
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/InverseJoinColumnAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/InverseJoinColumnAttributeTransformer.php
@@ -20,21 +20,22 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $joinTableMapping = $entityMapping->matchManyToManyPropertyMapping($property)['joinTable'] ?? null;
         if (! is_array($joinTableMapping)) {
-            return;
+            return false;
         }
 
         $joinColumns = $joinTableMapping['inverseJoinColumns'] ?? null;
         if (! is_array($joinColumns)) {
-            return;
+            return false;
         }
 
         foreach ($joinColumns as $columnName => $joinColumn) {
             $property->attrGroups[] = $this->createInverseJoinColumnAttrGroup($columnName, $joinColumn);
         }
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/JoinColumnAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/JoinColumnAttributeTransformer.php
@@ -20,13 +20,17 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
-        $this->transformMapping(
+        $hasChangedManyToMany = $this->transformMapping(
             $property,
             $entityMapping->matchManyToManyPropertyMapping($property)['joinTable'] ?? null
         );
-        $this->transformMapping($property, $entityMapping->matchManyToOnePropertyMapping($property));
+        $hasChangedManyToOne = $this->transformMapping(
+            $property,
+            $entityMapping->matchManyToOnePropertyMapping($property)
+        );
+        return $hasChangedManyToMany || $hasChangedManyToOne;
     }
 
     public function getClassName(): string
@@ -37,10 +41,10 @@ public function getClassName(): string
     /**
      * @param array<string, array<string, mixed>>|null $mapping
      */
-    private function transformMapping(Property|Param $property, ?array $mapping): void
+    private function transformMapping(Property|Param $property, ?array $mapping): bool
     {
         if (! is_array($mapping)) {
-            return;
+            return false;
         }
 
         $singleJoinColumn = $mapping['joinColumn'] ?? null;
@@ -52,12 +56,15 @@ private function transformMapping(Property|Param $property, ?array $mapping): vo
 
         $joinColumns = $mapping['joinColumns'] ?? null;
         if (! is_array($joinColumns)) {
-            return;
+            return false;
         }
 
+        $hasChanged = false;
         foreach ($joinColumns as $columnName => $joinColumn) {
+            $hasChanged = true;
             $property->attrGroups[] = $this->createJoinColumnAttrGroup($columnName, $joinColumn);
         }
+        return $hasChanged;
     }
 
     private function createJoinColumnAttrGroup(int|string $columnName, mixed $joinColumn): AttributeGroup
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/JoinTableAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/JoinTableAttributeTransformer.php
@@ -21,11 +21,11 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $joinTableMapping = $entityMapping->matchManyToManyPropertyMapping($property)['joinTable'] ?? null;
         if (! is_array($joinTableMapping)) {
-            return;
+            return false;
         }
 
         // handled by another mapper
@@ -35,6 +35,7 @@ public function transform(EntityMapping $entityMapping, Property|Param $property
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
 
         NodeValueNormalizer::ensureKeyIsClassConstFetch($args, EntityMappingKey::TARGET_ENTITY);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/ManyToManyAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/ManyToManyAttributeTransformer.php
@@ -21,11 +21,11 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $manyToManyMapping = $entityMapping->matchManyToManyPropertyMapping($property);
         if (! is_array($manyToManyMapping)) {
-            return;
+            return false;
         }
 
         // handled by another mapper
@@ -35,6 +35,7 @@ public function transform(EntityMapping $entityMapping, Property|Param $property
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
 
         NodeValueNormalizer::ensureKeyIsClassConstFetch($args, EntityMappingKey::TARGET_ENTITY);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/ManyToOneAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/ManyToOneAttributeTransformer.php
@@ -21,11 +21,11 @@ public function __construct(
     ) {
     }
 
-    public function transform(EntityMapping $entityMapping, Property|Param $property): void
+    public function transform(EntityMapping $entityMapping, Property|Param $property): bool
     {
         $manyToOneMapping = $entityMapping->matchManyToOnePropertyMapping($property);
         if (! is_array($manyToOneMapping)) {
-            return;
+            return false;
         }
 
         // handled by another mapper
@@ -38,6 +38,7 @@ public function transform(EntityMapping $entityMapping, Property|Param $property
         $property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);
 
         NodeValueNormalizer::ensureKeyIsClassConstFetch($args, EntityMappingKey::TARGET_ENTITY);
+        return true;
     }
 
     public function getClassName(): string
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/OneToManyAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/OneToManyAttributeTransformer.php
diff --git a/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/OrderByAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/PropertyAttributeTransformer/OrderByAttributeTransformer.php
diff --git a/rules/CodeQuality/AttributeTransformer/YamlToAttributeTransformer.php b/rules/CodeQuality/AttributeTransformer/YamlToAttributeTransformer.php
diff --git a/rules/CodeQuality/Contract/ClassAttributeTransformerInterface.php b/rules/CodeQuality/Contract/ClassAttributeTransformerInterface.php
diff --git a/rules/CodeQuality/Contract/PropertyAttributeTransformerInterface.php b/rules/CodeQuality/Contract/PropertyAttributeTransformerInterface.php
diff --git a/rules/CodeQuality/Rector/Class_/YamlToAttributeDoctrineMappingRector.php b/rules/CodeQuality/Rector/Class_/YamlToAttributeDoctrineMappingRector.php

Original file line number	Diff line number	Diff line change
`@@ -12,16 +12,17 @@`
`12`	`12`
`13`	`13`	`final class EmbeddableClassAttributeTransformer implements ClassAttributeTransformerInterface`
`14`	`14`	`{`
`15`		`- public function transform(EntityMapping $entityMapping, Class_ $class): void`
	`15`	`+ public function transform(EntityMapping $entityMapping, Class_ $class): bool`
`16`	`16`	`{`
`17`	`17`	`$classMapping = $entityMapping->getClassMapping();`
`18`	`18`
`19`	`19`	`$type = $classMapping['type'] ?? null;`
`20`	`20`	`if ($type !== 'embeddable') {`
`21`		`- return;`
	`21`	`+ return false;`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`$class->attrGroups[] = AttributeFactory::createGroup($this->getClassName());`
	`25`	`+ return true;`
`25`	`26`	`}`
`26`	`27`
`27`	`28`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -19,13 +19,13 @@ final class EntityClassAttributeTransformer implements ClassAttributeTransformer`
`19`	`19`	`*/`
`20`	`20`	`private const REPOSITORY_CLASS_KEY = 'repositoryClass';`
`21`	`21`
`22`		`- public function transform(EntityMapping $entityMapping, Class_ $class): void`
	`22`	`+ public function transform(EntityMapping $entityMapping, Class_ $class): bool`
`23`	`23`	`{`
`24`	`24`	`$classMapping = $entityMapping->getClassMapping();`
`25`	`25`
`26`	`26`	`$type = $classMapping['type'] ?? null;`
`27`	`27`	`if ($type !== 'entity') {`
`28`		`- return;`
	`28`	`+ return false;`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`$args = [];`
`@@ -37,6 +37,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`$class->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);`
	`40`	`+ return true;`
`40`	`41`	`}`
`41`	`42`
`42`	`43`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -24,13 +24,13 @@ public function __construct(`
`24`	`24`	`) {`
`25`	`25`	`}`
`26`	`26`
`27`		`- public function transform(EntityMapping $entityMapping, Class_ $class): void`
	`27`	`+ public function transform(EntityMapping $entityMapping, Class_ $class): bool`
`28`	`28`	`{`
`29`	`29`	`$classMapping = $entityMapping->getClassMapping();`
`30`	`30`
`31`	`31`	`$inheritanceType = $classMapping['inheritanceType'] ?? null;`
`32`	`32`	`if ($inheritanceType === null) {`
`33`		`- return;`
	`33`	`+ return false;`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`$class->attrGroups[] = AttributeFactory::createGroup(MappingClass::INHERITANCE_TYPE, [$inheritanceType]);`
`@@ -45,6 +45,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void`
`45`	`45`	`if (isset($classMapping['discriminatorMap'])) {`
`46`	`46`	`$this->addDiscriminatorMap($classMapping['discriminatorMap'], $class);`
`47`	`47`	`}`
	`48`	`+ return true;`
`48`	`49`	`}`
`49`	`50`
`50`	`51`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -20,13 +20,13 @@ public function __construct(`
`20`	`20`	`) {`
`21`	`21`	`}`
`22`	`22`
`23`		`- public function transform(EntityMapping $entityMapping, Class_ $class): void`
	`23`	`+ public function transform(EntityMapping $entityMapping, Class_ $class): bool`
`24`	`24`	`{`
`25`	`25`	`$classMapping = $entityMapping->getClassMapping();`
`26`	`26`
`27`	`27`	`$softDeletableMapping = $classMapping['gedmo']['soft_deleteable'] ?? null;`
`28`	`28`	`if (! is_array($softDeletableMapping)) {`
`29`		`- return;`
	`29`	`+ return false;`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`$args = $this->nodeFactory->createArgs($softDeletableMapping);`
`@@ -40,6 +40,7 @@ public function transform(EntityMapping $entityMapping, Class_ $class): void`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`$class->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);`
	`43`	`+ return true;`
`43`	`44`	`}`
`44`	`45`
`45`	`46`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -20,11 +20,11 @@ public function __construct(`
`20`	`20`	`) {`
`21`	`21`	`}`
`22`	`22`
`23`		`- public function transform(EntityMapping $entityMapping, Property\|Param $property): void`
	`23`	`+ public function transform(EntityMapping $entityMapping, Property\|Param $property): bool`
`24`	`24`	`{`
`25`	`25`	`$propertyMapping = $entityMapping->matchFieldPropertyMapping($property);`
`26`	`26`	`if ($propertyMapping === null) {`
`27`		`- return;`
	`27`	`+ return false;`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`// handled in another mapper`
`@@ -42,6 +42,7 @@ public function transform(EntityMapping $entityMapping, Property\|Param $property`
`42`	`42`
`43`	`43`	`$args = array_merge($args, $this->nodeFactory->createArgs($propertyMapping));`
`44`	`44`	`$property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);`
	`45`	`+ return true;`
`45`	`46`	`}`
`46`	`47`
`47`	`48`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -19,17 +19,18 @@ public function __construct(`
`19`	`19`	`) {`
`20`	`20`	`}`
`21`	`21`
`22`		`- public function transform(EntityMapping $entityMapping, Property\|Param $property): void`
	`22`	`+ public function transform(EntityMapping $entityMapping, Property\|Param $property): bool`
`23`	`23`	`{`
`24`	`24`	`$fieldPropertyMapping = $entityMapping->matchFieldPropertyMapping($property);`
`25`	`25`
`26`	`26`	`$timestampableMapping = $fieldPropertyMapping['gedmo']['timestampable'] ?? null;`
`27`	`27`	`if (! is_array($timestampableMapping)) {`
`28`		`- return;`
	`28`	`+ return false;`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`$args = $this->nodeFactory->createArgs($timestampableMapping);`
`32`	`32`	`$property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);`
	`33`	`+ return true;`
`33`	`34`	`}`
`34`	`35`
`35`	`36`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -13,14 +13,15 @@`
`13`	`13`
`14`	`14`	`final class IdAttributeTransformer implements PropertyAttributeTransformerInterface`
`15`	`15`	`{`
`16`		`- public function transform(EntityMapping $entityMapping, Property\|Param $property): void`
	`16`	`+ public function transform(EntityMapping $entityMapping, Property\|Param $property): bool`
`17`	`17`	`{`
`18`	`18`	`$idMapping = $entityMapping->matchIdPropertyMapping($property);`
`19`	`19`	`if (! is_array($idMapping)) {`
`20`		`- return;`
	`20`	`+ return false;`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`$property->attrGroups[] = AttributeFactory::createGroup($this->getClassName());`
	`24`	`+ return true;`
`24`	`25`	`}`
`25`	`26`
`26`	`27`	`public function getClassName(): string`
Original file line number	Diff line number	Diff line change
`@@ -13,11 +13,11 @@`
`13`	`13`
`14`	`14`	`final class IdColumnAttributeTransformer implements PropertyAttributeTransformerInterface`
`15`	`15`	`{`
`16`		`- public function transform(EntityMapping $entityMapping, Property\|Param $property): void`
	`16`	`+ public function transform(EntityMapping $entityMapping, Property\|Param $property): bool`
`17`	`17`	`{`
`18`	`18`	`$idMapping = $entityMapping->matchIdPropertyMapping($property);`
`19`	`19`	`if (! is_array($idMapping)) {`
`20`		`- return;`
	`20`	`+ return false;`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`$args = [];`
`@@ -28,6 +28,7 @@ public function transform(EntityMapping $entityMapping, Property\|Param $property`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`$property->attrGroups[] = AttributeFactory::createGroup($this->getClassName(), $args);`
	`31`	`+ return true;`
`31`	`32`	`}`
`32`	`33`
`33`	`34`	`public function getClassName(): string`