Youshido / GraphQL

src/Introspection/Field/TypesField.php

Indentation +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         }
 
         foreach ($value->getTypesList()->getTypes() as $type) {
-          $this->collectTypes($type);
+            $this->collectTypes($type);
         }
 
         return array_values($this->types);

src/Introspection/Traits/TypeCollectorTrait.php

Braces +3 added lines, -1 removed lines

@@ -20,7 +20,9 @@
 
     protected function collectTypes(AbstractType $type)
     {
-        if (is_object($type) && array_key_exists($type->getName(), $this->types)) return;
+        if (is_object($type) && array_key_exists($type->getName(), $this->types)) {
+            return;
+        }
 
         switch ($type->getKind()) {
             case TypeMap::KIND_INTERFACE:

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/Processor.php

Unused Use Statements -1 removed lines

@@ -24,7 +24,6 @@
 use Youshido\GraphQL\Parser\Ast\Interfaces\FieldInterface as AstFieldInterface;
 use Youshido\GraphQL\Parser\Ast\Mutation as AstMutation;
 use Youshido\GraphQL\Parser\Ast\Query as AstQuery;
-use Youshido\GraphQL\Parser\Ast\Query;
 use Youshido\GraphQL\Parser\Ast\TypedFragmentReference;
 use Youshido\GraphQL\Parser\Parser;
 use Youshido\GraphQL\Schema\AbstractSchema;

Doc Comments +3 added lines

@@ -185,6 +185,9 @@
         }
     }
 
+    /**
+     * @param \Youshido\GraphQL\Parser\Ast\Interfaces\ValueInterface $argumentValue
+     */
     private function prepareArgumentValue($argumentValue, AbstractType $argumentType, Request $request)
     {
         switch ($argumentType->getKind()) {

Spacing +2 added lines, -2 removed lines

@@ -401,7 +401,7 @@ 
         $result = [];
         foreach ($resolvedValue as $resolvedValueItem) {
             try {
-                $fakeField->getConfig()->set('resolve', function () use ($resolvedValueItem) {
+                $fakeField->getConfig()->set('resolve', function() use ($resolvedValueItem) {
                     return $resolvedValueItem;
                 });
 
@@ -451,7 +451,7 @@ 
         }
 
         /** @var AbstractUnionType $type */
-        $type         = $field->getType()->getNullableType();
+        $type = $field->getType()->getNullableType();
         $resolveInfo = new ResolveInfo(
             $field,
             $ast instanceof AstQuery ? $ast->getFields() : [],

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()
     {

Braces +6 added lines, -2 removed lines

@@ -219,11 +219,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]);

Unused Use Statements -2 removed lines

@@ -8,14 +8,12 @@
 namespace Youshido\GraphQL\Execution;
 
 use Youshido\GraphQL\Exception\Parser\InvalidRequestException;
-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;
 use Youshido\GraphQL\Parser\Ast\FragmentReference;
 use Youshido\GraphQL\Parser\Ast\Mutation;
 use Youshido\GraphQL\Parser\Ast\Query;
-use Youshido\GraphQL\Parser\Location;
 
 class Request
 {

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();
                         }
                     ]