We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -49,7 +49,7 @@ |
||
49 | 49 | $expectedData = [ |
50 | 50 | 'user' => [ |
51 | 51 | 'friends' => [ |
52 | - 'totalCount' => 4, |
|
52 | + 'totalCount' => 4, |
|
53 | 53 | 'edges' => [ |
54 | 54 | [ |
55 | 55 | 'friendshipTime' => 'Yesterday', |
@@ -125,7 +125,7 @@ discard block |
||
125 | 125 | $edges, |
126 | 126 | new PageInfo( |
127 | 127 | $firstEdge instanceof Edge ? $firstEdge->cursor : null, |
128 | - $lastEdge instanceof Edge ? $lastEdge->cursor : null, |
|
128 | + $lastEdge instanceof Edge ? $lastEdge->cursor : null, |
|
129 | 129 | $last !== null ? $startOffset > $lowerBound : false, |
130 | 130 | $first !== null ? $endOffset < $upperBound : false |
131 | 131 | ) |
@@ -178,7 +178,7 @@ discard block |
||
178 | 178 | } |
179 | 179 | $offset = static::cursorToOffset($cursor); |
180 | 180 | |
181 | - return !is_numeric($offset) ? $defaultOffset : (int) $offset; |
|
181 | + return !is_numeric($offset) ? $defaultOffset : (int) $offset; |
|
182 | 182 | } |
183 | 183 | |
184 | 184 | /** |
@@ -13,6 +13,9 @@ |
||
13 | 13 | |
14 | 14 | trait DIContainerMockTrait |
15 | 15 | { |
16 | + /** |
|
17 | + * @return null|\Symfony\Component\DependencyInjection\ContainerInterface |
|
18 | + */ |
|
16 | 19 | private function getDIContainerMock(array $services = [], array $parameters = []) |
17 | 20 | { |
18 | 21 | $container = $this->getMock('Symfony\\Component\\DependencyInjection\\Container', ['get', 'getParameter', 'has']); |
@@ -165,7 +165,7 @@ |
||
165 | 165 | $client->request('POST', '/?batch', [], [], ['CONTENT_TYPE' => 'application/json'], json_encode($data)); |
166 | 166 | $result = $client->getResponse()->getContent(); |
167 | 167 | |
168 | - $expected = [ |
|
168 | + $expected = [ |
|
169 | 169 | 'friends' => ['data' => $this->expectedData], |
170 | 170 | 'friendsTotalCount' => ['data' => ['user' => ['friends' => ['totalCount' => 4]]]], |
171 | 171 | ]; |
@@ -111,12 +111,12 @@ |
||
111 | 111 | $node |
112 | 112 | ->info('Disabled if equal to false.') |
113 | 113 | ->beforeNormalization() |
114 | - ->ifTrue(function ($v) { return false === $v; }) |
|
115 | - ->then(function () use ($disabledValue) { return $disabledValue; }) |
|
114 | + ->ifTrue(function($v) { return false === $v; }) |
|
115 | + ->then(function() use ($disabledValue) { return $disabledValue; }) |
|
116 | 116 | ->end() |
117 | 117 | ->defaultFalse() |
118 | 118 | ->validate() |
119 | - ->ifTrue(function ($v) { return $v < 0; }) |
|
119 | + ->ifTrue(function($v) { return $v < 0; }) |
|
120 | 120 | ->thenInvalid('"overblog_graphql.security.'.$name.'" must be greater or equal to 0.') |
121 | 121 | ->end() |
122 | 122 | ; |
@@ -127,9 +127,9 @@ |
||
127 | 127 | // from config |
128 | 128 | if (!empty($config['definitions']['mappings']['types'])) { |
129 | 129 | $typesMappings = array_filter(array_map( |
130 | - function (array $typeMapping) use ($container) { |
|
130 | + function(array $typeMapping) use ($container) { |
|
131 | 131 | |
132 | - $params = $this->detectConfigFiles($container, $typeMapping['dir'], $typeMapping['type']); |
|
132 | + $params = $this->detectConfigFiles($container, $typeMapping['dir'], $typeMapping['type']); |
|
133 | 133 | |
134 | 134 | return $params; |
135 | 135 | }, |
@@ -12,7 +12,6 @@ |
||
12 | 12 | namespace Overblog\GraphQLBundle\Request; |
13 | 13 | |
14 | 14 | use GraphQL\GraphQL; |
15 | -use GraphQL\Language\Source; |
|
16 | 15 | use GraphQL\Schema; |
17 | 16 | use GraphQL\Validator\DocumentValidator; |
18 | 17 | use GraphQL\Validator\Rules\QueryComplexity; |
@@ -15,7 +15,7 @@ |
||
15 | 15 | { |
16 | 16 | private static $hasMutate = false; |
17 | 17 | |
18 | - public static function hasMutate($reset = false) |
|
18 | + public static function hasMutate($reset = false) |
|
19 | 19 | { |
20 | 20 | $hasMutate = self::$hasMutate; |
21 | 21 |
@@ -51,6 +51,9 @@ discard block |
||
51 | 51 | self::$fieldBuilderClassMap[$name] = $fieldBuilderClass; |
52 | 52 | } |
53 | 53 | |
54 | + /** |
|
55 | + * @param string $type |
|
56 | + */ |
|
54 | 57 | protected static function checkBuilderClass($builderClass, $type) |
55 | 58 | { |
56 | 59 | $interface = 'Overblog\\GraphQLBundle\\Definition\\Builder\\MappingInterface'; |
@@ -113,6 +116,9 @@ discard block |
||
113 | 116 | } |
114 | 117 | } |
115 | 118 | |
119 | + /** |
|
120 | + * @param string $name |
|
121 | + */ |
|
116 | 122 | protected function outputFieldsSelection($name, $withAccess = false) |
117 | 123 | { |
118 | 124 | $builder = new TreeBuilder(); |
@@ -127,10 +127,10 @@ discard block |
||
127 | 127 | $prototype |
128 | 128 | // build args if argsBuilder exists |
129 | 129 | ->beforeNormalization() |
130 | - ->ifTrue(function ($field) { |
|
130 | + ->ifTrue(function($field) { |
|
131 | 131 | return isset($field['argsBuilder']); |
132 | 132 | }) |
133 | - ->then(function ($field) { |
|
133 | + ->then(function($field) { |
|
134 | 134 | $argsBuilderName = null; |
135 | 135 | |
136 | 136 | if (is_string($field['argsBuilder'])) { |
@@ -156,7 +156,7 @@ discard block |
||
156 | 156 | ->end() |
157 | 157 | // build field if builder exists |
158 | 158 | ->beforeNormalization() |
159 | - ->always(function ($field) { |
|
159 | + ->always(function($field) { |
|
160 | 160 | $fieldBuilderName = null; |
161 | 161 | |
162 | 162 | if (isset($field['builder']) && is_string($field['builder'])) { |