Fix ImproveDoctrineCollectionDocTypeInEntityRector (#378)

Author: carlos-granados

rules/CodeQuality/Rector/Property/ImproveDoctrineCollectionDocTypeInEntityRector.php

@@ -177,15 +177,17 @@ private function refactorClassMethod(Class_ $class): ?Class_
             /** @var string $parameterName */
             $parameterName = $this->getName($param);
 
-            $this->phpDocTypeChanger->changeParamType(
+            $hasChangedParamType = $this->phpDocTypeChanger->changeParamType(
                 $classMethod,
                 $phpDocInfo,
                 $collectionObjectType,
                 $param,
                 $parameterName
             );
 
-            $hasChanged = true;
+            if ($hasChangedParamType) {
+                $hasChanged = true;
+            }
         }
 
         if ($hasChanged) {
@@ -213,7 +215,7 @@ private function refactorPropertyPhpDocInfo(Property $property, PhpDocInfo $phpD
                 $this->collectionTypeResolver->hasIndexBy($property),
                 $property
             );
-            $this->phpDocTypeChanger->changeVarType($property, $phpDocInfo, $newVarType);
+            $hasChanged = $this->phpDocTypeChanger->changeVarType($property, $phpDocInfo, $newVarType);
         } else {
             $collectionObjectType = $this->collectionTypeResolver->resolveFromToManyProperty($property);
             if (! $collectionObjectType instanceof FullyQualifiedObjectType) {
@@ -225,7 +227,11 @@ private function refactorPropertyPhpDocInfo(Property $property, PhpDocInfo $phpD
                 $this->collectionTypeResolver->hasIndexBy($property),
                 $property
             );
-            $this->phpDocTypeChanger->changeVarType($property, $phpDocInfo, $newVarType);
+            $hasChanged = $this->phpDocTypeChanger->changeVarType($property, $phpDocInfo, $newVarType);
+        }
+
+        if (!$hasChanged) {
+            return null;
         }
 
         return $property;
@@ -262,7 +268,11 @@ private function refactorAttribute(Expr $expr, PhpDocInfo $phpDocInfo, Property
             $this->collectionTypeResolver->hasIndexBy($property),
             $property
         );
-        $this->phpDocTypeChanger->changeVarType($property, $phpDocInfo, $genericObjectType);
+        $hasChanged = $this->phpDocTypeChanger->changeVarType($property, $phpDocInfo, $genericObjectType);
+
+        if (! $hasChanged) {
+            return null;
+        }
 
         return $property;
     }