perf(doctrine): don't load every value for a non-resource collection (#5922)

Author: mwierzbi

src/Doctrine/EventListener/PurgeHttpCacheListener.php

@@ -124,6 +124,11 @@ private function gatherRelationTags(EntityManagerInterface $em, object $entity):
     {
         $associationMappings = $em->getClassMetadata(ClassUtils::getClass($entity))->getAssociationMappings();
         foreach (array_keys($associationMappings) as $property) {
+            if (
+                \array_key_exists('targetEntity', $associationMappings[$property])
+                && !$this->resourceClassResolver->isResourceClass($associationMappings[$property]['targetEntity'])) {
+                return;
+            }
             if ($this->propertyAccessor->isReadable($entity, $property)) {
                 $this->addTagsFor($this->propertyAccessor->getValue($entity, $property));
             }

tests/Doctrine/EventListener/PurgeHttpCacheListenerTest.php

@@ -201,13 +201,16 @@ public function testNotAResourceClass(): void
         // @phpstan-ignore-next-line
         $dummyClassMetadata->associationMappings = [
             'notAResource' => [],
+            'collectionOfNotAResource' => ['targetEntity' => NotAResource::class],
         ];
         $emProphecy->getClassMetadata(ContainNonResource::class)->willReturn($dummyClassMetadata);
         $eventArgs = new OnFlushEventArgs($emProphecy->reveal());
 
         $propertyAccessorProphecy = $this->prophesize(PropertyAccessorInterface::class);
         $propertyAccessorProphecy->isReadable(Argument::type(ContainNonResource::class), 'notAResource')->willReturn(true);
+        $propertyAccessorProphecy->isReadable(Argument::type(ContainNonResource::class), 'collectionOfNotAResource')->shouldNotBeCalled();
         $propertyAccessorProphecy->getValue(Argument::type(ContainNonResource::class), 'notAResource')->shouldBeCalled()->willReturn($nonResource);
+        $propertyAccessorProphecy->getValue(Argument::type(ContainNonResource::class), 'collectionOfNotAResource')->shouldNotBeCalled();
 
         $listener = new PurgeHttpCacheListener($purgerProphecy->reveal(), $iriConverterProphecy->reveal(), $resourceClassResolverProphecy->reveal(), $propertyAccessorProphecy->reveal());
         $listener->onFlush($eventArgs);