fix: return null instead of exception for GraphQL Query operation (#6118)

Author: Zippovich2

src/GraphQl/State/Provider/ReadProvider.php

@@ -13,12 +13,12 @@
 
 namespace ApiPlatform\GraphQl\State\Provider;
 
-use ApiPlatform\Exception\ItemNotFoundException;
 use ApiPlatform\GraphQl\Resolver\Util\IdentifierTrait;
 use ApiPlatform\GraphQl\Serializer\ItemNormalizer;
 use ApiPlatform\GraphQl\Serializer\SerializerContextBuilderInterface;
 use ApiPlatform\GraphQl\Util\ArrayTrait;
 use ApiPlatform\Metadata\CollectionOperationInterface;
+use ApiPlatform\Metadata\Exception\ItemNotFoundException;
 use ApiPlatform\Metadata\GraphQl\Mutation;
 use ApiPlatform\Metadata\GraphQl\Operation as GraphQlOperation;
 use ApiPlatform\Metadata\GraphQl\QueryCollection;
@@ -78,6 +78,10 @@ public function provide(Operation $operation, array $uriVariables = [], array $c
                 }
             }
 
+            if (null === $item) {
+                return $item;
+            }
+
             if (!\is_object($item)) {
                 throw new \LogicException('Item from read provider should be a nullable object.');
             }

src/GraphQl/Tests/State/Provider/ReadProviderTest.php

@@ -15,6 +15,7 @@
 
 use ApiPlatform\GraphQl\Serializer\SerializerContextBuilderInterface;
 use ApiPlatform\GraphQl\State\Provider\ReadProvider;
+use ApiPlatform\Metadata\Exception\ItemNotFoundException;
 use ApiPlatform\Metadata\GraphQl\Query;
 use ApiPlatform\Metadata\GraphQl\QueryCollection;
 use ApiPlatform\Metadata\IriConverterInterface;
@@ -36,6 +37,26 @@ public function testProvide(): void
         $provider->provide($operation, [], $context);
     }
 
+    /**
+     * Tests that provider returns null if resource is not found.
+     *
+     * @see https://github.com/api-platform/core/issues/6072
+     */
+    public function testProvideNotExistedResource(): void
+    {
+        $context = ['args' => ['id' => '/dummy/1']];
+        $operation = new Query(class: 'dummy');
+        $decorated = $this->createMock(ProviderInterface::class);
+        $iriConverter = $this->createMock(IriConverterInterface::class);
+        $iriConverter->expects($this->once())->method('getResourceFromIri')->with('/dummy/1');
+        $iriConverter->method('getResourceFromIri')->willThrowException(new ItemNotFoundException());
+        $serializerContextBuilder = $this->createMock(SerializerContextBuilderInterface::class);
+        $provider = new ReadProvider($decorated, $iriConverter, $serializerContextBuilder, '.');
+        $result = $provider->provide($operation, [], $context);
+
+        $this->assertNull($result);
+    }
+
     public function testProvideCollection(): void
     {
         $info = $this->createMock(ResolveInfo::class);