Youshido / GraphQL

Tests/Parser/RequestTest.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
     public function testSetVariableParseJson()
     {
         $variables = '{"foo": "bar"}';
-        $expectedVariableArray = [ 'foo' => 'bar' ];
+        $expectedVariableArray = ['foo' => 'bar'];
 
         $request = new Request([], $variables);
         $this->assertEquals($expectedVariableArray, $request->getVariables());

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/Introspection/Field/TypesField.php

Indentation +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         }
 
         foreach ($value->getTypes() as $type) {
-          $this->collectTypes($type);
+            $this->collectTypes($type);
         }
 
         return array_values($this->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/InputObject/AbstractInputObjectType.php

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
 
     public function isValidValue($value)
     {
-        if($value instanceof InputObject) {
+        if ($value instanceof InputObject) {
             $value = $value->getValue();
         }
 
@@ -51,12 +51,12 @@ 
         }
 
         $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;
         });
 
         foreach ($value as $valueKey => $valueItem) {
-            if($valueItem instanceof Variable) {
+            if ($valueItem instanceof Variable) {
                 $valueItem = $valueItem->getValue();
             }
 
@@ -86,7 +86,7 @@ 
     {
         $typeConfig = $this->getConfig();
         foreach ($value as $valueKey => $item) {
-            if($item instanceof Variable) {
+            if ($item instanceof Variable) {
                 $item = $item->getValue();
             }
 

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

Doc Comments +6 added lines

@@ -48,11 +48,17 @@
         return $this;
     }
 
+    /**
+     * @param string $typeName
+     */
     public function isTypeNameRegistered($typeName)
     {
         return (isset($this->typesList[$typeName]));
     }
 
+    /**
+     * @param TypeInterface $type
+     */
     private function getTypeName($type) {
         if (is_string($type)) return $type;
         if (is_object($type) && $type instanceof AbstractType) {

examples/03_relay_star_wars/schema-bootstrap.php

Spacing +5 added lines, -5 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-require_once __DIR__ . '/../../../../../vendor/autoload.php';
-require_once __DIR__ . '/Schema/FactionType.php';
-require_once __DIR__ . '/Schema/ShipType.php';
-require_once __DIR__ . '/Schema/TestDataProvider.php';
-require_once __DIR__ . '/Schema/StarWarsRelaySchema.php';
+require_once __DIR__.'/../../../../../vendor/autoload.php';
+require_once __DIR__.'/Schema/FactionType.php';
+require_once __DIR__.'/Schema/ShipType.php';
+require_once __DIR__.'/Schema/TestDataProvider.php';
+require_once __DIR__.'/Schema/StarWarsRelaySchema.php';

examples/02_blog/schema-bootstrap.php

Spacing +10 added lines, -10 removed lines

@@ -2,13 +2,13 @@
 
 namespace BlogTest;
 
-require_once __DIR__ . '/../../../../../vendor/autoload.php';
-require_once __DIR__ . '/Schema/DataProvider.php';
-require_once __DIR__ . '/Schema/PostType.php';
-require_once __DIR__ . '/Schema/PostStatus.php';
-require_once __DIR__ . '/Schema/ContentBlockInterface.php';
-require_once __DIR__ . '/Schema/LikePostField.php';
-require_once __DIR__ . '/Schema/BannerType.php';
-require_once __DIR__ . '/Schema/ContentBlockUnion.php';
-require_once __DIR__ . '/Schema/PostInputType.php';
-require_once __DIR__ . '/Schema/BlogSchema.php';
+require_once __DIR__.'/../../../../../vendor/autoload.php';
+require_once __DIR__.'/Schema/DataProvider.php';
+require_once __DIR__.'/Schema/PostType.php';
+require_once __DIR__.'/Schema/PostStatus.php';
+require_once __DIR__.'/Schema/ContentBlockInterface.php';
+require_once __DIR__.'/Schema/LikePostField.php';
+require_once __DIR__.'/Schema/BannerType.php';
+require_once __DIR__.'/Schema/ContentBlockUnion.php';
+require_once __DIR__.'/Schema/PostInputType.php';
+require_once __DIR__.'/Schema/BlogSchema.php';