overblog / GraphQLBundle

Config/TypeDefinition.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
         $node = $builder->root('name', 'scalar');
         $node->isRequired();
         $node->validate()
-            ->ifTrue(function ($name) {
+            ->ifTrue(function($name) {
                 return !preg_match('/^[_a-z][_0-9a-z]*$/i', $name);
             })
                 ->thenInvalid('Invalid type name "%s". (see https://facebook.github.io/graphql/#Name)')

Tests/Resolver/Toto.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
     public function getPrivatePropertyWithoutGetterUsingCallBack()
     {
-        return function () {
+        return function() {
             return $this->privatePropertyWithoutGetter;
         };
     }

Tests/Request/ExecutorTest.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
             'fields' => [
                 'myField' => [
                     'type' => Type::boolean(),
-                    'resolve' => function () {
+                    'resolve' => function() {
                         return false;
                     },
                 ],

Config/TypeWithOutputFieldsDefinition.php

Spacing +5 added lines, -5 removed lines

@@ -132,10 +132,10 @@ 
         $prototype
             // build args if argsBuilder exists
             ->beforeNormalization()
-                ->ifTrue(function ($field) {
+                ->ifTrue(function($field) {
                     return isset($field['argsBuilder']);
                 })
-                ->then(function ($field) {
+                ->then(function($field) {
                     $argsBuilderName = null;
 
                     if (is_string($field['argsBuilder'])) {
@@ -156,7 +156,7 @@ 
             ->end()
             // build field if builder exists
             ->beforeNormalization()
-                ->always(function ($field) {
+                ->always(function($field) {
                     $fieldBuilderName = null;
 
                     if (isset($field['builder']) && is_string($field['builder'])) {
@@ -197,10 +197,10 @@ 
                     ->prototype('array')
                         // Allow arg type short syntax (Arg: Type => Arg: {type: Type})
                         ->beforeNormalization()
-                            ->ifTrue(function ($options) {
+                            ->ifTrue(function($options) {
                                 return is_string($options);
                             })
-                            ->then(function ($options) {
+                            ->then(function($options) {
                                 return ['type' => $options];
                             })
                         ->end()

Doc Comments +6 added lines

@@ -61,6 +61,9 @@ 
         self::$fieldBuilderClassMap[$name] = $fieldBuilderClass;
     }
 
+    /**
+     * @param string $type
+     */
     protected static function checkBuilderClass($builderClass, $type)
     {
         $interface = MappingInterface::class;
@@ -123,6 +126,9 @@ 
         throw new InvalidConfigurationException(sprintf('%s builder "%s" not found.', ucfirst($type), $name));
     }
 
+    /**
+     * @param string $name
+     */
     protected function outputFieldsSelection($name)
     {
         $builder = new TreeBuilder();

Relay/Connection/Paginator.php

Spacing +3 added lines, -3 removed lines

@@ -58,7 +58,7 @@ 
 
         $entities = call_user_func($this->fetcher, $offset, $limit);
 
-        return $this->handleEntities($entities, function ($entities) use ($args, $offset, $total) {
+        return $this->handleEntities($entities, function($entities) use ($args, $offset, $total) {
             return ConnectionBuilder::connectionFromArraySlice($entities, $args, [
                 'sliceStart' => $offset,
                 'arrayLength' => $total,
@@ -81,13 +81,13 @@ 
         if (!is_numeric(ConnectionBuilder::cursorToOffset($args['after'])) || !$args['after']) {
             $entities = call_user_func($this->fetcher, $offset, $limit + 1);
 
-            return $this->handleEntities($entities, function ($entities) use ($args) {
+            return $this->handleEntities($entities, function($entities) use ($args) {
                 return ConnectionBuilder::connectionFromArray($entities, $args);
             });
         } else {
             $entities = call_user_func($this->fetcher, $offset, $limit + 2);
 
-            return $this->handleEntities($entities, function ($entities) use ($args, $offset) {
+            return $this->handleEntities($entities, function($entities) use ($args, $offset) {
                 return ConnectionBuilder::connectionFromArraySlice($entities, $args, [
                     'sliceStart' => $offset,
                     'arrayLength' => $offset + count($entities),

Tests/Relay/Connection/PaginatorTest.php

Spacing +15 added lines, -15 removed lines

@@ -55,7 +55,7 @@ 
 
     public function testForward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit); // Includes the extra element to check if next page is available
 
@@ -71,7 +71,7 @@ 
 
     public function testForwardAfterInMiddle()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(2, $offset);
             $this->assertSame(3, $limit); // Includes the extra element to check if next page is available
 
@@ -87,7 +87,7 @@ 
 
     public function testForwardAfterAtTheEnd()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(2, $offset);
             $this->assertSame(4, $limit); // Includes the extra element to check if next page is available
 
@@ -103,7 +103,7 @@ 
 
     public function testForwardAfterLast()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(4, $offset);
             $this->assertSame(7, $limit); // Includes the extra element to check if next page is available
 
@@ -119,7 +119,7 @@ 
 
     public function testForwardAfterWithUnvalidCursorAndSlice()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit); // Includes the extra element to check if next page is available
 
@@ -136,7 +136,7 @@ 
 
     public function testBackward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(2, $offset);
             $this->assertSame(3, $limit);
 
@@ -153,7 +153,7 @@ 
 
     public function testBackwardWithLimitEqualsToTotal()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit);
 
@@ -170,7 +170,7 @@ 
 
     public function testBackwardBeforeLast()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(4, $limit);
 
             return $this->getData($offset);
@@ -185,7 +185,7 @@ 
 
     public function testBackwardPartialBeforeInMiddle()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(2, $limit);
 
@@ -201,7 +201,7 @@ 
 
     public function testAutoBackward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(4, $limit);
 
@@ -218,7 +218,7 @@ 
 
     public function testAutoForward()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit); // Includes the extra element to check if next page is available
 
@@ -234,7 +234,7 @@ 
 
     public function testAutoBackwardWithCallable()
     {
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(4, $limit);
 
@@ -242,7 +242,7 @@ 
         });
 
         $countCalled = false;
-        $result = $paginator->auto(new Argument(['last' => 4]), function () use (&$countCalled) {
+        $result = $paginator->auto(new Argument(['last' => 4]), function() use (&$countCalled) {
             $countCalled = true;
 
             return 5;
@@ -265,7 +265,7 @@ 
 
         $this->assertSame(5, call_user_func_array($callable, ['array' => $this->data]));
 
-        $paginator = new Paginator(function ($offset, $limit) {
+        $paginator = new Paginator(function($offset, $limit) {
             $this->assertSame(1, $offset);
             $this->assertSame(4, $limit);
 
@@ -292,7 +292,7 @@ 
 
         $promise->expects($this->once())->method('then');
 
-        $paginator = new Paginator(function ($offset, $limit) use ($promise) {
+        $paginator = new Paginator(function($offset, $limit) use ($promise) {
             $this->assertSame(0, $offset);
             $this->assertSame(5, $limit);
 

ExpressionLanguage/ExpressionFunction.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
     public function __construct($name, callable $compiler)
     {
-        parent::__construct($name, $compiler, function () {
+        parent::__construct($name, $compiler, function() {
             throw new \RuntimeException('No need evaluator.');
         });
     }

Executor/Promise/Adapter/ReactPromiseAdapter.php

Spacing +2 added lines, -2 removed lines

@@ -58,10 +58,10 @@
         /** @var \React\Promise\PromiseInterface $reactPromise */
         $reactPromise = $promise->adoptedPromise;
 
-        $reactPromise->then(function ($values) use (&$resolvedValue, &$wait) {
+        $reactPromise->then(function($values) use (&$resolvedValue, &$wait) {
             $resolvedValue = $values;
             $wait = false;
-        }, function ($reason) use (&$exception, &$wait) {
+        }, function($reason) use (&$exception, &$wait) {
             $exception = $reason;
             $wait = false;
         });

Tests/Executor/Promise/Adapter/ReactPromiseAdapterTest.php

Spacing +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@ 
 EOF
         );
 
-        $promise = $this->adapter->create(function (callable $resolve) use (&$process) {
-            $process->start(function () use ($resolve, &$process) {
+        $promise = $this->adapter->create(function(callable $resolve) use (&$process) {
+            $process->start(function() use ($resolve, &$process) {
                 $output = $process->getOutput();
                 $resolve($output);
             });
@@ -64,7 +64,7 @@ 
 
         $this->assertEquals(
             $output,
-            $this->adapter->wait($promise, function () use (&$process) {
+            $this->adapter->wait($promise, function() use (&$process) {
                 $process->wait();
             })
         );