fix(metadata): graphql can be disabled but with an existing operation

soyuka · Oct 4, 2024 · 14e2c8e · 14e2c8e
1 parent ab71ee5
commit 14e2c8e
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 3 deletions.
diff --git a/src/Laravel/ApiPlatformProvider.php b/src/Laravel/ApiPlatformProvider.php
@@ -366,7 +366,8 @@ public function register(): void
                                                                     $config->get('api-platform.graphql.enabled'),
                                                                 ),
                                                             )
-                                                        )
+                                                        ),
+                                                        $config->get('api-platform.graphql.enabled')
                                                     )
                                                 )
                                             )

diff --git a/src/Metadata/Resource/Factory/LinkResourceMetadataCollectionFactory.php b/src/Metadata/Resource/Factory/LinkResourceMetadataCollectionFactory.php
@@ -23,7 +23,7 @@
  */
 final class LinkResourceMetadataCollectionFactory implements ResourceMetadataCollectionFactoryInterface
 {
-    public function __construct(private readonly LinkFactoryInterface $linkFactory, private readonly ?ResourceMetadataCollectionFactoryInterface $decorated = null)
+    public function __construct(private readonly LinkFactoryInterface $linkFactory, private readonly ?ResourceMetadataCollectionFactoryInterface $decorated = null, private bool $graphQlEnabled = false)
     {
     }
 
@@ -37,6 +37,10 @@ public function create(string $resourceClass): ResourceMetadataCollection
             $resourceMetadataCollection = $this->decorated->create($resourceClass);
         }
 
+        if (!$this->graphQlEnabled) {
+            return $resourceMetadataCollection;
+        }
+
         foreach ($resourceMetadataCollection as $i => $resource) {
             $graphQlOperations = [];
             foreach ($resource->getGraphQlOperations() ?? [] as $graphQlOperation) {

diff --git a/src/Metadata/Resource/Factory/MetadataCollectionFactoryTrait.php b/src/Metadata/Resource/Factory/MetadataCollectionFactoryTrait.php
@@ -141,11 +141,11 @@ private function buildResourceOperations(array $metadataCollection, string $reso
                 $resources[$index] = $resource = $resource->withOperations(new Operations($operations)); // @phpstan-ignore-line
             }
 
-            $graphQlOperations = $resource->getGraphQlOperations();
             if (!$this->graphQlEnabled) {
                 continue;
             }
 
+            $graphQlOperations = $resource->getGraphQlOperations();
             if (null === $graphQlOperations) {
                 if (!$hasApiResource) {
                     $resources[$index] = $resources[$index]->withGraphQlOperations([]);

diff --git a/src/Symfony/Bundle/Resources/config/metadata/resource.xml b/src/Symfony/Bundle/Resources/config/metadata/resource.xml
@@ -52,6 +52,7 @@
         <service id="api_platform.metadata.resource.metadata_collection_factory.link" class="ApiPlatform\Metadata\Resource\Factory\LinkResourceMetadataCollectionFactory" decorates="api_platform.metadata.resource.metadata_collection_factory" decoration-priority="500" public="false">
             <argument type="service" id="api_platform.metadata.resource.link_factory" />
             <argument type="service" id="api_platform.metadata.resource.metadata_collection_factory.link.inner" />
+            <argument>%api_platform.graphql.enabled%</argument>
         </service>
 
         <!-- must run after uri template, do not change uriTemplate neither the operation names in < 500 priorities -->
-Original file line number
+Diff line change
@@ Expand Up / @@ -366,7 +366,8 @@ public function register(): void @@
                                                                         $config->get('api-platform.graphql.enabled'),
                                                                     ),
                                                                 )
-                                                            )
+                                                            ),
+                                                            $config->get('api-platform.graphql.enabled')
                                                         )
                                                     )
                                                 )
@@ Expand Down @@