overblog · mcg-web · Oct 4, 2016 · Oct 4, 2016 · Oct 4, 2016 · Oct 4, 2016
diff --git a/Config/EnumTypeDefinition.php b/Config/EnumTypeDefinition.php
@@ -27,8 +27,12 @@ public function getDefinition()
                     ->useAttributeAsKey('name')
                     ->prototype('array')
                         ->beforeNormalization()
-                            ->ifTrue(function ($v) { return !is_null($v) && !is_array($v); })
-                            ->then(function ($v) { return ['value' => $v]; })
+                            ->ifTrue(function ($v) {
+                                return !is_null($v) && !is_array($v);
+                            })
+                            ->then(function ($v) {
+                                return ['value' => $v];
+                            })
                         ->end()
                         ->isRequired()
                         ->children()

diff --git a/Config/ObjectTypeDefinition.php b/Config/ObjectTypeDefinition.php
@@ -37,7 +37,6 @@ public function getDefinition()
 
         $node->validate()
             ->ifTrue(function ($v) {
-
                 return array_key_exists('fieldsDefaultAccess', $v) && null !== $v['fieldsDefaultAccess'];
             })
             ->then(function ($v) {

diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -30,7 +30,7 @@ class Configuration implements ConfigurationInterface
      */
     public function __construct($debug)
     {
-        $this->debug = (Boolean) $debug;
+        $this->debug = (bool) $debug;
     }
 
     public function getConfigTreeBuilder()
@@ -44,6 +44,7 @@ public function getConfigTreeBuilder()
                     ->addDefaultsIfNotSet()
                     ->children()
                         ->scalarNode('internal_error_message')->defaultNull()->end()
+                        ->booleanNode('show_debug_info')->defaultValue(false)->end()
                         ->booleanNode('config_validation')->defaultValue($this->debug)->end()
                         ->arrayNode('schema')
                             ->beforeNormalization()
@@ -84,11 +85,11 @@ public function getConfigTreeBuilder()
                         ->arrayNode('exceptions')
                             ->children()
                                 ->arrayNode('warnings')
-                                    ->treatNullLike(array())
+                                    ->treatNullLike([])
                                     ->prototype('scalar')->end()
                                 ->end()
                                 ->arrayNode('errors')
-                                    ->treatNullLike(array())
+                                    ->treatNullLike([])
                                     ->prototype('scalar')->end()
                                 ->end()
                                 ->arrayNode('types')
@@ -174,12 +175,18 @@ private function addSecurityQuerySection($name, $disabledValue)
         $node
             ->info('Disabled if equal to false.')
             ->beforeNormalization()
-                ->ifTrue(function ($v) { return false === $v; })
-                ->then(function () use ($disabledValue) { return $disabledValue; })
+                ->ifTrue(function ($v) {
+                    return false === $v;
+                })
+                ->then(function () use ($disabledValue) {
+                    return $disabledValue;
+                })
             ->end()
             ->defaultFalse()
             ->validate()
-                ->ifTrue(function ($v) { return $v < 0; })
+                ->ifTrue(function ($v) {
+                    return $v < 0;
+                })
                 ->thenInvalid('"overblog_graphql.security.'.$name.'" must be greater or equal to 0.')
             ->end()
         ;

diff --git a/DependencyInjection/OverblogGraphQLExtension.php b/DependencyInjection/OverblogGraphQLExtension.php
@@ -40,6 +40,7 @@ public function load(array $configs, ContainerBuilder $container)
         $this->setConfigBuilders($config);
         $this->setVersions($config, $container);
 
+        $container->getDefinition('overblog_graphql.request_executor')->replaceArgument(3, $config['definitions']['show_debug_info']);
         $container->setParameter($this->getAlias().'.resources_dir', realpath(__DIR__.'/../Resources'));
     }
 
@@ -161,7 +162,8 @@ public function getConfiguration(array $config, ContainerBuilder $container)
      * Returns a list of custom exceptions mapped to error/warning classes.
      *
      * @param array $exceptionConfig
-     * @return array Custom exception map, [exception => UserError/UserWarning].
+     *
+     * @return array Custom exception map, [exception => UserError/UserWarning]
      */
     private function buildExceptionMap(array $exceptionConfig)
     {

diff --git a/DependencyInjection/OverblogGraphQLTypesExtension.php b/DependencyInjection/OverblogGraphQLTypesExtension.php
@@ -105,7 +105,6 @@ private function typesConfigsMappingFromConfig(array $config, ContainerBuilder $
         if (!empty($config['definitions']['mappings']['types'])) {
             $typesMappings = array_filter(array_map(
                 function (array $typeMapping) use ($container) {
-
                     $params = $this->detectConfigFiles($container, $typeMapping['dir'],  $typeMapping['type']);
 
                     return $params;

diff --git a/DependencyInjection/TypesConfiguration.php b/DependencyInjection/TypesConfiguration.php
@@ -115,7 +115,9 @@ function ($type) {
 
     private function addBeforeNormalization(ArrayNodeDefinition $node)
     {
-        $typeKeyExists = function ($types) { return !empty($types) && is_array($types); };
+        $typeKeyExists = function ($types) {
+            return !empty($types) && is_array($types);
+        };
 
         $node
             // set type config.name
@@ -147,7 +149,9 @@ private function addBeforeNormalization(ArrayNodeDefinition $node)
             ->end()
             // normalized relay-mutation-payload
             ->beforeNormalization()
-                ->ifTrue(function ($types) { return !empty($types) && is_array($types); })
+                ->ifTrue(function ($types) {
+                    return !empty($types) && is_array($types);
+                })
                 ->then($this->relayNormalizer('relay-mutation-payload', 'Overblog\GraphQLBundle\Relay\Mutation\PayloadDefinition'))
             ->end();
     }

diff --git a/Error/ErrorHandler.php b/Error/ErrorHandler.php
@@ -139,12 +139,13 @@ public function handleErrors(ExecutionResult $executionResult, $throwRawExceptio
      * that is displayed to the user.
      *
      * @param \Exception $rawException
+     *
      * @return \Exception
      */
     protected function convertException(\Exception $rawException = null)
     {
         if (null === $rawException) {
-            return null;
+            return;
         }
 
         if (!empty($this->exceptionMap[get_class($rawException)])) {

diff --git a/ExpressionLanguage/AuthorizationExpressionProvider.php b/ExpressionLanguage/AuthorizationExpressionProvider.php
@@ -24,7 +24,8 @@ public function getFunctions()
                 function ($role) {
                     return sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s)', $role);
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -34,39 +35,44 @@ function ($roles) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'isAnonymous',
                 function () {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_ANONYMOUSLY\')';
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'isRememberMe',
                 function () {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\')';
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'isFullyAuthenticated',
                 function () {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'isAuthenticated',
                 function () {
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\') || $container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -76,7 +82,8 @@ function ($object, $permission) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -86,7 +93,8 @@ function ($object, $permissions) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
         ];
     }

diff --git a/ExpressionLanguage/ConfigExpressionProvider.php b/ExpressionLanguage/ConfigExpressionProvider.php
@@ -25,31 +25,35 @@ public function getFunctions()
                 function ($value) {
                     return sprintf('$container->get(%s)', $value);
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'parameter',
                 function ($value) {
                     return sprintf('$container->getParameter(%s)', $value);
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'isTypeOf',
                 function ($className) {
                     return sprintf('($className = %s) && $value instanceof $className', $className);
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'resolver',
                 function ($alias, $args = '[]') {
                     return sprintf('$container->get(\'overblog_graphql.resolver_resolver\')->resolve([%s, %s])', $alias, $args);
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -60,7 +64,8 @@ function ($mutateAndGetPayload) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -71,7 +76,8 @@ function ($mutateAndGetPayload) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -82,7 +88,8 @@ function ($idFetcher) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -93,15 +100,17 @@ function ($resolveSingleInput) {
 
                     return $code;
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'mutation',
                 function ($alias, $args = '[]') {
                     return sprintf('$container->get(\'overblog_graphql.mutation_resolver\')->resolve([%s, %s])', $alias, $args);
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -115,7 +124,8 @@ function ($id, $typeName = null) {
                         $id
                     );
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
@@ -126,15 +136,17 @@ function ($globalId) {
                         $globalId
                     );
                 },
-                function () {}
+                function () {
+                }
             ),
 
             new ExpressionFunction(
                 'newObject',
                 function ($className, $args = '[]') {
                     return sprintf('(new \ReflectionClass(%s))->newInstanceArgs(%s)', $className, $args);
                 },
-                function () {}
+                function () {
+                }
             ),
         ];
     }

diff --git a/Generator/TypeGenerator.php b/Generator/TypeGenerator.php
@@ -54,7 +54,7 @@ public function setCacheDir($cacheDir)
 
     protected function generateClassDocBlock(array $value)
     {
-        return <<<EOF
+        return <<<'EOF'
 
 /**
  * THIS FIELD WAS GENERATED AND SHOULD NOT BE MODIFIED!
@@ -73,9 +73,10 @@ protected function resolveTypeCode($alias)
     }
 
     /**
-     * todo replace generateResolve in vendor after spec-april2016 is merged
+     * todo replace generateResolve in vendor after spec-april2016 is merged.
      *
      * @param array $value
+     *
      * @return string
      */
     protected function generateResolve2(array $value)

diff --git a/README.md b/README.md
@@ -935,6 +935,32 @@ simple request | `/graphql/bar`
 batch request | `/graphql/bar/batch`
 graphiQL | `/graphiql/bar`
 
+Debug information
+-----------------
+
+To enabled or disabled debug information:
+
+```yaml
+# app/config/config.yml
+
+overblog_graphql:
+    definitions:
+        show_debug_info: true # Debug info is disabled by default
+```
+
+here an example of an answer when debug information is enabled
+```json
+{
+  "data": [{"isEnabled": true}],
+  "extensions": {
+    "debug": {
+      "executionTime": "400 ms",
+      "memoryUsage": "1.00 MiB"
+    }
+  }
+}
+```
+
 Contribute
 ----------