move JoinColumns to NestedAnnotationToAttributeRector

rectorphp · Aug 18, 2022 · df1bdaf · df1bdaf
1 parent 8b366be
commit df1bdaf
Show file tree

Hide file tree

Showing 9 changed files with 37 additions and 94 deletions.
diff --git a/packages/PhpAttribute/RemovableAnnotationAnalyzer.php b/packages/PhpAttribute/RemovableAnnotationAnalyzer.php
diff --git a/...s/Php80/Rector/Class_/AnnotationToAttributeRector/Fixture/Doctrine/trailing_comma.php.inc b/...s/Php80/Rector/Class_/AnnotationToAttributeRector/Fixture/Doctrine/trailing_comma.php.inc
@@ -2,13 +2,13 @@
 
 namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Doctrine;
 
-use Doctrine\ORM\Mapping as ORM;
+use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\GenericAnnotation;
 
 final class TrailingComma
 {
     /**
-     * @ORM\JoinColumn(name="parent_id", referencedColumnName="id", nullable=true, )
-     **/
+     * @GenericAnnotation(key="value", )
+     */
     protected $someColumn;
 }
 
@@ -18,11 +18,11 @@ final class TrailingComma
 
 namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Doctrine;
 
-use Doctrine\ORM\Mapping as ORM;
+use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\GenericAnnotation;
 
 final class TrailingComma
 {
-    #[ORM\JoinColumn(name: 'parent_id', referencedColumnName: 'id', nullable: true)]
+    #[GenericAnnotation(key: 'value')]
     protected $someColumn;
 }
 

diff --git a/rules-tests/Php80/Rector/Class_/AnnotationToAttributeRector/config/configured_rule.php b/rules-tests/Php80/Rector/Class_/AnnotationToAttributeRector/config/configured_rule.php
@@ -34,8 +34,6 @@
             new AnnotationToAttribute('Doctrine\ORM\Mapping\Table'),
             new AnnotationToAttribute('Doctrine\ORM\Mapping\Index'),
             new AnnotationToAttribute('Doctrine\ORM\Mapping\UniqueConstraint'),
-            new AnnotationToAttribute('Doctrine\ORM\Mapping\JoinColumns'),
-            new AnnotationToAttribute('Doctrine\ORM\Mapping\JoinColumn'),
             new AnnotationToAttribute('Doctrine\ORM\Mapping\DiscriminatorMap'),
 
             // validation

diff --git a/...rine/doctrine_nested_join_columns.php.inc → ...ture/doctrine_nested_join_columns.php.inc b/...rine/doctrine_nested_join_columns.php.inc → ...ture/doctrine_nested_join_columns.php.inc
@@ -1,6 +1,6 @@
 <?php
 
-namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Doctrine;
+namespace Rector\Tests\Php80\Rector\Property\NestedAnnotationToAttributeRector\Fixture;
 
 use Doctrine\ORM\Mapping as ORM;
 
@@ -19,7 +19,7 @@ final class DoctrineNestedJoinColumns
 -----
 <?php
 
-namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Doctrine;
+namespace Rector\Tests\Php80\Rector\Property\NestedAnnotationToAttributeRector\Fixture;
 
 use Doctrine\ORM\Mapping as ORM;
 

diff --git a/...-tests/Php80/Rector/Property/NestedAnnotationToAttributeRector/config/configured_rule.php b/...-tests/Php80/Rector/Property/NestedAnnotationToAttributeRector/config/configured_rule.php
@@ -12,5 +12,9 @@
             'joinColumns' => 'Doctrine\ORM\Mapping\JoinColumn',
             'inverseJoinColumns' => 'Doctrine\ORM\Mapping\InverseJoinColumn',
         ]),
+
+        new NestedAnnotationToAttribute('Doctrine\ORM\Mapping\JoinColumns', [
+            'Doctrine\ORM\Mapping\JoinColumn',
+        ], true),
     ]);
 };
diff --git a/rules/Php80/NodeFactory/NestedAttrGroupsFactory.php b/rules/Php80/NodeFactory/NestedAttrGroupsFactory.php
@@ -28,12 +28,17 @@ public function create(array $nestedDoctrineTagAndAnnotationToAttributes, array
         foreach ($nestedDoctrineTagAndAnnotationToAttributes as $nestedDoctrineTagAndAnnotationToAttribute) {
             $doctrineAnnotationTagValueNode = $nestedDoctrineTagAndAnnotationToAttribute->getDoctrineAnnotationTagValueNode();
 
-            // add attributes
-            $attributeGroups[] = $this->phpNestedAttributeGroupFactory->create(
-                $doctrineAnnotationTagValueNode,
-                $nestedDoctrineTagAndAnnotationToAttribute->getNestedAnnotationToAttribute(),
-                $uses
-            );
+            $nestedAnnotationToAttribute = $nestedDoctrineTagAndAnnotationToAttribute->getNestedAnnotationToAttribute();
+
+            // do not create alternative for the annotation, only unwrap
+            if (! $nestedAnnotationToAttribute->shouldRemoveOriginal()) {
+                // add attributes
+                $attributeGroups[] = $this->phpNestedAttributeGroupFactory->create(
+                    $doctrineAnnotationTagValueNode,
+                    $nestedDoctrineTagAndAnnotationToAttribute->getNestedAnnotationToAttribute(),
+                    $uses
+                );
+            }
 
             $nestedAttributeGroups = $this->phpNestedAttributeGroupFactory->createNested(
                 $doctrineAnnotationTagValueNode,

diff --git a/rules/Php80/Rector/Class_/AnnotationToAttributeRector.php b/rules/Php80/Rector/Class_/AnnotationToAttributeRector.php
@@ -30,7 +30,6 @@
 use Rector\Php80\ValueObject\AnnotationToAttribute;
 use Rector\Php80\ValueObject\DoctrineTagAndAnnotationToAttribute;
 use Rector\PhpAttribute\NodeFactory\PhpAttributeGroupFactory;
-use Rector\PhpAttribute\RemovableAnnotationAnalyzer;
 use Rector\PhpAttribute\UnwrapableAnnotationAnalyzer;
 use Rector\VersionBonding\Contract\MinPhpVersionInterface;
 use Symplify\Astral\PhpDocParser\PhpDocNodeTraverser;
@@ -56,7 +55,6 @@ public function __construct(
         private readonly PhpDocTagRemover $phpDocTagRemover,
         private readonly PhpDocNodeFinder $phpDocNodeFinder,
         private readonly UnwrapableAnnotationAnalyzer $unwrapableAnnotationAnalyzer,
-        private readonly RemovableAnnotationAnalyzer $removableAnnotationAnalyzer,
         private readonly AttributeGroupNamedArgumentManipulator $attributeGroupNamedArgumentManipulator,
         private readonly PhpVersionProvider $phpVersionProvider,
         private readonly UseImportsResolver $useImportsResolver,
@@ -152,7 +150,6 @@ public function configure(array $configuration): void
         $this->annotationsToAttributes = $configuration;
 
         $this->unwrapableAnnotationAnalyzer->configure($configuration);
-        $this->removableAnnotationAnalyzer->configure($configuration);
     }
 
     public function provideMinPhpVersion(): int
@@ -258,14 +255,10 @@ private function processDoctrineAnnotationClasses(PhpDocInfo $phpDocInfo, array
                 $shouldInlinedNested = true;
             }
 
-            if (! $this->removableAnnotationAnalyzer->isRemovable($doctrineTagValueNode)) {
-                $doctrineTagAndAnnotationToAttributes[] = new DoctrineTagAndAnnotationToAttribute(
-                    $doctrineTagValueNode,
-                    $annotationToAttribute,
-                );
-            } else {
-                $shouldInlinedNested = true;
-            }
+            $doctrineTagAndAnnotationToAttributes[] = new DoctrineTagAndAnnotationToAttribute(
+                $doctrineTagValueNode,
+                $annotationToAttribute,
+            );
 
             if ($shouldInlinedNested) {
                 // inline nested

diff --git a/rules/Php80/Rector/Property/NestedAnnotationToAttributeRector.php b/rules/Php80/Rector/Property/NestedAnnotationToAttributeRector.php
@@ -155,12 +155,12 @@ private function transformDoctrineAnnotationClassesToAttributeGroups(PhpDocInfo
                 continue;
             }
 
+            $this->phpDocTagRemover->removeTagValueFromNode($phpDocInfo, $doctrineTagValueNode);
+
             $nestedDoctrineTagAndAnnotationToAttributes[] = new NestedDoctrineTagAndAnnotationToAttribute(
                 $doctrineTagValueNode,
                 $nestedAnnotationToAttribute,
             );
-
-            $this->phpDocTagRemover->removeTagValueFromNode($phpDocInfo, $doctrineTagValueNode);
         }
 
         return $this->nestedAttrGroupsFactory->create($nestedDoctrineTagAndAnnotationToAttributes, $uses);

diff --git a/rules/Php80/ValueObject/NestedAnnotationToAttribute.php b/rules/Php80/ValueObject/NestedAnnotationToAttribute.php
@@ -10,11 +10,12 @@
 final class NestedAnnotationToAttribute implements AnnotationToAttributeInterface
 {
     /**
-     * @param array<string, string> $annotationPropertiesToAttributeClasses
+     * @param array<string, string>|string[] $annotationPropertiesToAttributeClasses
      */
     public function __construct(
         private readonly string $tag,
-        private readonly array $annotationPropertiesToAttributeClasses
+        private readonly array $annotationPropertiesToAttributeClasses,
+        private bool $removeOriginal = false
     ) {
         RectorAssert::className($tag);
     }
@@ -25,7 +26,7 @@ public function getTag(): string
     }
 
     /**
-     * @return array<string, string>
+     * @return array<string, string>|string[]
      */
     public function getAnnotationPropertiesToAttributeClasses(): array
     {
@@ -36,4 +37,9 @@ public function getAttributeClass(): string
     {
         return $this->tag;
     }
+
+    public function shouldRemoveOriginal(): bool
+    {
+        return $this->removeOriginal;
+    }
 }