fix(graphql): type loader needs to return null (#5467)

api-platform · Mar 16, 2023 · 16a1a61 · 16a1a61
1 parent 0400611
commit 16a1a61
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/features/graphql/introspection.feature b/features/graphql/introspection.feature
@@ -562,7 +562,6 @@ Feature: GraphQL introspection support
     Then the response status code should be 200
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/json"
-    And the GraphQL debug message should be equal to 'Type with id "VoDummyInspectionCursorConnection" is not present in the types container'
     And the JSON node "data.typeNotAvailable" should be null
     And the JSON node "data.typeOwner.fields[1].type.name" should be equal to "VoDummyInspectionCursorConnection"
 

diff --git a/src/GraphQl/Type/SchemaBuilder.php b/src/GraphQl/Type/SchemaBuilder.php
@@ -87,8 +87,12 @@ public function getSchema(): Schema
 
         $schema = [
             'query' => $queryType,
-            'typeLoader' => function (string $typeName): Type&NamedType {
-                $type = $this->typesContainer->get($typeName);
+            'typeLoader' => function (string $typeName): ?NamedType {
+                try {
+                    $type = $this->typesContainer->get($typeName);
+                } catch (TypeNotFoundException) {
+                    return null;
+                }
 
                 return Type::getNamedType($type);
             },