refactor(executor): Replace deprecated AST::getOperation() with AST::getOperationAST() (#1210)

Author: klausi

.github/workflows/testing.yml

@@ -83,7 +83,7 @@ jobs:
 
       - name: Install GraphQL dependencies
         run: composer --no-interaction --no-progress require \
-          webonyx/graphql-php:^14.3 \
+          webonyx/graphql-php:^14.8 \
           drupal/typed_data:^1.0 \
           drupal/redirect:^1.6 \
           phpstan/phpstan:^0.12.70 \

composer.json

@@ -6,7 +6,7 @@
   "license": "GPL-2.0+",
   "require": {
     "php": ">=7.2",
-    "webonyx/graphql-php": "^14.5.0"
+    "webonyx/graphql-php": "^14.8.0"
   },
   "minimum-stability": "dev"
 }

src/GraphQL/Execution/Executor.php

@@ -213,8 +213,8 @@ public static function create(
    */
   public function doExecute(): Promise {
     $server = $this->context->getServer();
-    $type = AST::getOperation($this->document, $this->operation);
-    if ($type === 'query' && !!$server->get('caching')) {
+    $operation_def = AST::getOperationAST($this->document, $this->operation);
+    if ($operation_def && $operation_def->operation === 'query' && !!$server->get('caching')) {
       return $this->doExecuteCached($this->cachePrefix());
     }