Youshido / GraphQL

Tests/Schema/IntrospectionTest.php

Spacing +7 added lines, -7 removed lines

@@ -136,7 +136,7 @@ 
             'description'       => 'latest description',
             'deprecationReason' => 'for test',
             'isDeprecated'      => true,
-            'resolve'           => function () {
+            'resolve'           => function() {
                 return [
                     'id'   => 1,
                     'name' => 'Alex'
@@ -201,9 +201,9 @@ 
                                 ['name' => 'String', 'fields' => null],
                                 ['name' => '__Schema', 'fields' => [['name' => 'queryType', 'args' => []], ['name' => 'mutationType', 'args' => []], ['name' => 'subscriptionType', 'args' => []], ['name' => 'types', 'args' => []], ['name' => 'directives', 'args' => []]]],
                                 ['name' => '__Type', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'kind', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'ofType', 'args' => []], ['name' => 'inputFields', 'args' => []], ['name' => 'enumValues', 'args' => [['defaultValue' => 'false']]], ['name' => 'fields', 'args' => [['defaultValue' => 'false']]], ['name' => 'interfaces', 'args' => []], ['name' => 'possibleTypes', 'args' => []]]],
-                                ['name' => '__InputValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'type', 'args' => []], ['name' => 'defaultValue', 'args' => []],]],
+                                ['name' => '__InputValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'type', 'args' => []], ['name' => 'defaultValue', 'args' => []], ]],
                                 ['name' => 'Boolean', 'fields' => null],
-                                ['name' => '__EnumValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'deprecationReason', 'args' => []], ['name' => 'isDeprecated', 'args' => []],]],
+                                ['name' => '__EnumValue', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'deprecationReason', 'args' => []], ['name' => 'isDeprecated', 'args' => []], ]],
                                 ['name' => '__Field', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'isDeprecated', 'args' => []], ['name' => 'deprecationReason', 'args' => []], ['name' => 'type', 'args' => []], ['name' => 'args', 'args' => []]]],
                                 ['name' => '__Subscription', 'fields' => [['name' => 'name', 'args' => []]]],
                                 ['name' => '__Directive', 'fields' => [['name' => 'name', 'args' => []], ['name' => 'description', 'args' => []], ['name' => 'args', 'args' => []], ['name' => 'onOperation', 'args' => []], ['name' => 'onFragment', 'args' => []], ['name' => 'onField', 'args' => []]]],
@@ -291,7 +291,7 @@ 
                 'id'   => ['type' => new IntType()],
                 'name' => ['type' => new IntType()],
             ],
-            'resolveType' => function ($type) {
+            'resolveType' => function($type) {
 
             }
         ]);
@@ -319,7 +319,7 @@ 
         $unionType = new UnionType([
             'name'        => 'UnionType',
             'types'       => [$object1, $object2],
-            'resolveType' => function () {
+            'resolveType' => function() {
 
             }
         ]);
@@ -330,7 +330,7 @@ 
             'args'    => [
                 'id' => ['type' => TypeMap::TYPE_INT]
             ],
-            'resolve' => function () {
+            'resolve' => function() {
                 return [
                     'id'   => 1,
                     'name' => 'Alex'
@@ -356,7 +356,7 @@ 
                     ]
                 ])
             ],
-            'resolve' => function () {
+            'resolve' => function() {
                 return null;
             }
         ]));

src/Type/Object/AbstractObjectType.php

Doc Comments +3 added lines

@@ -87,6 +87,9 @@
         return $this->getConfigValue('interfaces', []);
     }
 
+    /**
+     * @param ObjectType|null $value
+     */
     public function isValidValue($value)
     {
         return is_array($value) || is_null($value) || is_object($value);

src/Config/AbstractConfig.php

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
         $validator = ConfigValidator::getInstance();
 
         if (!$validator->validate($this->data, $this->getContextRules(), $this->extraFieldsAllowed)) {
-            throw new ConfigurationException('Config is not valid for ' . ($this->contextObject ? get_class($this->contextObject) : null) . "\n" . implode("\n", $validator->getErrorsArray(false)));
+            throw new ConfigurationException('Config is not valid for '.($this->contextObject ? get_class($this->contextObject) : null)."\n".implode("\n", $validator->getErrorsArray(false)));
         }
     }
 
@@ -151,7 +151,7 @@ 
         } elseif (substr($method, 0, 2) == 'is') {
             $propertyName = lcfirst(substr($method, 2));
         } else {
-            throw new \Exception('Call to undefined method ' . $method);
+            throw new \Exception('Call to undefined method '.$method);
         }
 
         return $this->get($propertyName);

src/Execution/Container/Container.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         if (isset($this->values[$id])) {
             unset($this->values[$id]);
         }
-        $this->keyset[$id]   = true;
+        $this->keyset[$id] = true;
     }
 
     public function remove($id)

Tests/StarWars/Schema/StarWarsQueryType.php

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;

src/Field/AbstractInputField.php

Unused Use Statements -2 removed lines

@@ -9,8 +9,6 @@
 
 
 use Youshido\GraphQL\Config\Field\InputFieldConfig;
-use Youshido\GraphQL\Config\Traits\ResolvableObjectTrait;
-use Youshido\GraphQL\Type\InputObject\AbstractInputObjectType;
 use Youshido\GraphQL\Type\InputTypeInterface;
 use Youshido\GraphQL\Type\Traits\AutoNameTrait;
 use Youshido\GraphQL\Type\Traits\FieldsArgumentsAwareObjectTrait;

src/Schema/AbstractSchema.php

Indentation +1 added lines, -1 removed lines

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

Unused Use Statements -2 removed lines

@@ -10,9 +10,7 @@
 
 
 use Youshido\GraphQL\Config\Schema\SchemaConfig;
-use Youshido\GraphQL\Type\Object\ObjectType;
 use Youshido\GraphQL\Type\SchemaTypesList;
-use Youshido\GraphQL\Type\TypeInterface;
 
 abstract class AbstractSchema
 {

Spacing +2 added lines, -2 removed lines

@@ -23,10 +23,10 @@
     public function __construct($config = [])
     {
         if (!array_key_exists('query', $config)) {
-            $config['query'] = new InternalSchemaQueryObject(['name' => $this->getName() . 'Query']);
+            $config['query'] = new InternalSchemaQueryObject(['name' => $this->getName().'Query']);
         }
         if (!array_key_exists('mutation', $config)) {
-            $config['mutation'] = new InternalSchemaMutationObject(['name' => $this->getName() . 'Mutation']);
+            $config['mutation'] = new InternalSchemaMutationObject(['name' => $this->getName().'Mutation']);
         }
         if (!array_key_exists('types', $config)) {
           $config['types'] = [];

src/Relay/Connection/Connection.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
         $edgeFields = !empty($config['edgeFields']) ? $config['edgeFields'] : [];
 
         $edgeType = new ObjectType([
-            'name'        => $name . 'Edge',
+            'name'        => $name.'Edge',
             'description' => 'An edge in a connection.',
             'fields'      => array_merge([
                 'node'   => [
@@ -84,7 +84,7 @@ 
         $connectionFields = !empty($config['connectionFields']) ? $config['connectionFields'] : [];
 
         $connectionType = new ObjectType([
-            'name'        => $name . 'Connection',
+            'name'        => $name.'Connection',
             'description' => 'A connection to a list of items.',
             'fields'      => array_merge([
                 'pageInfo' => [

src/Type/SchemaTypesList.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         if (!is_array($types)) {
             throw new \Exception('addTypes accept only array of types');
         }
-        foreach($types as $type) {
+        foreach ($types as $type) {
             $this->addType($type);
         }
         return $this;

Braces +6 added lines, -2 removed lines

@@ -42,7 +42,9 @@ 
     public function addType(TypeInterface $type)
     {
         $typeName = $this->getTypeName($type);
-        if ($this->isTypeNameRegistered($typeName)) return $this;
+        if ($this->isTypeNameRegistered($typeName)) {
+            return $this;
+        }
 
         $this->typesList[$typeName] = $type;
         return $this;
@@ -54,7 +56,9 @@ 
     }
 
     private function getTypeName($type) {
-        if (is_string($type)) return $type;
+        if (is_string($type)) {
+            return $type;
+        }
         if (is_object($type) && $type instanceof AbstractType) {
             return $type->getName();
         }