Youshido / GraphQL

src/Parser/Ast/ArgumentValue/Variable.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
     public function getValue()
     {
         if (null === $this->value) {
-            throw new \LogicException('Value is not set for variable "' . $this->name . '"');
+            throw new \LogicException('Value is not set for variable "'.$this->name.'"');
         }
 
         return $this->value;

src/Validator/ResolveValidator/ResolveValidator.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
     public function assertValidArguments(FieldInterface $field, AstFieldInterface $query, Request $request)
     {
-        $requiredArguments = array_filter($field->getArguments(), function (InputField $argument) {
+        $requiredArguments = array_filter($field->getArguments(), function(InputField $argument) {
             return $argument->getType()->getKind() === TypeMap::KIND_NON_NULL;
         });
 

Tests/Issues/Issue99/Issue99Test.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
         self::assertTrue(isset($res['data']['items']));
 
-        foreach($res['data']['items'] as $item) {
+        foreach ($res['data']['items'] as $item) {
             self::assertTrue(isset($item['custom']['value']));
             self::assertEquals(self::BUG_NOT_EXISTS_VALUE, $item['custom']['value']);
         }

Tests/Issues/Issue99/Issue99Schema.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
                         'args'    => [
                             'example' => new StringType()
                         ],
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 ['id' => 1],
                                 ['id' => 2],

src/Execution/Request.php

Doc Comments +4 added lines, -1 removed lines

@@ -223,6 +223,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $name
+     */
     public function getVariable($name)
     {
         return $this->hasVariable($name) ? $this->variables[$name] : null;
@@ -266,7 +269,7 @@ 
     }
 
     /**
-     * @return array|VariableReference[]
+     * @return VariableReference[]
      */
     public function getVariableReferences()
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 namespace Youshido\GraphQL\Execution;
 
 
-use Youshido\GraphQL\Parser\Ast\ArgumentValue\Literal;
 use Youshido\GraphQL\Parser\Ast\ArgumentValue\Variable;
 use Youshido\GraphQL\Parser\Ast\ArgumentValue\VariableReference;
 use Youshido\GraphQL\Parser\Ast\Fragment;

Spacing +1 added lines, -1 removed lines

@@ -208,7 +208,7 @@
         }
 
         $this->variables = $variables;
-        foreach($this->variableReferences as $reference) {
+        foreach ($this->variableReferences as $reference) {
             /** invalid request with no variable */
             if (!$reference->getVariable()) continue;
             $variableName = $reference->getVariable()->getName();

Braces +6 added lines, -2 removed lines

@@ -210,11 +210,15 @@
         $this->variables = $variables;
         foreach($this->variableReferences as $reference) {
             /** invalid request with no variable */
-            if (!$reference->getVariable()) continue;
+            if (!$reference->getVariable()) {
+                continue;
+            }
             $variableName = $reference->getVariable()->getName();
 
             /** no variable was set at the time */
-            if (!array_key_exists($variableName, $variables)) continue;
+            if (!array_key_exists($variableName, $variables)) {
+                continue;
+            }
 
             $reference->getVariable()->setValue($variables[$variableName]);
             $reference->setValue($variables[$variableName]);

Tests/Issues/Issue109/Issue109Schema.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
                                 ]
                             ]
                         ]),
-                        'resolve' => function ($source, array $args, ResolveInfo $info) {
+                        'resolve' => function($source, array $args, ResolveInfo $info) {
                             $internalArgs = [
                                 'comment_id' => 200
                             ];

Tests/Performance/NPlusOneTest.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
             ['id' => 2, 'name' => 'Alex'],
             ['id' => 3, 'name' => 'Mike'],
         ];
-        $posts   = [];
+        $posts = [];
         for ($i = 0; $i < 10; $i++) {
             $posts[] = [
                 'id'     => $i + 1,
@@ -67,7 +67,7 @@ 
                 'fields' => [
                     'posts' => [
                         'type'    => new ListType($postType),
-                        'resolve' => function ($source, $args, $info) {
+                        'resolve' => function($source, $args, $info) {
                             return $this->getDataForPosts();
                         }
                     ]

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;

Indentation +16 added lines, -16 removed lines

@@ -87,14 +87,14 @@ 
             [
                 'data'   => ['createList' => null],
                 'errors' => [[
-                                 'message'   => 'Not valid type for argument "posts" in query "createList"',
-                                 'locations' => [
-                                     [
-                                         'line'   => 1,
-                                         'column' => 23
-                                     ]
-                                 ]
-                             ]]
+                                    'message'   => 'Not valid type for argument "posts" in query "createList"',
+                                    'locations' => [
+                                        [
+                                            'line'   => 1,
+                                            'column' => 23
+                                        ]
+                                    ]
+                                ]]
             ],
             $processor->getResponseData()
         );
@@ -185,14 +185,14 @@ 
         $this->assertEquals([
             'data'   => ['createList' => null],
             'errors' => [[
-                             'message'   => 'Not valid type for argument "topArgument" in query "createList"',
-                             'locations' => [
-                                 [
-                                     'line'   => 1,
-                                     'column' => 23
-                                 ]
-                             ]
-                         ]],
+                                'message'   => 'Not valid type for argument "topArgument" in query "createList"',
+                                'locations' => [
+                                    [
+                                        'line'   => 1,
+                                        'column' => 23
+                                    ]
+                                ]
+                            ]],
         ], $processor->getResponseData());
         $processor->getExecutionContext()->clearErrors();
         $processor->processPayload('mutation { createList(topArgument:{

Spacing +9 added lines, -9 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'],
                             ];
                         }
                     ],