Youshido / GraphQL

src/Parser/Parser.php

Spacing +2 added lines, -2 removed lines

@@ -235,7 +235,7 @@ 
 
     protected function findVariable($name)
     {
-        foreach ((array) $this->data['variables'] as $variable) {
+        foreach ((array)$this->data['variables'] as $variable) {
             /** @var $variable Variable */
             if ($variable->getName() === $name) {
                 return $variable;
@@ -478,7 +478,7 @@ 
 
     protected function matchMulti($types)
     {
-        foreach ((array) $types as $type) {
+        foreach ((array)$types as $type) {
             if ($this->peek()->getType() === $type) {
                 return true;
             }

Tests/Schema/ResolveInfoTest.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 {
     public function testMethods()
     {
-        $fieldAst         = new FieldAST('name', null, [], [], new Location(1,1));
+        $fieldAst         = new FieldAST('name', null, [], [], new Location(1, 1));
         $field            = new Field(['name' => 'id', 'type' => new IntType()]);
         $returnType       = new IntType();
         $executionContext = new ExecutionContext(new TestSchema());

Tests/Library/Validator/RequestValidatorTest.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
                             new Argument('test2', new VariableReference('test2', $variable2, new Location(1, 1)), new Location(1, 1)),
                         ], [
                             new Field('test', null, [], [], new Location(1, 1))
-                        ], [], new Location(1,1))
+                        ], [], new Location(1, 1))
                     ],
                     'variables'          => [
                         $variable1,

Tests/Parser/AstTest.php

Spacing +22 added lines, -22 removed lines

@@ -25,7 +25,7 @@ 
 
     public function testArgument()
     {
-        $argument = new Argument('test', new Literal('test', new Location(1,1)), new Location(1,1));
+        $argument = new Argument('test', new Literal('test', new Location(1, 1)), new Location(1, 1));
 
         $this->assertNotNull($argument->getValue());
         $this->assertEquals($argument->getName(), 'test');
@@ -39,7 +39,7 @@ 
 
     public function testField()
     {
-        $field = new Field('field', null, [], [], new Location(1,1));
+        $field = new Field('field', null, [], [], new Location(1, 1));
 
         $this->assertEquals($field->getName(), 'field');
         $this->assertEmpty($field->getArguments());
@@ -50,7 +50,7 @@ 
         $this->assertEquals($field->getAlias(), 'alias');
         $this->assertEquals($field->getName(), 'alias');
 
-        $field->addArgument(new Argument('argument', new Literal('argument value', new Location(1,1)), new Location(1,1)));
+        $field->addArgument(new Argument('argument', new Literal('argument value', new Location(1, 1)), new Location(1, 1)));
         $this->assertTrue($field->hasArguments());
         $this->assertEquals(['argument' => 'argument value'], $field->getKeyValueArguments());
     }
@@ -58,10 +58,10 @@ 
     public function testFragment()
     {
         $fields = [
-            new Field('field', null, [], [], new Location(1,1))
+            new Field('field', null, [], [], new Location(1, 1))
         ];
 
-        $fragment = new Fragment('shipInfo', 'Ship', [], $fields, new Location(1,1));
+        $fragment = new Fragment('shipInfo', 'Ship', [], $fields, new Location(1, 1));
 
         $this->assertEquals('shipInfo', $fragment->getName());
         $this->assertEquals('Ship', $fragment->getModel());
@@ -74,7 +74,7 @@ 
         $this->assertEquals('Boat', $fragment->getModel());
 
         $newField = [
-            new Field('id', null, [], [], new Location(1,1))
+            new Field('id', null, [], [], new Location(1, 1))
         ];
         $fragment->setFields($newField);
         $this->assertEquals($newField, $fragment->getFields());
@@ -86,7 +86,7 @@ 
 
     public function testFragmentReference()
     {
-        $reference = new FragmentReference('shipInfo', new Location(1,1));
+        $reference = new FragmentReference('shipInfo', new Location(1, 1));
 
         $this->assertEquals('shipInfo', $reference->getName());
 
@@ -97,10 +97,10 @@ 
     public function testTypedFragmentReference()
     {
         $fields = [
-            new Field('id', null, [], [], new Location(1,1))
+            new Field('id', null, [], [], new Location(1, 1))
         ];
 
-        $reference = new TypedFragmentReference('Ship', $fields, [], new Location(1,1));
+        $reference = new TypedFragmentReference('Ship', $fields, [], new Location(1, 1));
 
         $this->assertEquals('Ship', $reference->getTypeName());
         $this->assertEquals($fields, $reference->getFields());
@@ -109,8 +109,8 @@ 
         $this->assertEquals('BigBoat', $reference->getTypeName());
 
         $newFields = [
-            new Field('name', null, [], [], new Location(1,1)),
-            new Field('id', null, [], [], new Location(1,1))
+            new Field('name', null, [], [], new Location(1, 1)),
+            new Field('id', null, [], [], new Location(1, 1))
         ];
 
         $reference->setFields($newFields);
@@ -120,14 +120,14 @@ 
     public function testQuery()
     {
         $arguments = [
-            new Argument('limit', new Literal('10', new Location(1,1)), new Location(1,1))
+            new Argument('limit', new Literal('10', new Location(1, 1)), new Location(1, 1))
         ];
 
         $fields = [
-            new Field('id', null, [], [], new Location(1,1))
+            new Field('id', null, [], [], new Location(1, 1))
         ];
 
-        $query = new Query('ships', 'lastShips', $arguments, $fields,[], new Location(1,1));
+        $query = new Query('ships', 'lastShips', $arguments, $fields, [], new Location(1, 1));
 
         $this->assertEquals('ships', $query->getName());
         $this->assertEquals('lastShips', $query->getAlias());
@@ -147,23 +147,23 @@ 
         $this->assertFalse($query->hasArguments());
         $this->assertFalse($query->hasFields());
 
-        $query->addArgument(new Argument('offset', new Literal(10, new Location(1,1)), new Location(1,1)));
+        $query->addArgument(new Argument('offset', new Literal(10, new Location(1, 1)), new Location(1, 1)));
         $this->assertTrue($query->hasArguments());
     }
 
     public function testArgumentValues()
     {
-        $list = new InputList(['a', 'b'], new Location(1,1));
+        $list = new InputList(['a', 'b'], new Location(1, 1));
         $this->assertEquals(['a', 'b'], $list->getValue());
         $list->setValue(['a']);
         $this->assertEquals(['a'], $list->getValue());
 
-        $inputObject = new InputObject(['a', 'b'], new Location(1,1));
+        $inputObject = new InputObject(['a', 'b'], new Location(1, 1));
         $this->assertEquals(['a', 'b'], $inputObject->getValue());
         $inputObject->setValue(['a']);
         $this->assertEquals(['a'], $inputObject->getValue());
 
-        $literal = new Literal('text', new Location(1,1));
+        $literal = new Literal('text', new Location(1, 1));
         $this->assertEquals('text', $literal->getValue());
         $literal->setValue('new text');
         $this->assertEquals('new text', $literal->getValue());
@@ -171,7 +171,7 @@ 
 
     public function testVariable()
     {
-        $variable = new Variable('id', 'int', false, false, true, new Location(1,1));
+        $variable = new Variable('id', 'int', false, false, true, new Location(1, 1));
 
         $this->assertEquals('id', $variable->getName());
         $this->assertEquals('int', $variable->getTypeName());
@@ -190,8 +190,8 @@ 
         $this->assertTrue($variable->isNullable());
         $this->assertTrue($variable->isArray());
 
-        $variable->setValue(new Literal('text', new Location(1,1)));
-        $this->assertEquals(new Literal('text', new Location(1,1)), $variable->getValue());
+        $variable->setValue(new Literal('text', new Location(1, 1)));
+        $this->assertEquals(new Literal('text', new Location(1, 1)), $variable->getValue());
     }
 
     /**
@@ -199,7 +199,7 @@ 
      */
     public function testVariableLogicException()
     {
-        $variable = new Variable('id', 'int', false, false, true, new Location(1,1));
+        $variable = new Variable('id', 'int', false, false, true, new Location(1, 1));
         $variable->getValue();
     }
 }

Tests/Parser/RequestTest.php

Spacing +3 added lines, -3 removed lines

@@ -18,8 +18,8 @@ 
 
     public function testMethods()
     {
-        $fragment1     = new Fragment('fragmentName1', 'test', [], [], new Location(1,1));
-        $fragment2     = new Fragment('fragmentName2', 'test', [], [], new Location(1,1));
+        $fragment1     = new Fragment('fragmentName1', 'test', [], [], new Location(1, 1));
+        $fragment2     = new Fragment('fragmentName2', 'test', [], [], new Location(1, 1));
         $queriesData   = ['query1', 'query2'];
         $mutationsData = ['mutation1', 'mutation2'];
         $fragmentsData = [$fragment1];
@@ -54,7 +54,7 @@ 
     public function testSetVariableParseJson()
     {
         $variables = '{"foo": "bar"}';
-        $expectedVariableArray = [ 'foo' => 'bar' ];
+        $expectedVariableArray = ['foo' => 'bar'];
 
         $request = new Request([], $variables);
         $this->assertEquals($expectedVariableArray, $request->getVariables());

Tests/Parser/VariableTest.php

Spacing +6 added lines, -6 removed lines

@@ -14,7 +14,7 @@ 
      */
     public function testGetValue($actual, $expected)
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setValue($actual);
         $this->assertEquals($var->getValue(), $expected);
     }
@@ -25,13 +25,13 @@ 
      */
     public function testGetNullValueException()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->getValue();
     }
 
     public function testGetValueReturnsDefaultValueIfNoValueSet()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setDefaultValue('default-value');
 
         $this->assertEquals(
@@ -42,7 +42,7 @@ 
 
     public function testGetValueReturnsSetValueEvenWithDefaultValue()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setValue('real-value');
         $var->setDefaultValue('default-value');
 
@@ -54,7 +54,7 @@ 
 
     public function testIndicatesDefaultValuePresent()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
         $var->setDefaultValue('default-value');
 
         $this->assertTrue(
@@ -64,7 +64,7 @@ 
 
     public function testHasNoDefaultValue()
     {
-        $var = new Variable('foo', 'bar', false, false, true, new Location(1,1));
+        $var = new Variable('foo', 'bar', false, false, true, new Location(1, 1));
 
         $this->assertFalse(
             $var->hasDefaultValue()

src/Validator/ResolveValidator/ResolveValidator.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
         }
         
         if (!(TypeService::isObjectType($objectType) || TypeService::isInputObjectType($objectType)) || !$objectType->hasField($ast->getName())) {
-            $availableFieldNames = implode(', ', array_map(function (FieldInterface $field) {
+            $availableFieldNames = implode(', ', array_map(function(FieldInterface $field) {
                 return sprintf('"%s"', $field->getName());
             }, $objectType->getFields()));
             throw new ResolveException(sprintf('Field "%s" not found in type "%s". Available fields are: %s', $ast->getName(), $objectType->getNamedType()->getName(), $availableFieldNames), $ast->getLocation());
@@ -55,7 +55,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;
         });
 

src/Schema/AbstractSchema.php

Indentation +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
             $config['mutation'] = new InternalSchemaMutationObject(['name' => $this->getName($config) . 'Mutation']);
         }
         if (!array_key_exists('types', $config)) {
-          $config['types'] = [];
+            $config['types'] = [];
         }
 
         $this->config = new SchemaConfig($config, $this);

Spacing +3 added lines, -3 removed lines

@@ -24,10 +24,10 @@ 
     public function __construct($config = [])
     {
         if (!array_key_exists('query', $config)) {
-            $config['query'] = new InternalSchemaQueryObject(['name' => $this->getName($config) . 'Query']);
+            $config['query'] = new InternalSchemaQueryObject(['name' => $this->getName($config).'Query']);
         }
         if (!array_key_exists('mutation', $config)) {
-            $config['mutation'] = new InternalSchemaMutationObject(['name' => $this->getName($config) . 'Mutation']);
+            $config['mutation'] = new InternalSchemaMutationObject(['name' => $this->getName($config).'Mutation']);
         }
         if (!array_key_exists('types', $config)) {
           $config['types'] = [];
@@ -80,6 +80,6 @@ 
     {
         $defaultName = 'RootSchema';
 
-        return isset($config["name"])? $config["name"] : $defaultName;
+        return isset($config["name"]) ? $config["name"] : $defaultName;
     }
 }

src/Execution/Processor.php

Doc Comments +6 added lines

@@ -149,6 +149,9 @@ 
         return [$this->getAlias($query) => $value];
     }
 
+    /**
+     * @return AstFieldInterface
+     */
     protected function resolveField(FieldInterface $field, AstFieldInterface $ast, $parentValue = null, $fromObject = false)
     {
         try {
@@ -220,6 +223,9 @@ 
         }
     }
 
+    /**
+     * @param \Youshido\GraphQL\Parser\Ast\Interfaces\ValueInterface $argumentValue
+     */
     private function prepareArgumentValue($argumentValue, AbstractType $argumentType, Request $request)
     {
         switch ($argumentType->getKind()) {

Indentation +21 added lines, -21 removed lines

@@ -97,21 +97,21 @@ 
 
             // If the processor found any deferred results, resolve them now.
             if (!empty($this->data) && (!empty($this->deferredResultsLeaf) || !empty($this->deferredResultsComplex))) {
-              try {
-                  while ($deferredResolver = array_shift($this->deferredResultsComplex)) {
-                      $deferredResolver->resolve();
-                  }
-
-                  // Deferred scalar and enum fields should be resolved last to
-                  // pick up as many as possible for a single batch.
-                  while ($deferredResolver = array_shift($this->deferredResultsLeaf)) {
-                      $deferredResolver->resolve();
-                  }
-              } catch (\Exception $e) {
-                  $this->executionContext->addError($e);
-              } finally {
-                  $this->data = static::unpackDeferredResults($this->data);
-              }
+                try {
+                    while ($deferredResolver = array_shift($this->deferredResultsComplex)) {
+                        $deferredResolver->resolve();
+                    }
+
+                    // Deferred scalar and enum fields should be resolved last to
+                    // pick up as many as possible for a single batch.
+                    while ($deferredResolver = array_shift($this->deferredResultsLeaf)) {
+                        $deferredResolver->resolve();
+                    }
+                } catch (\Exception $e) {
+                    $this->executionContext->addError($e);
+                } finally {
+                    $this->data = static::unpackDeferredResults($this->data);
+                }
             }
 
         } catch (\Exception $e) {
@@ -445,9 +445,9 @@ 
             }
 
             $fakeField = new Field([
-              'name' => $field->getName(),
-              'type' => $itemType,
-              'args' => $field->getArguments(),
+                'name' => $field->getName(),
+                'type' => $itemType,
+                'args' => $field->getArguments(),
             ]);
 
             $result = [];
@@ -547,9 +547,9 @@ 
             }
 
             $fakeField = new Field([
-              'name' => $field->getName(),
-              'type' => $resolvedType,
-              'args' => $field->getArguments(),
+                'name' => $field->getName(),
+                'type' => $resolvedType,
+                'args' => $field->getArguments(),
             ]);
 
             return $this->resolveObject($fakeField, $ast, $resolvedValue, true);

Spacing +6 added lines, -6 removed lines

@@ -393,7 +393,7 @@ 
      */
     protected function deferredResolve($resolvedValue, FieldInterface $field, callable $callback) {
         if ($resolvedValue instanceof DeferredResolverInterface) {
-            $deferredResult = new DeferredResult($resolvedValue, function ($resolvedValue) use ($field, $callback) {
+            $deferredResult = new DeferredResult($resolvedValue, function($resolvedValue) use ($field, $callback) {
                 // Allow nested deferred resolvers.
                 return $this->deferredResolve($resolvedValue, $field, $callback);
             });
@@ -431,7 +431,7 @@ 
         /** @var AstQuery $ast */
         $resolvedValue = $this->doResolve($field, $ast, $parentValue);
 
-        return $this->deferredResolve($resolvedValue, $field, function ($resolvedValue) use ($field, $ast, $parentValue) {
+        return $this->deferredResolve($resolvedValue, $field, function($resolvedValue) use ($field, $ast, $parentValue) {
             $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue);
 
             if (null === $resolvedValue) {
@@ -456,7 +456,7 @@ 
             $result = [];
             foreach ($resolvedValue as $resolvedValueItem) {
                 try {
-                    $fakeField->getConfig()->set('resolve', function () use ($resolvedValueItem) {
+                    $fakeField->getConfig()->set('resolve', function() use ($resolvedValueItem) {
                         return $resolvedValueItem;
                     });
 
@@ -502,7 +502,7 @@ 
             $resolvedValue = $this->doResolve($field, $ast, $parentValue);
         }
 
-        return $this->deferredResolve($resolvedValue, $field, function ($resolvedValue) use ($field, $ast, $parentValue) {
+        return $this->deferredResolve($resolvedValue, $field, function($resolvedValue) use ($field, $ast, $parentValue) {
             $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue);
 
             if (null === $resolvedValue) {
@@ -523,7 +523,7 @@ 
     {
         /** @var AstQuery $ast */
         $resolvedValue = $this->doResolve($field, $ast, $parentValue);
-        return $this->deferredResolve($resolvedValue, $field, function ($resolvedValue) use ($field, $ast, $parentValue) {
+        return $this->deferredResolve($resolvedValue, $field, function($resolvedValue) use ($field, $ast, $parentValue) {
             $this->resolveValidator->assertValidResolvedValueForField($field, $resolvedValue);
 
             if (null === $resolvedValue) {
@@ -531,7 +531,7 @@ 
             }
 
             /** @var AbstractUnionType $type */
-            $type         = $field->getType()->getNullableType();
+            $type = $field->getType()->getNullableType();
             $resolveInfo = new ResolveInfo(
                 $field,
                 $ast instanceof AstQuery ? $ast->getFields() : [],