overblog / GraphQLBundle

Tests/DIContainerMockTrait.php

Doc Comments +3 added lines

@@ -13,6 +13,9 @@
 
 trait DIContainerMockTrait
 {
+    /**
+     * @return null|\Symfony\Component\DependencyInjection\ContainerInterface
+     */
     private function getDIContainerMock(array $services = [], array $parameters = [])
     {
         $container = $this->getMock('Symfony\\Component\\DependencyInjection\\Container', ['get', 'getParameter', 'has']);

Tests/Functional/Controller/GraphControllerTest.php

Spacing +1 added lines, -1 removed lines

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

Config/TypeWithOutputFieldsDefinition.php

Doc Comments +6 added lines

@@ -51,6 +51,9 @@ 
         self::$fieldBuilderClassMap[$name] = $fieldBuilderClass;
     }
 
+    /**
+     * @param string $type
+     */
     protected static function checkBuilderClass($builderClass, $type)
     {
         $interface = 'Overblog\\GraphQLBundle\\Definition\\Builder\\MappingInterface';
@@ -113,6 +116,9 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     */
     protected function outputFieldsSelection($name, $withAccess = false)
     {
         $builder = new TreeBuilder();

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

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

Config/ObjectTypeDefinition.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
             ->end();
 
         $node->validate()
-            ->ifTrue(function ($v) {
+            ->ifTrue(function($v) {
 
                 return array_key_exists('fieldsDefaultAccess', $v) && null !== $v['fieldsDefaultAccess'];
             })
-            ->then(function ($v) {
+            ->then(function($v) {
                 foreach ($v['fields'] as &$field) {
                     if (array_key_exists('access', $field) && null !== $field['access']) {
                         continue;

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

DependencyInjection/Compiler/TypesPass.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     private function processConfig(array $configs)
     {
         return array_map(
-            function ($v) {
+            function($v) {
                 if (is_array($v)) {
                     return call_user_func([$this, 'processConfig'], $v);
                 } elseif (is_string($v) && 0 === strpos($v, '@=')) {

Resolver/AccessResolver.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         switch (true) {
             case is_array($result):
                 $result = array_map(
-                    function ($object) use ($accessChecker, $resolveArgs) {
+                    function($object) use ($accessChecker, $resolveArgs) {
                         return $this->checkAccess($accessChecker, $object, $resolveArgs) ? $object : null;
                     },
                     $result
@@ -44,7 +44,7 @@ 
                 break;
             case $result instanceof Connection:
                 $result->edges = array_map(
-                    function (Edge $edge) use ($accessChecker, $resolveArgs) {
+                    function(Edge $edge) use ($accessChecker, $resolveArgs) {
                         $edge->node = $this->checkAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
 
                         return $edge;

DependencyInjection/TypesConfiguration.php

Spacing +11 added lines, -11 removed lines

@@ -36,7 +36,7 @@ 
         $rootNode = $treeBuilder->root('overblog_graphql_types');
 
         $configTypeKeys = array_map(
-            function ($type) {
+            function($type) {
                 return $this->normalizedConfigTypeKey($type);
             },
             self::$types
@@ -49,7 +49,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) {
@@ -64,16 +64,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])) {
@@ -96,10 +96,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]);
@@ -115,7 +115,7 @@ 
 
     private function addBeforeNormalization(ArrayNodeDefinition $node)
     {
-        $typeKeyExists = function ($types) {
+        $typeKeyExists = function($types) {
             return !empty($types) && is_array($types);
         };
 
@@ -123,7 +123,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;
@@ -149,7 +149,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', 'Overblog\GraphQLBundle\Relay\Mutation\PayloadDefinition'))
@@ -158,7 +158,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'] : [];