[TypeDeclaration] Fix missing change type to DateTimeInterface on Mer…

…geDateTimePropertyTypeDeclarationRector (#5329)
rectorphp · Dec 5, 2023 · d22741e · d22741e
1 parent b1a50d4
commit d22741e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/...ation/Rector/Class_/MergeDateTimePropertyTypeDeclarationRector/Fixture/some_class.php.inc b/...ation/Rector/Class_/MergeDateTimePropertyTypeDeclarationRector/Fixture/some_class.php.inc
@@ -18,7 +18,7 @@ namespace Rector\Tests\TypeDeclaration\Rector\Class_\MergeDateTimePropertyTypeDe
 
 final class SomeClass
 {
-    private \DateTime $dateTime;
+    private \DateTimeInterface $dateTime;
 }
 
 ?>
diff --git a/rules/TypeDeclaration/Rector/Class_/MergeDateTimePropertyTypeDeclarationRector.php b/rules/TypeDeclaration/Rector/Class_/MergeDateTimePropertyTypeDeclarationRector.php
@@ -95,6 +95,8 @@ public function refactor(Node $node): ?Node
             if ($varType instanceof FullyQualifiedObjectType && $varType->getClassName() === 'DateTimeInterface') {
                 $phpDocInfo->removeByType(VarTagValueNode::class);
                 $this->docBlockUpdater->updateRefactoredNodeWithPhpDocInfo($property);
+
+                $property->type = new FullyQualified('DateTimeInterface');
                 $hasChanged = true;
             }
         }