skip it (#435)

Author: TomasVotruba

rules-tests/TypedCollections/Rector/NullsafeMethodCall/RemoveNullsafeOnCollectionRector/Fixture/include_mix_of_docblock.php.inc

@@ -0,0 +1,41 @@
+<?php
+
+namespace Rector\Doctrine\Tests\TypedCollections\Rector\NullsafeMethodCall\RemoveNullsafeOnCollectionRector\Fixture;
+
+use Doctrine\Common\Collections\Collection;
+
+final class IncludeMixOfDocblock
+{
+    /**
+     * @var Collection<string>|null
+     */
+    public Collection $items;
+
+    public function someMethod()
+    {
+        $values = $this->items?->count();
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Doctrine\Tests\TypedCollections\Rector\NullsafeMethodCall\RemoveNullsafeOnCollectionRector\Fixture;
+
+use Doctrine\Common\Collections\Collection;
+
+final class IncludeMixOfDocblock
+{
+    /**
+     * @var Collection<string>|null
+     */
+    public Collection $items;
+
+    public function someMethod()
+    {
+        $values = $this->items->count();
+    }
+}
+
+?>

rules/TypedCollections/TypeAnalyzer/CollectionTypeDetector.php

@@ -23,13 +23,13 @@ public function __construct(
     public function isCollectionNonNullableType(Expr $expr): bool
     {
         $exprType = $this->nodeTypeResolver->getType($expr);
+
         return $this->isCollectionObjectType($exprType);
     }
 
     public function isCollectionType(Expr $expr): bool
     {
         $exprType = $this->nodeTypeResolver->getType($expr);
-
         if ($exprType instanceof IntersectionType) {
             foreach ($exprType->getTypes() as $intersectionedType) {
                 if ($this->isCollectionObjectType($intersectionedType)) {
@@ -48,6 +48,14 @@ public function isCollectionType(Expr $expr): bool
 
     private function isCollectionObjectType(Type $exprType): bool
     {
+        if ($exprType instanceof IntersectionType) {
+            foreach ($exprType->getTypes() as $intersectionedType) {
+                if ($this->isCollectionObjectType($intersectionedType)) {
+                    return true;
+                }
+            }
+        }
+
         if (! $exprType instanceof ObjectType) {
             return false;
         }

stubs/Common/Collections/Collection.php

@@ -8,6 +8,12 @@
     return;
 }
 
-interface Collection
+/**
+ * @psalm-template TKey of array-key
+ * @psalm-template T
+ * @template-extends ReadableCollection<TKey, T>
+ * @template-extends ArrayAccess<TKey, T>
+ */
+interface Collection extends ReadableCollection, \ArrayAccess
 {
 }