Youshido / GraphQL

examples/01_sandbox/index.php

Spacing +3 added lines, -3 removed lines

@@ -6,7 +6,7 @@ 
 use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\Scalar\StringType;
 
-require_once __DIR__ . '/../../../../../vendor/autoload.php';
+require_once __DIR__.'/../../../../../vendor/autoload.php';
 
 $processor = new Processor(new Schema([
     'query' => new ObjectType([
@@ -14,7 +14,7 @@ 
         'fields' => [
             'currentTime' => [
                 'type'    => new StringType(),
-                'resolve' => function () {
+                'resolve' => function() {
                     return date('Y-m-d H:ia');
                 }
             ]
@@ -23,4 +23,4 @@ 
 ]));
 
 $processor->processPayload('{ currentTime }');
-echo json_encode($processor->getResponseData()) . "\n";
+echo json_encode($processor->getResponseData())."\n";

src/Execution/Visitor/MaxComplexityQueryVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         $this->memo += $cost;
 
         if ($this->memo > $this->maxScore) {
-            throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
+            throw new \Exception('query exceeded max allowed complexity of '.$this->maxScore);
         }
 
         return $cost;

src/Config/Traits/ResolvableObjectTrait.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
 trait ResolvableObjectTrait
 {
 
+    /**
+     * @param string|boolean $value
+     */
     public function resolve($value, array $args, ResolveInfo $info)
     {
         if ($resolveFunction = $this->getConfig()->getResolveFunction()) {

Tests/Schema/ProcessorTest.php

Spacing +35 added lines, -35 removed lines

@@ -96,14 +96,14 @@ 
                 'fields' => [
                     'list' => [
                         'type'    => new ListType(new StringType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
                 ]
             ])
         ]));
-        $data      = $processor->processPayload(' { list }')->getResponseData();
+        $data = $processor->processPayload(' { list }')->getResponseData();
         $this->assertEquals(['data' => ['list' => null]], $data);
     }
 
@@ -116,20 +116,20 @@ 
                 'fields' => [
                     'list' => [
                         'type'    => new ListType(new StringType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
                 ]
             ])
         ]));
-        $data      = $processor->processPayload(' { __schema { subscriptionType { name } } }')->getResponseData();
+        $data = $processor->processPayload(' { __schema { subscriptionType { name } } }')->getResponseData();
         $this->assertEquals(['data' => ['__schema' => ['subscriptionType' => null]]], $data);
     }
 
     public function testSchemaOperations()
     {
-        $schema    = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -142,13 +142,13 @@ 
                                     'args'    => [
                                         'shorten' => new BooleanType()
                                     ],
-                                    'resolve' => function ($value, $args) {
+                                    'resolve' => function($value, $args) {
                                         return empty($args['shorten']) ? $value['firstName'] : $value['firstName'];
                                     }
                                 ],
                                 'id_alias'  => [
                                     'type'    => new IdType(),
-                                    'resolve' => function ($value) {
+                                    'resolve' => function($value) {
                                         return $value['id'];
                                     }
                                 ],
@@ -156,7 +156,7 @@ 
                                 'code'      => new StringType(),
                             ]
                         ]),
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             $data = ['id' => '123', 'firstName' => 'John', 'code' => '007'];
                             if (!empty($args['upper'])) {
                                 foreach ($data as $key => $value) {
@@ -175,19 +175,19 @@ 
                     ],
                     'randomUser'        => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['invalidField' => 'John'];
                         }
                     ],
                     'invalidValueQuery' => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'stringValue';
                         }
                     ],
                     'labels'            => [
                         'type'    => new ListType(new StringType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['one', 'two'];
                         }
                     ]
@@ -218,7 +218,7 @@ 
             ->addField(new Field([
                 'name'    => 'increaseCounter',
                 'type'    => new IntType(),
-                'resolve' => function ($value, $args, ResolveInfo $info) {
+                'resolve' => function($value, $args, ResolveInfo $info) {
                     return $this->_counter += $args['amount'];
                 },
                 'args'    => [
@@ -230,13 +230,13 @@ 
             ]))->addField(new Field([
                 'name'    => 'invalidResolveTypeMutation',
                 'type'    => new NonNullType(new IntType()),
-                'resolve' => function () {
+                'resolve' => function() {
                     return null;
                 }
             ]))->addField(new Field([
                 'name'    => 'interfacedMutation',
                 'type'    => new TestInterfaceType(),
-                'resolve' => function () {
+                'resolve' => function() {
                     return ['name' => 'John'];
                 }
             ]));
@@ -317,7 +317,7 @@ 
                             ]))
                         ],
                         'type'    => new StringType(),
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             return $args['argument1'];
                         }
                     ]
@@ -362,43 +362,43 @@ 
                 'fields' => [
                     'listQuery'                 => [
                         'type'    => new ListType(new TestEnumType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'invalid list';
                         }
                     ],
                     'listEnumQuery'             => [
                         'type'    => new ListType(new TestEnumType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['invalid enum'];
                         }
                     ],
                     'invalidEnumQuery'          => [
                         'type'    => new TestEnumType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'invalid enum';
                         }
                     ],
                     'enumQuery'                 => [
                         'type'    => new TestEnumType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 1;
                         }
                     ],
                     'invalidNonNullQuery'       => [
                         'type'    => new NonNullType(new IntType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ],
                     'invalidNonNullInsideQuery' => [
                         'type'    => new NonNullType(new IntType()),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'hello';
                         }
                     ],
                     'objectQuery'               => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['name' => 'John'];
                         }
                     ],
@@ -410,7 +410,7 @@ 
                                 'enum'   => new TestEnumType(),
                             ],
                         ]),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'object' => [
                                     'name' => 'John'
@@ -489,10 +489,10 @@ 
             ]
         ]);
 
-        $union        = new UnionType([
+        $union = new UnionType([
             'name'        => 'TestUnion',
             'types'       => [$object1, $object2],
-            'resolveType' => function ($object) use ($object1, $object2) {
+            'resolveType' => function($object) use ($object1, $object2) {
                 if (isset($object['id'])) {
                     return $object1;
                 }
@@ -503,11 +503,11 @@ 
         $invalidUnion = new UnionType([
             'name'        => 'TestUnion',
             'types'       => [$object1, $object2],
-            'resolveType' => function ($object) use ($object3) {
+            'resolveType' => function($object) use ($object3) {
                 return $object3;
             }
         ]);
-        $processor    = new Processor(new Schema([
+        $processor = new Processor(new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -517,7 +517,7 @@ 
                             'type' => ['type' => 'string']
                         ],
                         'cost'    => 10,
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             if ($args['type'] == 'object1') {
                                 return [
                                     'id' => 43
@@ -531,7 +531,7 @@ 
                     ],
                     'invalidUnion' => [
                         'type'    => $invalidUnion,
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['name' => 'name resolved'];
                         }
                     ],
@@ -593,7 +593,7 @@ 
                 'fields' => [
                     'currentUser' => [
                         'type'    => new StringType(),
-                        'resolve' => function ($source, $args, ResolveInfo $info) {
+                        'resolve' => function($source, $args, ResolveInfo $info) {
                             return $info->getContainer()->get('user')['name'];
                         }
                     ]
@@ -611,7 +611,7 @@ 
 
     public function testComplexityReducer()
     {
-        $schema    = new Schema(
+        $schema = new Schema(
             [
                 'query' => new ObjectType(
                     [
@@ -627,7 +627,7 @@ 
                                                 'args'    => [
                                                     'shorten' => new BooleanType()
                                                 ],
-                                                'resolve' => function ($value, $args) {
+                                                'resolve' => function($value, $args) {
                                                     return empty($args['shorten']) ? $value['firstName'] : $value['firstName'];
                                                 }
                                             ],
@@ -636,19 +636,19 @@ 
                                             'likes'     => [
                                                 'type'    => new IntType(),
                                                 'cost'    => 10,
-                                                'resolve' => function () {
+                                                'resolve' => function() {
                                                     return 42;
                                                 }
                                             ]
                                         ]
                                     ]
                                 ),
-                                'cost'    => function ($args, $context, $childCost) {
+                                'cost'    => function($args, $context, $childCost) {
                                     $argsCost = isset($args['cost']) ? $args['cost'] : 1;
 
                                     return 1 + $argsCost * $childCost;
                                 },
-                                'resolve' => function ($value, $args) {
+                                'resolve' => function($value, $args) {
                                     $data = ['firstName' => 'John', 'code' => '007'];
 
                                     return $data;

src/Type/InputObject/AbstractInputObjectType.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,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;
         });
 
@@ -81,7 +81,7 @@ 
 
     public function parseValue($value)
     {
-        if($value instanceof InputObject) {
+        if ($value instanceof InputObject) {
             $value = $value->getValue();
         }
 

src/Execution/Processor.php

Spacing +1 added lines, -1 removed lines

@@ -375,7 +375,7 @@
         $result = [];
         foreach ($resolvedValue as $resolvedValueItem) {
             try {
-                $fakeField->getConfig()->set('resolve', function () use ($resolvedValueItem) {
+                $fakeField->getConfig()->set('resolve', function() use ($resolvedValueItem) {
                     return $resolvedValueItem;
                 });
 

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

Tests/Library/Config/ConfigTest.php

Spacing +3 added lines, -3 removed lines

@@ -80,7 +80,7 @@ 
             ]
         ]);
 
-        $finalConfig = new TestConfig(['name' => $name . 'final', 'resolve' => function () { return []; }], $object, true);
+        $finalConfig = new TestConfig(['name' => $name.'final', 'resolve' => function() { return []; }], $object, true);
         $this->assertEquals($finalConfig->getType(), null);
 
         $rules['resolve']['required'] = true;
@@ -100,7 +100,7 @@ 
      */
     public function testFinalRule()
     {
-        ConfigValidator::getInstance()->assertValidConfig(new TestConfig(['name' => 'Test' . 'final'], null, true));
+        ConfigValidator::getInstance()->assertValidConfig(new TestConfig(['name' => 'Test'.'final'], null, true));
     }
 
     /**
@@ -127,7 +127,7 @@ 
                 ]
             ]
         ]);
-        $object   = new ObjectType([
+        $object = new ObjectType([
             'name' => 'Project',
             'fields' => [
                 'id' => new IdType(),

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

Doc Comments +1 added lines, -1 removed lines

@@ -294,7 +294,7 @@
     }
 
     /**
-     * @return array|InputList|InputObject|Literal|VariableReference
+     * @return Ast\Interfaces\ValueInterface
      *
      * @throws SyntaxErrorException
      */