Youshido / GraphQL

Tests/DataProvider/TestObjectType.php

Indentation +15 added lines, -15 removed lines

@@ -30,21 +30,21 @@
                 ],
             ]))
             ->addField('location', [
-                 'type'    => new ObjectType(
-                     [
-                         'name'   => 'Location',
-                         'fields' => [
-                             'address'    => new StringType()
-                         ]
-                     ]
-                 ),
-                 'args'    => [
-                     'noop' => new IntType()
-                 ],
-                 'resolve' => function ($value, $args, $info) {
-                   return ['address' => '1234 Street'];
-                 }
-             ]
+                    'type'    => new ObjectType(
+                        [
+                            'name'   => 'Location',
+                            'fields' => [
+                                'address'    => new StringType()
+                            ]
+                        ]
+                    ),
+                    'args'    => [
+                        'noop' => new IntType()
+                    ],
+                    'resolve' => function ($value, $args, $info) {
+                    return ['address' => '1234 Street'];
+                    }
+                ]
             )
             ->addField(
                 'echo', [

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
                  'args'    => [
                      'noop' => new IntType()
                  ],
-                 'resolve' => function ($value, $args, $info) {
+                 'resolve' => function($value, $args, $info) {
                    return ['address' => '1234 Street'];
                  }
              ]
@@ -52,7 +52,7 @@ 
                     'args'    => [
                         'value' => new NonNullType(new StringType())
                     ],
-                    'resolve' => function ($value, $args, $info) {
+                    'resolve' => function($value, $args, $info) {
                         return $args['value'];
                     }
                 ]

Tests/Library/Utilities/TypeUtilitiesTest.php

Spacing +1 added lines, -1 removed lines

@@ -61,6 +61,6 @@
         $arrayData = (new TestObjectType())->getData();
 
         $this->assertEquals('John', TypeService::getPropertyValue($arrayData, 'name'));
-        $this->assertEquals('John', TypeService::getPropertyValue((object) $arrayData, 'name'));
+        $this->assertEquals('John', TypeService::getPropertyValue((object)$arrayData, 'name'));
     }
 }

src/Execution/Visitor/MaxComplexityQueryVisitor.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     }
     $this->memo += $cost ?: $this->defaultScore;
     if ($this->memo > $this->maxScore) {
-      throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
+      throw new \Exception('query exceeded max allowed complexity of '.$this->maxScore);
     }
     return $this->memo;
   }

Indentation +24 added lines, -24 removed lines

@@ -16,40 +16,40 @@
 
 class MaxComplexityQueryVisitor extends AbstractQueryVisitor {
 
-  /**
-   * @var int max score allowed before throwing an exception (causing processing to stop)
-   */
-  public $maxScore;
-
-  /**
-   * @var int default score for nodes without explicit cost functions
-   */
-  protected $defaultScore = 1;
-
-  /**
-   * MaxComplexityQueryVisitor constructor.
-   *
-   * @param int $max max allowed complexity score
-   */
-  public function __construct($max) {
+    /**
+     * @var int max score allowed before throwing an exception (causing processing to stop)
+     */
+    public $maxScore;
+
+    /**
+     * @var int default score for nodes without explicit cost functions
+     */
+    protected $defaultScore = 1;
+
+    /**
+     * MaxComplexityQueryVisitor constructor.
+     *
+     * @param int $max max allowed complexity score
+     */
+    public function __construct($max) {
     parent::__construct();
 
     $this->maxScore = $max;
-  }
+    }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) {
+    /**
+     * {@inheritdoc}
+     */
+    public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0) {
     $cost = $fieldConfig->get('cost');
     if (is_callable($cost)) {
-      $cost = $cost($args, $fieldConfig, $childScore);
+        $cost = $cost($args, $fieldConfig, $childScore);
     }
     $cost = $cost ?: $this->defaultScore;
     $this->memo += $cost;
     if ($this->memo > $this->maxScore) {
-      throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
+        throw new \Exception('query exceeded max allowed complexity of ' . $this->maxScore);
     }
     return $cost;
-  }
+    }
 }
\ No newline at end of file

src/Type/TypeFactory.php

Spacing +2 added lines, -2 removed lines

@@ -35,13 +35,13 @@
             if (empty(self::$objectsHash[$type])) {
                 $name = ucfirst($type);
 
-                $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);
         }
     }
 

Tests/Library/Type/ScalarExtendTypeTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,13 +27,13 @@
                 'title' => new StringType(),
             ]
         ]);
-        $processor  = new Processor(new Schema([
+        $processor = new Processor(new Schema([
                 'query' => new ObjectType([
                     'name'   => 'RootQueryType',
                     'fields' => [
                         'latestReport' => [
                             'type'    => $reportType,
-                            'resolve' => function () {
+                            'resolve' => function() {
                                 return [
                                     'title' => 'Accident #1',
                                     'time'  => '13:30:12',

src/Execution/Processor.php

Braces +3 added lines, -1 removed lines

@@ -168,7 +168,9 @@
         $value = [];
 
         if ($fieldType->getKind() == TypeMap::KIND_LIST) {
-            if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) return null;
+            if (!$this->resolveValidator->hasArrayAccess($resolvedValue)) {
+                return null;
+            }
             foreach ($resolvedValue as $resolvedValueItem) {
                 $value[] = [];
                 $index   = count($value) - 1;

Doc Comments +6 added lines, -3 removed lines

@@ -104,7 +104,7 @@ 
     }
 
     /**
-     * @param Query|Field        $query
+     * @param Query        $query
      * @param AbstractObjectType $currentLevelSchema
      * @return array|bool|mixed
      */
@@ -236,6 +236,9 @@ 
         return $value;
     }
 
+    /**
+     * @param FieldInterface $field
+     */
     protected function createResolveInfo($field, $fields)
     {
         return new ResolveInfo($field, $fields, $this->executionContext);
@@ -284,7 +287,7 @@ 
     }
 
     /**
-     * @param $query         Query|FragmentInterface
+     * @param Query $query         Query|FragmentInterface
      * @param $queryType     AbstractObjectType|TypeInterface|Field|AbstractType
      * @param $resolvedValue mixed
      * @param $value         array
@@ -462,7 +465,7 @@ 
      * Fragments (anonymous and named), and Fields.  The core of the function is simple: recurse until we hit the base
      * case of a Field and yield that back up to the visitor up in `doVisit`.
      *
-     * @param Query|Field|FragmentInterface $queryNode
+     * @param Query $queryNode
      * @param FieldInterface                 $currentLevelAST
      *
      * @return \Generator

src/Execution/Visitor/AbstractQueryVisitor.php

Indentation +28 added lines, -28 removed lines

@@ -29,38 +29,38 @@
 
 abstract class AbstractQueryVisitor {
 
-  /**
-   * @var int initial value of $this->memo
-   */
-  protected $initialValue = 0;
+    /**
+     * @var int initial value of $this->memo
+     */
+    protected $initialValue = 0;
 
-  /**
-   * @var mixed the accumulator
-   */
-  protected $memo;
+    /**
+     * @var mixed the accumulator
+     */
+    protected $memo;
 
-  /**
-   * AbstractQueryVisitor constructor.
-   */
-  public function __construct() {
+    /**
+     * AbstractQueryVisitor constructor.
+     */
+    public function __construct() {
     $this->memo = $this->initialValue;
-  }
+    }
 
-  /**
-   * @return mixed getter for the memo, in case callers want to inspect it after a process run
-   */
-  public function getMemo() {
+    /**
+     * @return mixed getter for the memo, in case callers want to inspect it after a process run
+     */
+    public function getMemo() {
     return $this->memo;
-  }
+    }
 
-  /**
-   * Visit a query node.  See class docstring.
-   *
-   * @param array       $args
-   * @param FieldConfig $fieldConfig
-   * @param int         $childScore
-   *
-   * @return int|null
-   */
-  abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0);
+    /**
+     * Visit a query node.  See class docstring.
+     *
+     * @param array       $args
+     * @param FieldConfig $fieldConfig
+     * @param int         $childScore
+     *
+     * @return int|null
+     */
+    abstract public function visit(array $args, FieldConfig $fieldConfig, $childScore = 0);
 }
\ No newline at end of file

Tests/Performance/LoadTest.php

Spacing +3 added lines, -3 removed lines

@@ -43,12 +43,12 @@ 
             $authors = [];
             while (count($authors) < rand(1, 4)) {
                 $authors[] = [
-                    'name' => 'Author ' . substr(md5(time()), 0, 4)
+                    'name' => 'Author '.substr(md5(time()), 0, 4)
                 ];
             }
             $data[] = [
                 'id'      => $i,
-                'title'   => 'Title of ' . $i,
+                'title'   => 'Title of '.$i,
                 'authors' => $authors,
             ];
         }
@@ -69,7 +69,7 @@ 
 //        return true;
         $p->processPayload('{ posts { id, title, authors { name } } }');
         $res = $p->getResponseData();
-        echo "Count: " . count($res['data']['posts']) . "\n";
+        echo "Count: ".count($res['data']['posts'])."\n";
         printf("Test Time: %04f\n", microtime(true) - $time);
 //        $p->getResponseData();
     }

Tests/Library/Type/InputObjectTypeTest.php

Unused Use Statements -7 removed lines

@@ -9,13 +9,7 @@ 
 namespace Youshido\Tests\Library\Type;
 
 
-use Youshido\GraphQL\Execution\Context\ExecutionContext;
 use Youshido\GraphQL\Execution\Processor;
-use Youshido\GraphQL\Execution\Request;
-use Youshido\GraphQL\Field\Field;
-use Youshido\GraphQL\Parser\Ast\Argument;
-use Youshido\GraphQL\Parser\Ast\ArgumentValue\Literal;
-use Youshido\GraphQL\Parser\Ast\Query;
 use Youshido\GraphQL\Schema\Schema;
 use Youshido\GraphQL\Type\InputObject\InputObjectType;
 use Youshido\GraphQL\Type\ListType\ListType;
@@ -24,7 +18,6 @@ 
 use Youshido\GraphQL\Type\Scalar\BooleanType;
 use Youshido\GraphQL\Type\Scalar\StringType;
 use Youshido\GraphQL\Type\TypeMap;
-use Youshido\GraphQL\Validator\ResolveValidator\ResolveValidator;
 use Youshido\Tests\DataProvider\TestInputObjectType;
 
 class InputObjectTypeTest extends \PHPUnit_Framework_TestCase

Spacing +4 added lines, -4 removed lines

@@ -60,7 +60,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () {
+                        'resolve' => function() {
                             return null;
                         }
                     ]
@@ -79,7 +79,7 @@ 
                             ]))
                         ],
                         'type'    => new BooleanType(),
-                        'resolve' => function ($object, $args) {
+                        'resolve' => function($object, $args) {
                             return true;
                         }
                     ]
@@ -100,7 +100,7 @@ 
                 'fields' => [
                     'empty' => [
                         'type'    => new StringType(),
-                        'resolve' => function () { }
+                        'resolve' => function() { }
                     ],
                 ]
             ]),
@@ -122,7 +122,7 @@ 
                                 ]
                             ])
                         ],
-                        'resolve' => function () { return 'success message'; }
+                        'resolve' => function() { return 'success message'; }
                     ]
                 ]
             ])