Youshido / GraphQL

src/Type/InterfaceType/AbstractInterfaceType.php

Doc Comments +3 added lines

@@ -45,6 +45,9 @@
         $this->config = new InterfaceTypeConfig($config, $this);
     }
 
+    /**
+     * @return AbstractType
+     */
     abstract public function resolveType($object);
 
     /**

Braces +3 added lines, -1 removed lines

@@ -59,7 +59,9 @@
     {
         if ($value instanceof AbstractObjectType) {
             foreach($value->getInterfaces() as $interface) {
-                if ($interface instanceof $this) return true;
+                if ($interface instanceof $this) {
+                    return true;
+                }
             }
         }
         return false;

src/Type/InterfaceType/InterfaceType.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
     {
     }
 
+    /**
+     * @param \Youshido\GraphQL\Type\Object\ObjectType $object
+     */
     public function resolveType($object)
     {
         return $this->getConfig()->resolveType($object);

src/Type/NonNullType.php

Doc Comments +3 added lines

@@ -50,6 +50,9 @@
         return $value;
     }
 
+    /**
+     * @param null|Scalar\StringType $value
+     */
     public function isValidValue($value)
     {
         return $value !== null;

src/Type/TypeService.php

Doc Comments +3 added lines

@@ -117,6 +117,9 @@
         return $type instanceof AbstractInputObjectType;
     }
 
+    /**
+     * @return \Youshido\GraphQL\Execution\ResolveInfo
+     */
     public static function getPropertyValue($data, $path)
     {
         if (is_object($data)) {

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
         if (is_object($data)) {
             $getter = $path;
             if (substr($path, 0, 2) != 'is') {
-                $getter = 'get' . self::classify($path);
+                $getter = 'get'.self::classify($path);
             }
 
             return is_callable([$data, $getter]) ? $data->$getter() : null;

src/Validator/SchemaValidator/SchemaValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      * @param Field                 $objField
      * @param AbstractInterfaceType $interface
      *
-     * @return bool
+     * @return boolean|null
      *
      * @throws ConfigurationException
      */

Tests/DataProvider/TestField.php

Doc Comments +4 added lines, -1 removed lines

@@ -16,13 +16,16 @@
 {
 
     /**
-     * @return AbstractObjectType
+     * @return IntType
      */
     public function getType()
     {
         return new IntType();
     }
 
+    /**
+     * @param string $value
+     */
     public function resolve($value, array $args, ResolveInfo $info)
     {
         return $value;

Tests/DataProvider/TestInputField.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 {
 
     /**
-     * @return AbstractInputObjectType
+     * @return IntType
      */
     public function getType()
     {

Tests/StarWars/Schema/QueryType.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 
 use Youshido\GraphQL\Field\Field;
-use Youshido\GraphQL\Field\FieldFactory;
 use Youshido\GraphQL\Type\Object\AbstractObjectType;
 use Youshido\GraphQL\Type\Scalar\IdType;
 

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
                 'args'    => [
                     'episode' => ['type' => new EpisodeEnum()]
                 ],
-                'resolve' => function ($root, $args) {
+                'resolve' => function($root, $args) {
                     return StarWarsData::getHero(isset($args['episode']) ? $args['episode'] : null);
                 },
             ])
@@ -46,7 +46,7 @@ 
                 'args'    => [
                     'id' => new IdType()
                 ],
-                'resolve' => function ($value = null, $args = []) {
+                'resolve' => function($value = null, $args = []) {
                     $humans = StarWarsData::humans();
 
                     return isset($humans[$args['id']]) ? $humans[$args['id']] : null;
@@ -58,7 +58,7 @@ 
                 'args'    => [
                     'id' => new IdType()
                 ],
-                'resolve' => function ($value = null, $args = []) {
+                'resolve' => function($value = null, $args = []) {
                     $droids = StarWarsData::droids();
 
                     return isset($droids[$args['id']]) ? $droids[$args['id']] : null;

Tests/Schema/ProcessorTest.php

Indentation +13 added lines, -13 removed lines

@@ -164,19 +164,19 @@
         $this->assertEquals(['data' => ['me' => ['firstName' => 'JOHN']]], $processor->getResponseData());
 
         $schema->getMutationType()
-               ->addField(new Field([
-                   'name'    => 'increaseCounter',
-                   'type'    => new IntType(),
-                   'resolve' => function ($value, $args, ResolveInfo $info) {
-                       return $this->_counter += $args['amount'];
-                   },
-                   'args'    => [
-                       'amount' => [
-                           'type'    => new IntType(),
-                           'default' => 1
-                       ]
-                   ]
-               ]))->addField(new Field([
+                ->addField(new Field([
+                    'name'    => 'increaseCounter',
+                    'type'    => new IntType(),
+                    'resolve' => function ($value, $args, ResolveInfo $info) {
+                        return $this->_counter += $args['amount'];
+                    },
+                    'args'    => [
+                        'amount' => [
+                            'type'    => new IntType(),
+                            'default' => 1
+                        ]
+                    ]
+                ]))->addField(new Field([
                 'name'    => 'invalidResolveTypeMutation',
                 'type'    => new NonNullType(new IntType()),
                 'resolve' => function () {

Spacing +22 added lines, -22 removed lines

@@ -96,7 +96,7 @@ 
 
     public function testSchemaOperations()
     {
-        $schema    = new Schema([
+        $schema = new Schema([
             'query' => new ObjectType([
                 'name'   => 'RootQuery',
                 'fields' => [
@@ -109,7 +109,7 @@ 
                                     'args'    => [
                                         'shorten' => new BooleanType()
                                     ],
-                                    'resolve' => function ($value, $args) {
+                                    'resolve' => function($value, $args) {
                                         return empty($args['shorten']) ? $value : $value;
                                     }
                                 ],
@@ -117,7 +117,7 @@ 
                                 'code'      => new StringType(),
                             ]
                         ]),
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             $data = ['firstName' => 'John', 'code' => '007'];
                             if (!empty($args['upper'])) {
                                 foreach ($data as $key => $value) {
@@ -136,13 +136,13 @@ 
                     ],
                     'randomUser'        => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['invalidField' => 'John'];
                         }
                     ],
                     'invalidValueQuery' => [
                         'type'    => new TestObjectType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return 'stringValue';
                         }
                     ],
@@ -167,7 +167,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'    => [
@@ -179,13 +179,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'];
                 }
             ]));
@@ -231,43 +231,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'];
                         }
                     ],
@@ -279,7 +279,7 @@ 
                                 'enum'   => new TestEnumType(),
                             ],
                         ]),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 'object' => [
                                     'name' => 'John'
@@ -348,10 +348,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;
                 }
@@ -362,11 +362,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' => [
@@ -375,7 +375,7 @@ 
                         'args'    => [
                             'type' => ['type' => 'string']
                         ],
-                        'resolve' => function ($value, $args) {
+                        'resolve' => function($value, $args) {
                             if ($args['type'] == 'object1') {
                                 return [
                                     'id' => 43
@@ -389,7 +389,7 @@ 
                     ],
                     'invalidUnion' => [
                         'type'    => $invalidUnion,
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return ['name' => 'name resolved'];
                         }
                     ],