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