Youshido / GraphQL

src/Type/TypeFactory.php

Spacing +2 added lines, -2 removed lines

@@ -34,13 +34,13 @@
                 $name = $name == 'Datetime' ? 'DateTime' : $name;
                 $name = $name == 'Datetimetz' ? 'DateTimeTz' : $name;
 
-                $className                = 'Youshido\GraphQL\Type\Scalar\\' . $name . 'Type';
+                $className                = 'Youshido\GraphQL\Type\Scalar\\'.$name.'Type';
                 self::$objectsHash[$type] = new $className();
             }
 
             return self::$objectsHash[$type];
         } else {
-            throw new ConfigurationException('Configuration problem with type ' . $type);
+            throw new ConfigurationException('Configuration problem with type '.$type);
         }
     }
 

src/Introspection/Field/TypesField.php

Indentation +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
         }
 
         foreach ($value->getTypesList()->getTypes() as $type) {
-          $this->collectTypes($type);
+            $this->collectTypes($type);
         }
 
         return array_values($this->types);

src/Introspection/Traits/TypeCollectorTrait.php

Braces +3 added lines, -1 removed lines

@@ -20,7 +20,9 @@
 
     protected function collectTypes(AbstractType $type)
     {
-        if (is_object($type) && array_key_exists($type->getName(), $this->types)) return;
+        if (is_object($type) && array_key_exists($type->getName(), $this->types)) {
+            return;
+        }
 
         switch ($type->getKind()) {
             case TypeMap::KIND_INTERFACE:

src/Parser/Ast/ArgumentValue/Variable.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
     public function getValue()
     {
         if (null === $this->value) {
-            throw new \LogicException('Value is not set for variable "' . $this->name . '"');
+            throw new \LogicException('Value is not set for variable "'.$this->name.'"');
         }
 
         return $this->value;

src/Validator/ResolveValidator/ResolveValidator.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
     public function assertValidArguments(FieldInterface $field, AstFieldInterface $query, Request $request)
     {
-        $requiredArguments = array_filter($field->getArguments(), function (InputField $argument) {
+        $requiredArguments = array_filter($field->getArguments(), function(InputField $argument) {
             return $argument->getType()->getKind() === TypeMap::KIND_NON_NULL;
         });
 

Tests/Issues/Issue99/Issue99Test.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
         self::assertTrue(isset($res['data']['items']));
 
-        foreach($res['data']['items'] as $item) {
+        foreach ($res['data']['items'] as $item) {
             self::assertTrue(isset($item['custom']['value']));
             self::assertEquals(self::BUG_NOT_EXISTS_VALUE, $item['custom']['value']);
         }

Tests/Issues/Issue99/Issue99Schema.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
                         'args'    => [
                             'example' => new StringType()
                         ],
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return [
                                 ['id' => 1],
                                 ['id' => 2],

src/Execution/Request.php

Doc Comments +4 added lines, -1 removed lines

@@ -223,6 +223,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $name
+     */
     public function getVariable($name)
     {
         return $this->hasVariable($name) ? $this->variables[$name] : null;
@@ -266,7 +269,7 @@ 
     }
 
     /**
-     * @return array|VariableReference[]
+     * @return VariableReference[]
      */
     public function getVariableReferences()
     {

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 namespace Youshido\GraphQL\Execution;
 
 
-use Youshido\GraphQL\Parser\Ast\ArgumentValue\Literal;
 use Youshido\GraphQL\Parser\Ast\ArgumentValue\Variable;
 use Youshido\GraphQL\Parser\Ast\ArgumentValue\VariableReference;
 use Youshido\GraphQL\Parser\Ast\Fragment;

Spacing +1 added lines, -1 removed lines

@@ -208,7 +208,7 @@
         }
 
         $this->variables = $variables;
-        foreach($this->variableReferences as $reference) {
+        foreach ($this->variableReferences as $reference) {
             /** invalid request with no variable */
             if (!$reference->getVariable()) continue;
             $variableName = $reference->getVariable()->getName();

Braces +6 added lines, -2 removed lines

@@ -210,11 +210,15 @@
         $this->variables = $variables;
         foreach($this->variableReferences as $reference) {
             /** invalid request with no variable */
-            if (!$reference->getVariable()) continue;
+            if (!$reference->getVariable()) {
+                continue;
+            }
             $variableName = $reference->getVariable()->getName();
 
             /** no variable was set at the time */
-            if (!array_key_exists($variableName, $variables)) continue;
+            if (!array_key_exists($variableName, $variables)) {
+                continue;
+            }
 
             $reference->getVariable()->setValue($variables[$variableName]);
             $reference->setValue($variables[$variableName]);

Tests/Issues/Issue109/Issue109Schema.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
                                 ]
                             ]
                         ]),
-                        'resolve' => function ($source, array $args, ResolveInfo $info) {
+                        'resolve' => function($source, array $args, ResolveInfo $info) {
                             $internalArgs = [
                                 'comment_id' => 200
                             ];