overblog / GraphQLBundle

Resolver/Config/FieldsConfigSolution.php

Doc Comments +3 added lines

@@ -149,6 +149,9 @@
         return $treatedOptions;
     }
 
+    /**
+     * @param callable $expression
+     */
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
         return function () use ($expression, $resolveCallback) {

Spacing +6 added lines, -6 removed lines

@@ -68,7 +68,7 @@ 
 
         $value = $treatedOptions['complexity'];
 
-        $treatedOptions['complexity'] = function () use ($value) {
+        $treatedOptions['complexity'] = function() use ($value) {
             $args = func_get_args();
             $complexity = $this->solveUsingExpressionLanguageIfNeeded(
                 $value,
@@ -179,10 +179,10 @@ 
 
     private function resolveAccessAndWrapResolveCallback($expression, callable $resolveCallback = null)
     {
-        return function () use ($expression, $resolveCallback) {
+        return function() use ($expression, $resolveCallback) {
             $args = func_get_args();
 
-            $result = null !== $resolveCallback  ? call_user_func_array($resolveCallback, $args) : null;
+            $result = null !== $resolveCallback ? call_user_func_array($resolveCallback, $args) : null;
 
             $values = call_user_func_array([$this, 'solveResolveCallbackArgs'], $args);
 
@@ -198,7 +198,7 @@ 
             case is_array($result) || $result instanceof \ArrayAccess:
                 $result = array_filter(
                     array_map(
-                        function ($object) use ($checkAccess) {
+                        function($object) use ($checkAccess) {
                             return $checkAccess($object) ? $object : null;
                         },
                         $result
@@ -208,7 +208,7 @@ 
 
             case $result instanceof Connection:
                 $result->edges = array_map(
-                    function (Edge $edge) use ($checkAccess) {
+                    function(Edge $edge) use ($checkAccess) {
                         $edge->node = $checkAccess($edge->node) ? $edge->node : null;
 
                         return $edge;
@@ -227,7 +227,7 @@ 
 
     private function checkAccessCallback($expression, $values)
     {
-        return function ($object, $throwException = false) use ($expression, $values) {
+        return function($object, $throwException = false) use ($expression, $values) {
             try {
                 $access = $this->solveUsingExpressionLanguageIfNeeded(
                     $expression,

Request/Validator/Rule/AbstractQuerySecurity.php

Doc Comments +1 added lines

@@ -41,6 +41,7 @@
      * check if equal to 0 no check is done. Must be greater or equal to 0.
      *
      * @param $value
+     * @param string $name
      */
     protected static function checkIfGreaterOrEqualToZero($name, $value)
     {

DependencyInjection/Configuration.php

Spacing +3 added lines, -3 removed lines

@@ -111,12 +111,12 @@
         $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()
         ;

Tests/Request/Validator/Rule/Schema.php

Spacing +2 added lines, -2 removed lines

@@ -70,14 +70,14 @@
                 'fields' => [
                     'firstName' => ['type' => Type::nonNull(Type::string())],
                     'dogs' => [
-                        'type' => function () {
+                        'type' => function() {
                             return Type::nonNull(
                                 Type::listOf(
                                     Type::nonNull(self::buildDogType())
                                 )
                             );
                         },
-                        'complexity' => function ($childrenComplexity, $args) {
+                        'complexity' => function($childrenComplexity, $args) {
                             $complexity = isset($args['name']) ? 1 : 10;
 
                             return $childrenComplexity + $complexity;

Request/Validator/Rule/QueryComplexity.php

Spacing +3 added lines, -3 removed lines

@@ -89,7 +89,7 @@ 
             [
                 // Visit FragmentDefinition after visiting FragmentSpread
                 'visitSpreadFragments' => true,
-                Node::SELECTION_SET => function (SelectionSet $selectionSet) use ($context) {
+                Node::SELECTION_SET => function(SelectionSet $selectionSet) use ($context) {
                     $this->fieldAstAndDefs = $this->collectFieldASTsAndDefs(
                         $context,
                         $context->getParentType(),
@@ -98,13 +98,13 @@ 
                         $this->fieldAstAndDefs
                     );
                 },
-                Node::VARIABLE_DEFINITION => function ($def) {
+                Node::VARIABLE_DEFINITION => function($def) {
                     $this->variableDefs[] = $def;
 
                     return Visitor::skipNode();
                 },
                 Node::OPERATION_DEFINITION => [
-                    'leave' => function (OperationDefinition $operationDefinition) use ($context, &$complexity) {
+                    'leave' => function(OperationDefinition $operationDefinition) use ($context, &$complexity) {
                         $complexity = $this->fieldComplexity($operationDefinition, $complexity);
 
                         if ($complexity > $this->getMaxQueryComplexity()) {

Request/Validator/Rule/QueryDepth.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
             $context,
             [
                 Node::OPERATION_DEFINITION => [
-                    'leave' => function (OperationDefinition $operationDefinition) use ($context) {
+                    'leave' => function(OperationDefinition $operationDefinition) use ($context) {
                         $maxDepth = $this->fieldDepth($operationDefinition);
 
                         if ($maxDepth > $this->getMaxQueryDepth()) {