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

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
             $config['type'] = $this->getType();
             $config['name'] = $this->getName();
             if (empty($config['name'])) {
-                $config['name'] =$this->getAutoName();
+                $config['name'] = $this->getAutoName();
             }
         }
 

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

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