overblog / GraphQLBundle

Tests/Functional/Controller/GraphControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
         $client->request('POST', $uri, [], [], ['CONTENT_TYPE' => 'application/json'], json_encode($data));
         $result = $client->getResponse()->getContent();
 
-        $expected  = [
+        $expected = [
             ['id' => 'friends', 'payload' => ['data' => $this->expectedData]],
             ['id' => 'friendsTotalCount', 'payload' => ['data' => ['user' => ['friends' => ['totalCount' => 4]]]]],
         ];

Config/EnumTypeDefinition.php

Spacing +2 added lines, -2 removed lines

@@ -26,10 +26,10 @@
                 ->arrayNode('values')
                     ->useAttributeAsKey('name')
                     ->beforeNormalization()
-                        ->ifTrue(function ($v) {
+                        ->ifTrue(function($v) {
                             return is_array($v);
                         })
-                        ->then(function ($v) {
+                        ->then(function($v) {
                             foreach ($v as $name => &$options) {
                                 // short syntax NAME: VALUE
                                 if (!is_null($options) && !is_array($options)) {

DependencyInjection/Configuration.php

Doc Comments +6 added lines

@@ -162,6 +162,9 @@ 
         return $treeBuilder;
     }
 
+    /**
+     * @param string $name
+     */
     private function addBuilderSection($name)
     {
         $builder = new TreeBuilder();
@@ -195,6 +198,9 @@ 
         return $node;
     }
 
+    /**
+     * @param string $name
+     */
     private function addSecurityQuerySection($name, $disabledValue)
     {
         $builder = new TreeBuilder();

Spacing +7 added lines, -7 removed lines

@@ -49,14 +49,14 @@ 
                         ->booleanNode('config_validation')->defaultValue($this->debug)->end()
                         ->arrayNode('schema')
                             ->beforeNormalization()
-                                ->ifTrue(function ($v) {
+                                ->ifTrue(function($v) {
                                     $needNormalization = isset($v['query']) && is_string($v['query']) ||
                                         isset($v['mutation']) && is_string($v['mutation']) ||
                                         isset($v['subscription']) && is_string($v['subscription']);
 
                                     return $needNormalization;
                                 })
-                                ->then(function ($v) {
+                                ->then(function($v) {
                                     return ['default' => $v];
                                 })
                             ->end()
@@ -167,10 +167,10 @@ 
         $builder = new TreeBuilder();
         $node = $builder->root($name);
         $node->beforeNormalization()
-            ->ifTrue(function ($v) {
+            ->ifTrue(function($v) {
                 return is_array($v) && !empty($v);
             })
-            ->then(function ($v) {
+            ->then(function($v) {
                 foreach ($v as $key => &$config) {
                     if (is_string($config)) {
                         $config = [
@@ -203,16 +203,16 @@ 
         $node
             ->info('Disabled if equal to false.')
             ->beforeNormalization()
-                ->ifTrue(function ($v) {
+                ->ifTrue(function($v) {
                     return false === $v;
                 })
-                ->then(function () use ($disabledValue) {
+                ->then(function() use ($disabledValue) {
                     return $disabledValue;
                 })
             ->end()
             ->defaultFalse()
             ->validate()
-                ->ifTrue(function ($v) {
+                ->ifTrue(function($v) {
                     return $v < 0;
                 })
                 ->thenInvalid('"overblog_graphql.security.'.$name.'" must be greater or equal to 0.')

Request/Executor.php

Doc Comments +8 added lines

@@ -88,6 +88,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $name
+     */
     public function addSchema($name, Schema $schema)
     {
         $this->schemas[$name] = $schema;
@@ -192,6 +195,11 @@ 
         return $this->prepareResult($result, $startTime, $startMemoryUsage);
     }
 
+    /**
+     * @param ExecutionResult $result
+     * @param double $startTime
+     * @param integer $startMemoryUsage
+     */
     private function prepareResult($result, $startTime, $startMemoryUsage)
     {
         if ($this->hasDebugInfo()) {

Resolver/AccessResolver.php

Spacing +3 added lines, -3 removed lines

@@ -55,7 +55,7 @@ 
         if ($this->promiseAdapter->isThenable($result) || $result instanceof SyncPromise) {
             return $this->promiseAdapter->then(
                 new Promise($result, $this->promiseAdapter),
-                function ($result) use ($accessChecker, $resolveArgs) {
+                function($result) use ($accessChecker, $resolveArgs) {
                     return $this->processFilter($result, $accessChecker, $resolveArgs);
                 }
             );
@@ -68,14 +68,14 @@ 
     {
         if (is_array($result)) {
             $result = array_map(
-                function ($object) use ($accessChecker, $resolveArgs) {
+                function($object) use ($accessChecker, $resolveArgs) {
                     return $this->hasAccess($accessChecker, $object, $resolveArgs) ? $object : null;
                 },
                 $result
             );
         } elseif ($result instanceof Connection) {
             $result->edges = array_map(
-                function (Edge $edge) use ($accessChecker, $resolveArgs) {
+                function(Edge $edge) use ($accessChecker, $resolveArgs) {
                     $edge->node = $this->hasAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
 
                     return $edge;

Tests/Relay/Connection/Output/ConnectionBuilderFromPromisedTest.php

Spacing +2 added lines, -2 removed lines

@@ -59,10 +59,10 @@
         $resolvedValue = null;
         $rejectedReason = null;
         $promise->then(
-            function ($value) use (&$resolvedValue) {
+            function($value) use (&$resolvedValue) {
                 $resolvedValue = $value;
             },
-            function ($reason) use (&$rejectedReason) {
+            function($reason) use (&$rejectedReason) {
                 $rejectedReason = $reason;
             }
         );

Relay/Connection/Output/ConnectionBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
     {
         self::checkPromise($dataPromise);
 
-        return $dataPromise->then(function ($data) use ($args) {
+        return $dataPromise->then(function($data) use ($args) {
             return static::connectionFromArray($data, $args);
         });
     }
@@ -171,7 +171,7 @@ 
     {
         self::checkPromise($dataPromise);
 
-        return $dataPromise->then(function ($arraySlice) use ($args, $meta) {
+        return $dataPromise->then(function($arraySlice) use ($args, $meta) {
             return static::connectionFromArraySlice($arraySlice, $args, $meta);
         });
     }

DependencyInjection/Compiler/TaggedServiceMappingPass.php

Doc Comments +6 added lines

@@ -64,7 +64,13 @@
 
     abstract protected function getTagName();
 
+    /**
+     * @return string
+     */
     abstract protected function getResolverServiceID();
 
+    /**
+     * @return string
+     */
     abstract protected function getParameterName();
 }

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
             $solutionDefinition = $container->findDefinition($options['id']);
 
             $methods = array_map(
-                function ($methodCall) {
+                function($methodCall) {
                     return $methodCall[0];
                 },
                 $solutionDefinition->getMethodCalls()

DependencyInjection/TypesConfiguration.php

Doc Comments +3 added lines

@@ -162,6 +162,9 @@
             ->end();
     }
 
+    /**
+     * @param string $typeToTreat
+     */
     private function relayNormalizer($typeToTreat, $definitionBuilderClass)
     {
         return function ($types) use ($typeToTreat, $definitionBuilderClass) {

Spacing +11 added lines, -11 removed lines

@@ -41,7 +41,7 @@ 
         $rootNode = $treeBuilder->root('overblog_graphql_types');
 
         $configTypeKeys = array_map(
-            function ($type) {
+            function($type) {
                 return $this->normalizedConfigTypeKey($type);
             },
             self::$types
@@ -54,7 +54,7 @@ 
             ->prototype('array')
                 // config is the unique config entry allowed
                 ->beforeNormalization()
-                    ->ifTrue(function ($v) use ($configTypeKeys) {
+                    ->ifTrue(function($v) use ($configTypeKeys) {
                         if (!empty($v) && is_array($v)) {
                             $keys = array_keys($v);
                             foreach ($configTypeKeys as $configTypeKey) {
@@ -69,16 +69,16 @@ 
                         ->thenInvalid(
                             sprintf(
                                 'Don\'t use internal config keys %s, replace it by "config" instead.',
-                                implode(', ',  $configTypeKeys)
+                                implode(', ', $configTypeKeys)
                             )
                         )
                 ->end()
                 // config is renamed _{TYPE}_config
                 ->beforeNormalization()
-                    ->ifTrue(function ($v) {
+                    ->ifTrue(function($v) {
                         return isset($v['type']) && is_string($v['type']);
                     })
-                    ->then(function ($v) {
+                    ->then(function($v) {
                         $key = $this->normalizedConfigTypeKey($v['type']);
 
                         if (empty($v[$key])) {
@@ -102,10 +102,10 @@ 
                 ->end()
                 // _{TYPE}_config is renamed config
                 ->validate()
-                    ->ifTrue(function ($v) {
+                    ->ifTrue(function($v) {
                         return isset($v[$this->normalizedConfigTypeKey($v['type'])]);
                     })
-                    ->then(function ($v) {
+                    ->then(function($v) {
                         $key = $this->normalizedConfigTypeKey($v['type']);
                         $v['config'] = $v[$key];
                         unset($v[$key]);
@@ -121,7 +121,7 @@ 
 
     private function addBeforeNormalization(ArrayNodeDefinition $node)
     {
-        $typeKeyExists = function ($types) {
+        $typeKeyExists = function($types) {
             return !empty($types) && is_array($types);
         };
 
@@ -129,7 +129,7 @@ 
             // set type config.name
             ->beforeNormalization()
                 ->ifTrue($typeKeyExists)
-                ->then(function ($types) {
+                ->then(function($types) {
                     foreach ($types as $name => &$type) {
                         $type['config'] = isset($type['config']) && is_array($type['config']) ? $type['config'] : [];
                         $type['config']['name'] = $name;
@@ -155,7 +155,7 @@ 
             ->end()
             // normalized relay-mutation-payload
             ->beforeNormalization()
-                ->ifTrue(function ($types) {
+                ->ifTrue(function($types) {
                     return !empty($types) && is_array($types);
                 })
                 ->then($this->relayNormalizer('relay-mutation-payload', PayloadDefinition::class))
@@ -164,7 +164,7 @@ 
 
     private function relayNormalizer($typeToTreat, $definitionBuilderClass)
     {
-        return function ($types) use ($typeToTreat, $definitionBuilderClass) {
+        return function($types) use ($typeToTreat, $definitionBuilderClass) {
             foreach ($types as $name => $type) {
                 if (isset($type['type']) && is_string($type['type']) && $typeToTreat === $type['type']) {
                     $config = isset($type['config']) && is_array($type['config']) ? $type['config'] : [];