Youshido / GraphQL

Tests/Library/Config/InterfaceTypeConfigTest.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
     public function testConfigNoFields()
     {
         ConfigValidator::getInstance()->assertValidateConfig(
-            new InterfaceTypeConfig(['name' => 'Test', 'resolveType' => function () { }], null, true)
+            new InterfaceTypeConfig(['name' => 'Test', 'resolveType' => function() { }], null, true)
         );
     }
 
@@ -59,11 +59,11 @@ 
         $interfaceConfig = new InterfaceTypeConfig([
             'name'        => 'Test',
             'fields'      => ['id' => new IntType()],
-            'resolveType' => function ($object) {
+            'resolveType' => function($object) {
                 return $object->getType();
             }
         ], null, true);
-        $object          = new ObjectType(['name' => 'User', 'fields' => ['name' => new StringType()]]);
+        $object = new ObjectType(['name' => 'User', 'fields' => ['name' => new StringType()]]);
 
         $this->assertEquals($interfaceConfig->getName(), 'Test');
         $this->assertEquals($interfaceConfig->resolveType($object), $object->getType());

Tests/Library/Config/ConfigTest.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,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;
@@ -98,7 +98,7 @@ 
      */
     public function testFinalRule()
     {
-        ConfigValidator::getInstance()->assertValidateConfig(new TestConfig(['name' => 'Test' . 'final'], null, true));
+        ConfigValidator::getInstance()->assertValidateConfig(new TestConfig(['name' => 'Test'.'final'], null, true));
     }
 
     /**

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)

src/Config/Traits/ResolvableObjectTrait.php

Doc Comments +3 added lines

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

Indentation +15 added lines, -15 removed lines

@@ -13,34 +13,34 @@
 
 trait ResolvableObjectTrait {
 
-  public function resolve($value, array $args, ResolveInfo $info)
-  {
+    public function resolve($value, array $args, ResolveInfo $info)
+    {
     if ($this->resolveFunctionCache === null) {
-      $this->resolveFunctionCache = $this->getConfig()->getResolveFunction();
+        $this->resolveFunctionCache = $this->getConfig()->getResolveFunction();
 
-      if (!$this->resolveFunctionCache) {
+        if (!$this->resolveFunctionCache) {
         $this->resolveFunctionCache = false;
-      }
+        }
     }
     if ($this->resolveFunctionCache) {
-      $resolveFunction = $this->resolveFunctionCache;
+        $resolveFunction = $this->resolveFunctionCache;
 
-      return $resolveFunction($value, $args, $info);
+        return $resolveFunction($value, $args, $info);
     } else {
-      if (is_array($value) && array_key_exists($this->getName(), $value)) {
+        if (is_array($value) && array_key_exists($this->getName(), $value)) {
         return $value[$this->getName()];
-      } elseif (is_object($value)) {
+        } elseif (is_object($value)) {
         return TypeService::getPropertyValue($value, $this->getName());
-      } elseif ($this->getType()->getNamedType()->getKind() == TypeMap::KIND_SCALAR) {
+        } elseif ($this->getType()->getNamedType()->getKind() == TypeMap::KIND_SCALAR) {
         return null;
-      } else {
+        } else {
         throw new \Exception(sprintf('Property "%s" not found in resolve result', $this->getName()));
-      }
+        }
+    }
     }
-  }
 
 
-  public function getResolveFunction() {
+    public function getResolveFunction() {
     return $this->getConfig()->getResolveFunction();
-  }
+    }
 }

src/Execution/Request.php

Doc Comments +4 added lines, -1 removed lines

@@ -93,7 +93,7 @@ 
      */
 
     /**
-     * @param $name
+     * @param string $name
      *
      * @return Field
      */
@@ -125,6 +125,9 @@ 
         return $this->fields;
     }
 
+    /**
+     * @param string $name
+     */
     public function removeField($name)
     {
         if ($this->hasField($name)) {

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

src/Config/Schema/SchemaConfig.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     }
 
     /**
-     * @param $query AbstractObjectType
+     * @param \Youshido\Tests\StarWars\Schema\StarWarsQueryType $query AbstractObjectType
      *
      * @return SchemaConfig
      */

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;