From a3d050ff6bd1694938e7017e93ef02f9169ac25e Mon Sep 17 00:00:00 2001
From: Torsten Blindert <info@by-torsten.com>
Date: Tue, 2 Oct 2018 09:09:12 +0200
Subject: [PATCH] TASK: Code style

---
 src/Language/Parser.php                 |  4 ++--
 src/Language/Printer.php                |  2 +-
 src/Language/Visitor.php                |  2 +-
 src/Type/Schema.php                     |  2 +-
 src/Type/SchemaConfig.php               |  4 ++--
 src/Validator/Rules/KnownDirectives.php | 27 +++++++++++++------------
 6 files changed, 21 insertions(+), 20 deletions(-)

diff --git a/src/Language/Parser.php b/src/Language/Parser.php
index 76939a0..9471981 100644
--- a/src/Language/Parser.php
+++ b/src/Language/Parser.php
@@ -1476,7 +1476,7 @@ class Parser
         $start = $this->lexer->token;
         $this->expectKeyword('extend');
         $this->expectKeyword('schema');
-        $directives = $this->parseDirectives(true);
+        $directives     = $this->parseDirectives(true);
         $operationTypes = $this->peek(Token::BRACE_L)
             ? $this->many(
                 Token::BRACE_L,
@@ -1490,7 +1490,7 @@ class Parser
         return new SchemaTypeExtensionNode([
             'directives' => $directives,
             'operationTypes' => $operationTypes,
-            'loc' => $this->loc($start)
+            'loc' => $this->loc($start),
         ]);
     }
 
diff --git a/src/Language/Printer.php b/src/Language/Printer.php
index 7c3bbb4..f3148ed 100644
--- a/src/Language/Printer.php
+++ b/src/Language/Printer.php
@@ -341,7 +341,7 @@ class Printer
                             [
                                 'extend schema',
                                 $this->join($def->directives, ' '),
-                                $this->block($def->operationTypes)
+                                $this->block($def->operationTypes),
                             ],
                             ' '
                         );
diff --git a/src/Language/Visitor.php b/src/Language/Visitor.php
index 6487913..1a27f5a 100644
--- a/src/Language/Visitor.php
+++ b/src/Language/Visitor.php
@@ -166,7 +166,7 @@ class Visitor
 
         NodeKind::DIRECTIVE_DEFINITION => ['description', 'name', 'arguments', 'locations'],
 
-        NodeKind::SCHEMA_EXTENSION => ['directives', 'operationTypes']
+        NodeKind::SCHEMA_EXTENSION => ['directives', 'operationTypes'],
     ];
 
     /**
diff --git a/src/Type/Schema.php b/src/Type/Schema.php
index a3f3294..9d37c71 100644
--- a/src/Type/Schema.php
+++ b/src/Type/Schema.php
@@ -114,7 +114,7 @@ class Schema
             );
         }
 
-        $this->config = $config;
+        $this->config            = $config;
         $this->extensionASTNodes = $config->extensionASTNodes;
 
         if ($config->query) {
diff --git a/src/Type/SchemaConfig.php b/src/Type/SchemaConfig.php
index 142bafd..f2e7138 100644
--- a/src/Type/SchemaConfig.php
+++ b/src/Type/SchemaConfig.php
@@ -278,7 +278,7 @@ class SchemaConfig
     /**
      * @return SchemaTypeExtensionNode[]
      */
-    public function getExtensionASTNodes(): array
+    public function getExtensionASTNodes()
     {
         return $this->extensionASTNodes;
     }
@@ -286,7 +286,7 @@ class SchemaConfig
     /**
      * @param SchemaTypeExtensionNode[] $extensionASTNodes
      */
-    public function setExtensionASTNodes(array $extensionASTNodes): void
+    public function setExtensionASTNodes(array $extensionASTNodes)
     {
         $this->extensionASTNodes = $extensionASTNodes;
     }
diff --git a/src/Validator/Rules/KnownDirectives.php b/src/Validator/Rules/KnownDirectives.php
index 519f50c..fee65f6 100644
--- a/src/Validator/Rules/KnownDirectives.php
+++ b/src/Validator/Rules/KnownDirectives.php
@@ -5,38 +5,37 @@ declare(strict_types=1);
 namespace GraphQL\Validator\Rules;
 
 use GraphQL\Error\Error;
+use GraphQL\Language\AST\DirectiveDefinitionNode;
 use GraphQL\Language\AST\DirectiveNode;
 use GraphQL\Language\AST\InputObjectTypeDefinitionNode;
 use GraphQL\Language\AST\Node;
 use GraphQL\Language\AST\NodeKind;
 use GraphQL\Language\AST\NodeList;
 use GraphQL\Language\DirectiveLocation;
-use GraphQL\Type\Definition\Directive;
 use GraphQL\Validator\ValidationContext;
 use function count;
 use function in_array;
 use function sprintf;
+use function array_map;
 
 class KnownDirectives extends ValidationRule
 {
-    /**
-     * @param ValidationContext $context
-     * @return array
-     */
+
     public function getVisitor(ValidationContext $context)
     {
         $locationsMap = [];
         $schema = $context->getSchema();
 
-        $definedDirectives = $schema ? $schema->getDirectives() : Directive::getInternalDirectives();
+        $definedDirectives = $schema->getDirectives();
 
         foreach ($definedDirectives as $directive) {
             $locationsMap[$directive->name] = $directive->locations;
         }
 
         $astDefinition = $context->getDocument()->definitions;
+
         foreach ($astDefinition as $def) {
-            if ($def->kind === NodeKind::DIRECTIVE_DEFINITION) {
+            if ($def instanceof DirectiveDefinitionNode) {
                 $locationsMap[$def->name->value] = array_map(function ($name) {
                     return $name->value;
                 }, $def->locations);
@@ -47,7 +46,7 @@ class KnownDirectives extends ValidationRule
                 $name = $node->name->value;
                 $locations = $locationsMap[$name] ?? null;
 
-                if (!$locations) {
+                if (! $locations) {
                     $context->reportError(new Error(
                         self::unknownDirectiveMessage($name),
                         [$node]
@@ -57,11 +56,13 @@ class KnownDirectives extends ValidationRule
 
                 $candidateLocation = $this->getDirectiveLocationForASTPath($ancestors);
 
-                if ($candidateLocation && !in_array($candidateLocation, $locations)) {
-                    $context->reportError(new Error(
-                        self::misplacedDirectiveMessage($name, $candidateLocation),
-                        [$node]
-                    ));
+                if ($candidateLocation && ! in_array($candidateLocation, $locations)) {
+                    $context->reportError(
+                        new Error(
+                            self::misplacedDirectiveMessage($name, $candidateLocation),
+                            [$node]
+                        )
+                    );
                 }
             }
         ];