Youshido / GraphQL

src/Execution/Reducer.php

Doc Comments +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
      * Fragments (anonymous and named), and Fields.  The core of the function is simple: recurse until we hit the base
      * case of a Field and yield that back up to the visitor up in `doVisit`.
      *
-     * @param Query|Field|\Youshido\GraphQL\Parser\Ast\Interfaces\FragmentInterface $queryNode
+     * @param Query $queryNode
      * @param FieldInterface                                                        $currentLevelAST
      *
      * @return \Generator

src/Parser/Token.php

Doc Comments +4 added lines

@@ -51,6 +51,10 @@
     /** @var integer */
     private $column;
 
+    /**
+     * @param integer $line
+     * @param integer $column
+     */
     public function __construct($type, $line, $column, $data = null)
     {
         $this->type = $type;

Tests/Schema/FragmentsTest.php

Unused Use Statements -4 removed lines

@@ -2,14 +2,10 @@
 
 namespace Youshido\Tests\Schema;
 
-use Youshido\GraphQL\Config\Object\InterfaceTypeConfig;
-use Youshido\GraphQL\Config\Object\ObjectTypeConfig;
 use Youshido\GraphQL\Execution\Processor;
 use Youshido\GraphQL\Schema\Schema;
 use Youshido\GraphQL\Type\InterfaceType\AbstractInterfaceType;
-use Youshido\GraphQL\Type\InterfaceType\InterfaceType;
 use Youshido\GraphQL\Type\ListType\ListType;
-use Youshido\GraphQL\Type\NonNullType;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\Scalar\IdType;

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
                 'fields' => [
                     'user' => [
                         'type'    => new UserType(),
-                        'resolve' => function ($args) {
+                        'resolve' => function($args) {
                             return [
                                 'id'           => 'user-id-1',
                                 'fullName'     => 'Alex',
@@ -217,7 +217,7 @@ 
                 'fields' => [
                     'user' => [
                         'type'    => new UserType(),
-                        'resolve' => function ($args) {
+                        'resolve' => function($args) {
                             return [
                                 'id'       => 'user-id-1',
                                 'fullName' => 'Alex',

Tests/Schema/NonNullableTest.php

Spacing +6 added lines, -6 removed lines

@@ -48,13 +48,13 @@ 
                 'fields' => [
                     'nonNullScalar'        => [
                         'type'    => new NonNullType(new IntType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         },
                     ],
                     'nonNullList'          => [
                         'type'    => new NonNullType(new ListType(new IntType())),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ],
@@ -66,7 +66,7 @@ 
                                 'name' => new StringType(),
                             ]
                         ])),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'id'   => new uid('6cfb044c-9c0a-4ddd-9ef8-a0b940818db3'),
                                 'name' => 'Alex'
@@ -75,7 +75,7 @@ 
                     ],
                     'nonNullListOfNpnNull' => [
                         'type'    => new NonNullType(new ListType(new NonNullType(new IntType()))),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [1, null];
                         }
                     ],
@@ -84,7 +84,7 @@ 
                             'ids' => new NonNullType(new ListType(new IntType()))
                         ],
                         'type'    => new IntType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 1;
                         }
                     ],
@@ -93,7 +93,7 @@ 
                             'ids' => new NonNullType(new ListType(new NonNullType(new IntType())))
                         ],
                         'type'    => new IntType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 1;
                         }
                     ],

src/Field/AbstractField.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     }
     protected $isFinal = false;
 
-    private $nameCache            = null;
+    private $nameCache = null;
 
     public function __construct(array $config = [])
     {
@@ -34,7 +34,7 @@ 
             $config['type'] = $this->getType();
             $config['name'] = $this->getName();
             if (empty($config['name'])) {
-                $config['name'] =$this->getAutoName();
+                $config['name'] = $this->getAutoName();
             }
         }
 

src/Type/Enum/AbstractEnumType.php

Braces +3 added lines, -1 removed lines

@@ -48,7 +48,9 @@
      */
     public function isValidValue($value)
     {
-        if (is_null($value)) return true;
+        if (is_null($value)) {
+            return true;
+        }
         foreach ($this->getConfig()->get('values') as $item) {
             if ($value === $item['name'] || $value === $item['value']) {
                 return true;

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 
     public function getValidationError($value = null)
     {
-        $allowedValues             = array_map(function (array $value) {
+        $allowedValues = array_map(function(array $value) {
             return sprintf('%s (%s)', $value['name'], $value['value']);
         }, $this->getConfig()->get('values'));
         return sprintf('Value must be one of the allowed ones: %s', implode(', ', $allowedValues));

Tests/Schema/InputObjectDefaultValuesTest.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
                 ],
             ]
         ]);
-        $schema   = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -49,7 +49,7 @@ 
                                 ]
                             ])
                         ],
-                        'resolve'    => function ($source, $args) {
+                        'resolve'    => function($source, $args) {
                             return sprintf('Result with level %s and status %s',
                                 $args['statObject']['level'], $args['statObject']['status']
                             );

Tests/Issues/Issue90/Issue90Schema.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
                         'args'    => [
                             'date' => new DateTimeType('Y-m-d H:ia')
                         ],
-                        'resolve' => function ($value, $args, $info) {
+                        'resolve' => function($value, $args, $info) {
 
                             if (isset($args['date'])) {
                                 return $args['date'];
@@ -42,7 +42,7 @@ 
                         'args'    => [
                             'date' => new DateTimeType('Y-m-d H:ia')
                         ],
-                        'resolve' => function ($value, $args, $info) {
+                        'resolve' => function($value, $args, $info) {
 
                             if (isset($args['date'])) {
                                 return $args['date'];

src/Config/Traits/ArgumentsAwareConfigTrait.php

Doc Comments +3 added lines

@@ -98,6 +98,9 @@
         return $this->arguments;
     }
 
+    /**
+     * @param string $name
+     */
     public function removeArgument($name)
     {
         if ($this->hasArgument($name)) {