Youshido / GraphQL

src/Introspection/SchemaType.php

Unused Use Statements -2 removed lines

@@ -12,8 +12,6 @@
 use Youshido\GraphQL\Schema\AbstractSchema;
 use Youshido\GraphQL\Type\ListType\ListType;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
-use Youshido\GraphQL\Type\Object\ObjectType;
-use Youshido\GraphQL\Type\TypeMap;
 
 class SchemaType extends AbstractObjectType
 {

Tests/Library/Type/InputObjectTypeTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 
 use Youshido\GraphQL\Execution\Processor;
-use Youshido\GraphQL\Parser\Ast\ArgumentValue\InputObject;
 use Youshido\GraphQL\Schema\Schema;
 use Youshido\GraphQL\Type\InputObject\InputObjectType;
 use Youshido\GraphQL\Type\ListType\ListType;

Spacing +10 added lines, -10 removed lines

@@ -55,7 +55,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
@@ -74,7 +74,7 @@ 
                             ]))
                         ],
                         'type'    => new BooleanType(),
-                        'resolve' => function ($object, $args) {
+                        'resolve' => function($object, $args) {
                             return true;
                         }
                     ]
@@ -108,7 +108,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
@@ -127,7 +127,7 @@ 
                             ])
                         ],
                         'type'    => new BooleanType(),
-                        'resolve' => function ($object, $args) {
+                        'resolve' => function($object, $args) {
                             return true;
                         }
                     ]
@@ -151,7 +151,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                         }
                     ],
                 ]
@@ -174,7 +174,7 @@ 
                                 ]
                             ])
                         ],
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'success message';
                         }
                     ]
@@ -223,10 +223,10 @@ 
                                 ],
                             ],
                         ],
-                        'resolve' => function ($source, $args) {
+                        'resolve' => function($source, $args) {
                             return [
-                                'limit is ' . $args['paging']['limit'],
-                                'offset is ' . $args['paging']['offset'],
+                                'limit is '.$args['paging']['limit'],
+                                'offset is '.$args['paging']['offset'],
                             ];
                         }
                     ],
@@ -270,7 +270,7 @@ 
                                 ]
                             ]),
                         ],
-                        'resolve' => function ($source, $args) {
+                        'resolve' => function($source, $args) {
                             return sprintf('%s by %s', $args['title'], $args['userId']);
                         }
                     ],

Tests/Schema/SchemaTest.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
         $this->assertEquals(1, count($schema->getMutationType()->getFields()));
 
-        $schema->addMutationField('changeUser', ['type' => new TestObjectType(), 'resolve' => function () {
+        $schema->addMutationField('changeUser', ['type' => new TestObjectType(), 'resolve' => function() {
         }]);
         $this->assertEquals(2, count($schema->getMutationType()->getFields()));
 
@@ -68,7 +68,7 @@ 
             'fields'      => [
                 'name' => new StringType(),
             ],
-            'resolveType' => function () use ($authorType) {
+            'resolveType' => function() use ($authorType) {
                 return $authorType;
             }
         ]);
@@ -87,7 +87,7 @@ 
                 'fields' => [
                     'user' => [
                         'type'    => $userInterface,
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'name' => 'Alex'
                             ];

Tests/Schema/IntrospectionTest.php

Spacing +7 added lines, -7 removed lines

@@ -136,7 +136,7 @@ 
             'description'       => 'latest description',
             'deprecationReason' => 'for test',
             'isDeprecated'      => true,
-            'resolve'           => function () {
+            'resolve'           => function() {
                 return [
                     'id'   => 1,
                     'name' => 'Alex'
@@ -202,9 +202,9 @@ 
                                 ['name' => 'String', 'fields' => null],
                                 ['name' => '__Schema', 'fields' => [['name' => 'queryType', 'args' => []], ['name' => 'mutationType', 'args' => []], ['name' => 'subscriptionType', 'args' => []], ['name' => 'types', 'args' => []], ['name' => 'directives', 'args' => []]]],
                                 ['name' => '__Type', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'kind', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'ofType', 'args' => []], ['name' => 'inputFields', 'args' => []], ['name' => 'enumValues', 'args' => [['defaultValue' => 'false']]], ['name' => 'fields', 'args' => [['defaultValue' => 'false']]], ['name' => 'interfaces', 'args' => []], ['name' => 'possibleTypes', 'args' => []]]],
-                                ['name' => '__InputValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'type', 'args' => []], ['name' => 'defaultValue', 'args' => []],]],
+                                ['name' => '__InputValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'type', 'args' => []], ['name' => 'defaultValue', 'args' => []], ]],
                                 ['name' => 'Boolean', 'fields' => null],
-                                ['name' => '__EnumValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'deprecationReason', 'args' => []], ['name' => 'isDeprecated', 'args' => []],]],
+                                ['name' => '__EnumValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'deprecationReason', 'args' => []], ['name' => 'isDeprecated', 'args' => []], ]],
                                 ['name' => '__Field', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'isDeprecated', 'args' => []], ['name' => 'deprecationReason', 'args' => []], ['name' => 'type', 'args' => []], ['name' => 'args', 'args' => []]]],
                                 ['name' => '__Directive', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'args', 'args' => []], ['name' => 'onOperation', 'args' => []], ['name' => 'onFragment', 'args' => []], ['name' => 'onField', 'args' => []]]],
                             ]
@@ -291,7 +291,7 @@ 
                 'id'   => ['type' => new IntType()],
                 'name' => ['type' => new IntType()],
             ],
-            'resolveType' => function ($type) {
+            'resolveType' => function($type) {
 
             }
         ]);
@@ -319,7 +319,7 @@ 
         $unionType = new UnionType([
             'name'        => 'UnionType',
             'types'       => [$object1, $object2],
-            'resolveType' => function () {
+            'resolveType' => function() {
 
             }
         ]);
@@ -330,7 +330,7 @@ 
             'args'    => [
                 'id' => ['type' => TypeMap::TYPE_INT]
             ],
-            'resolve' => function () {
+            'resolve' => function() {
                 return [
                     'id'   => 1,
                     'name' => 'Alex'
@@ -356,7 +356,7 @@ 
                     ]
                 ])
             ],
-            'resolve' => function () {
+            'resolve' => function() {
                 return null;
             }
         ]));

Tests/Issues/Issue116/Issue116Test.php

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 
     public function testInternalVariableArgument()
     {
-        $schema    = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -32,10 +32,10 @@ 
                                             'args'    => [
                                                 'size' => new NonNullType(new IntType()),
                                             ],
-                                            'resolve' => function ($source, $args) {
+                                            'resolve' => function($source, $args) {
                                                 $res = [];
                                                 foreach (range(1, $args['size']) as $i) {
-                                                    $res[] = 'Cursor #' . $i;
+                                                    $res[] = 'Cursor #'.$i;
                                                 }
 
                                                 return $res;
@@ -48,7 +48,7 @@ 
                         'args'    => [
                             'first' => new IntType(),
                         ],
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'pageInfo' => [
                                     'totalEdges' => 10,

Tests/Issues/Issue131/Issue131Test.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
     {
 
 
-        $schema    = new Schema([
+        $schema = new Schema([
             'query'    => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -47,7 +47,7 @@ 
                                 ]
                             ]))
                         ],
-                        'resolve' => function ($source, $args) {
+                        'resolve' => function($source, $args) {
                             return [
                                 'id' => '1',
                                 'name' => sprintf('Meeting with %d beans', count($args['related_beans'])),

src/Parser/Parser.php

Spacing +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
 
     protected function parseArgumentList()
     {
-        $args  = [];
+        $args = [];
 
         $this->expect(Token::TYPE_LPAREN);
 

Indentation +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
                 $this->eat(Token::TYPE_REQUIRED);
             }
 
-             $variable = new Variable(
+                $variable = new Variable(
                 $nameToken->getData(),
                 $type,
                 $required,

src/Type/InputObject/AbstractInputObjectType.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
         }
 
         $typeConfig     = $this->getConfig();
-        $requiredFields = array_filter($typeConfig->getFields(), function (FieldInterface $field) {
+        $requiredFields = array_filter($typeConfig->getFields(), function(FieldInterface $field) {
             return $field->getType()->getKind() == TypeMap::KIND_NON_NULL;
         });
 
@@ -87,7 +87,7 @@ 
     public function parseValue($value)
     {
         if (is_null($value)) return null;
-        if($value instanceof InputObject) {
+        if ($value instanceof InputObject) {
             $value = $value->getValue();
         }
 

Braces +3 added lines, -1 removed lines

@@ -86,7 +86,9 @@
 
     public function parseValue($value)
     {
-        if (is_null($value)) return null;
+        if (is_null($value)) {
+            return null;
+        }
         if($value instanceof InputObject) {
             $value = $value->getValue();
         }

examples/04_bookstore/index.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
 }
 if ($_SERVER['REQUEST_URI'] == '/graphiql.css') {
     header('Content-Type: text/css');
-    readfile(__DIR__ . '/../GraphiQL/graphiql.css');
+    readfile(__DIR__.'/../GraphiQL/graphiql.css');
     die();
 }
 
@@ -18,7 +18,7 @@ 
 use Youshido\GraphQL\Execution\Processor;
 use Youshido\GraphQL\Schema\Schema;
 
-require_once __DIR__ . '/schema-bootstrap.php';
+require_once __DIR__.'/schema-bootstrap.php';
 /** @var Schema $schema */
 $schema = new BookStoreSchema();
 
@@ -35,7 +35,7 @@ 
 $variables = isset($requestData['variables']) ? $requestData['variables'] : null;
 
 if (empty($payload)) {
-    $GraphiQLData = file_get_contents(__DIR__ . '/../GraphiQL/index.html');
+    $GraphiQLData = file_get_contents(__DIR__.'/../GraphiQL/index.html');
     echo $GraphiQLData;
     die();
 }