improve code readability

Author: joonlabs

README.md

@@ -29,7 +29,6 @@ use GraphQL\Schemas\Schema;
 use GraphQL\Types\GraphQLString;
 use GraphQL\Types\GraphQLObjectType;
 use GraphQL\Fields\GraphQLTypeField;
-use GraphQL\Arguments\GraphQLFieldArgument;
 
 // build the query type
 $QueryType = new GraphQLObjectType("Query", "Root Query", function (){

docs/schema-definition.md

@@ -10,7 +10,7 @@ The schema is a container for your type hierarchy that accepts object types as r
 It is passed to the validator and the executor and used to model your underlying data structures.
 
 ````php
-use GraphQL\Servers\Server;
+
 
 $schema = new Schema(
     $QueryType,

docs/types/object-types.md

@@ -26,7 +26,7 @@ $bookType = new GraphQLObjectType("Book", "The Book Type", function () use (&$au
             "name",
             new GraphQLString(),
             "Name of the Book",
-            function ($parent, $args, &$context) {
+            function ($parent, $args, $context) {
                 return ($args["prefix"] ?? "") . $parent["name"] . ($context["addOn"] ?? "");
             },
             [

docs/types/scalars.md

@@ -49,14 +49,14 @@ class MyCustomStringType extends GraphQLScalarType
     {
         if (!is_string($outputValue) and $outputValue !== null) {
             throw new GraphQLError(
-                "Value \"{$outputValue}\" is not of type \"{$this->getName()}\"."
+                "Value \"$outputValue\" is not of type \"{$this->getName()}\"."
             );
         }
         return $outputValue;
     }
 
     /**
-     * Parses an externally provided value (query variable) to use as an input
+     * Parses an externally provided literal value (hardcoded in GraphQL query) to use as an input.
      * 
      * @param $valueNode
      * @param $variables
@@ -75,7 +75,7 @@ class MyCustomStringType extends GraphQLScalarType
     }
 
     /**
-     * Parses an externally provided literal value (hardcoded in GraphQL query) to use as an input.
+     * Parses an externally provided value (query variable) to use as an input
      * 
      * @param $value
      * @return string|null
@@ -85,7 +85,7 @@ class MyCustomStringType extends GraphQLScalarType
     {
         if (!is_string($value) and $value !== null) {
             throw new GraphQLError(
-                "Value \"{$value}\" is not of type \"{$this->getName()}\"."
+                "Value \"$value\" is not of type \"{$this->getName()}\"."
             );
         }
         return $value;

examples/books/__data.php

@@ -15,4 +15,3 @@
     ["id" => 2, "name" => 'J. R. R. Tolkien'],
     ["id" => 3, "name" => 'Brent Weeks']
 ];
-?>

examples/books/__schema.php

@@ -25,7 +25,7 @@
             "name",
             new GraphQLString(),
             "Name of the Book",
-            function ($parent, $args, &$context) {
+            function ($parent, $args, $context) {
                 return ($args["prefix"] ?? "") . $parent["name"] . ($context["addOn"] ?? "");
             },
             [
@@ -191,4 +191,3 @@ function ($_, $args) use (&$books) {
         )
     ];
 });
-?>

examples/books/index.php

@@ -20,4 +20,3 @@
 $server = new Server($schema);
 $server->setInternalServerErrorPrint(true);
 $server->listen();
-?>

examples/parser/index.php

@@ -5,7 +5,8 @@
 // include the php-graphql-autoloader
 require '../../src/autoloader.php';
 
-use \GraphQL\Parser\Parser;
+use GraphQL\Errors\GraphQLError;
+use GraphQL\Parser\Parser;
 
 $parser = new Parser();
 $query = '
@@ -90,8 +91,7 @@ enumValues(includeDeprecated: true) {
 
 try {
     echo json_encode($parser->parse($query));
-} catch (\GraphQL\Errors\GraphQLError $e) {
+} catch (GraphQLError $e) {
     var_dump($e->getMessage(), $e->getLocations());
 }
 
-?>

examples/starwars/index.php

@@ -19,4 +19,3 @@
 $server = new Server($schema);
 $server->setInternalServerErrorPrint(true);
 $server->listen();
-?>

src/Arguments/GraphQLDirectiveArgument.php

@@ -2,8 +2,6 @@
 
 namespace GraphQL\Arguments;
 
-use GraphQL\Types\GraphQLType;
-
 class GraphQLDirectiveArgument extends GraphQLArgument
 {