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', |
@@ -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 | ]; |
@@ -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(); |
@@ -132,10 +132,10 @@ discard block |
||
132 | 132 | $prototype |
133 | 133 | // build args if argsBuilder exists |
134 | 134 | ->beforeNormalization() |
135 | - ->ifTrue(function ($field) { |
|
135 | + ->ifTrue(function($field) { |
|
136 | 136 | return isset($field['argsBuilder']); |
137 | 137 | }) |
138 | - ->then(function ($field) { |
|
138 | + ->then(function($field) { |
|
139 | 139 | $argsBuilderName = null; |
140 | 140 | |
141 | 141 | 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'])) { |
@@ -197,10 +197,10 @@ discard block |
||
197 | 197 | ->prototype('array') |
198 | 198 | // Allow arg type short syntax (Arg: Type => Arg: {type: Type}) |
199 | 199 | ->beforeNormalization() |
200 | - ->ifTrue(function ($options) { |
|
200 | + ->ifTrue(function($options) { |
|
201 | 201 | return is_string($options); |
202 | 202 | }) |
203 | - ->then(function ($options) { |
|
203 | + ->then(function($options) { |
|
204 | 204 | return ['type' => $options]; |
205 | 205 | }) |
206 | 206 | ->end() |
@@ -43,7 +43,7 @@ |
||
43 | 43 | $node = $builder->root('name', 'scalar'); |
44 | 44 | $node->isRequired(); |
45 | 45 | $node->validate() |
46 | - ->ifTrue(function ($name) { |
|
46 | + ->ifTrue(function($name) { |
|
47 | 47 | return !preg_match('/^[_a-z][_0-9a-z]*$/i', $name); |
48 | 48 | }) |
49 | 49 | ->thenInvalid('Invalid type name "%s". (see https://facebook.github.io/graphql/#Name)') |
@@ -36,11 +36,11 @@ |
||
36 | 36 | ->end(); |
37 | 37 | |
38 | 38 | $node->validate() |
39 | - ->ifTrue(function ($v) { |
|
39 | + ->ifTrue(function($v) { |
|
40 | 40 | |
41 | 41 | return array_key_exists('fieldsDefaultAccess', $v) && null !== $v['fieldsDefaultAccess']; |
42 | 42 | }) |
43 | - ->then(function ($v) { |
|
43 | + ->then(function($v) { |
|
44 | 44 | foreach ($v['fields'] as &$field) { |
45 | 45 | if (array_key_exists('access', $field) && null !== $field['access']) { |
46 | 46 | continue; |
@@ -40,7 +40,7 @@ |
||
40 | 40 | |
41 | 41 | public function getPrivatePropertyWithoutGetterUsingCallBack() |
42 | 42 | { |
43 | - return function () { |
|
43 | + return function() { |
|
44 | 44 | return $this->privatePropertyWithoutGetter; |
45 | 45 | }; |
46 | 46 | } |
@@ -40,7 +40,7 @@ |
||
40 | 40 | private function processConfig(array $configs) |
41 | 41 | { |
42 | 42 | return array_map( |
43 | - function ($v) { |
|
43 | + function($v) { |
|
44 | 44 | if (is_array($v)) { |
45 | 45 | return call_user_func([$this, 'processConfig'], $v); |
46 | 46 | } elseif (is_string($v) && 0 === strpos($v, '@=')) { |
@@ -36,7 +36,7 @@ discard block |
||
36 | 36 | switch (true) { |
37 | 37 | case is_array($result): |
38 | 38 | $result = array_map( |
39 | - function ($object) use ($accessChecker, $resolveArgs) { |
|
39 | + function($object) use ($accessChecker, $resolveArgs) { |
|
40 | 40 | return $this->checkAccess($accessChecker, $object, $resolveArgs) ? $object : null; |
41 | 41 | }, |
42 | 42 | $result |
@@ -44,7 +44,7 @@ discard block |
||
44 | 44 | break; |
45 | 45 | case $result instanceof Connection: |
46 | 46 | $result->edges = array_map( |
47 | - function (Edge $edge) use ($accessChecker, $resolveArgs) { |
|
47 | + function(Edge $edge) use ($accessChecker, $resolveArgs) { |
|
48 | 48 | $edge->node = $this->checkAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null; |
49 | 49 | |
50 | 50 | return $edge; |