overblog / GraphQLBundle

Config/TypeWithOutputFieldsDefinition.php

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

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

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'] : [];

Tests/DIContainerMockTrait.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
  */
 trait DIContainerMockTrait
 {
+    /**
+     * @return null|\Symfony\Component\DependencyInjection\ContainerInterface
+     */
     private function getDIContainerMock(array $services = [], array $parameters = [])
     {
         $container = $this->getMockBuilder(Container::class)

ExpressionLanguage/ConfigExpressionProvider.php

Spacing +11 added lines, -11 removed lines

@@ -22,35 +22,35 @@ 
         return [
             new ExpressionFunction(
                 'service',
-                function ($value) {
+                function($value) {
                     return sprintf('$container->get(%s)', $value);
                 }
             ),
 
             new ExpressionFunction(
                 'parameter',
-                function ($value) {
+                function($value) {
                     return sprintf('$container->getParameter(%s)', $value);
                 }
             ),
 
             new ExpressionFunction(
                 'isTypeOf',
-                function ($className) {
+                function($className) {
                     return sprintf('($className = %s) && $value instanceof $className', $className);
                 }
             ),
 
             new ExpressionFunction(
                 'resolver',
-                function ($alias, $args = '[]') {
+                function($alias, $args = '[]') {
                     return sprintf('$container->get(\'overblog_graphql.resolver_resolver\')->resolve([%s, %s])', $alias, $args);
                 }
             ),
 
             new ExpressionFunction(
                 'mutateAndGetPayloadCallback',
-                function ($mutateAndGetPayload) {
+                function($mutateAndGetPayload) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $context, $info) { ';
                     $code .= 'return '.$mutateAndGetPayload.'; }';
 
@@ -60,7 +60,7 @@ 
 
             new ExpressionFunction(
                 'idFetcherCallback',
-                function ($idFetcher) {
+                function($idFetcher) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $context, $info) { ';
                     $code .= 'return '.$idFetcher.'; }';
 
@@ -70,7 +70,7 @@ 
 
             new ExpressionFunction(
                 'resolveSingleInputCallback',
-                function ($resolveSingleInput) {
+                function($resolveSingleInput) {
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $context, $info) { ';
                     $code .= 'return '.$resolveSingleInput.'; }';
 
@@ -80,14 +80,14 @@ 
 
             new ExpressionFunction(
                 'mutation',
-                function ($alias, $args = '[]') {
+                function($alias, $args = '[]') {
                     return sprintf('$container->get(\'overblog_graphql.mutation_resolver\')->resolve([%s, %s])', $alias, $args);
                 }
             ),
 
             new ExpressionFunction(
                 'globalId',
-                function ($id, $typeName = null) {
+                function($id, $typeName = null) {
                     $typeNameEmpty = null === $typeName || '""' === $typeName || 'null' === $typeName || 'false' === $typeName;
 
                     return sprintf(
@@ -101,7 +101,7 @@ 
 
             new ExpressionFunction(
                 'fromGlobalId',
-                function ($globalId) {
+                function($globalId) {
                     return sprintf(
                         '%s::fromGlobalId(%s)',
                         GlobalId::class,
@@ -112,7 +112,7 @@ 
 
             new ExpressionFunction(
                 'newObject',
-                function ($className, $args = '[]') {
+                function($className, $args = '[]') {
                     return sprintf('(new \ReflectionClass(%s))->newInstanceArgs(%s)', $className, $args);
                 }
             ),