View GitHub Repository GraphQLBundle

Schedule Inspection
Subscribe

Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Completed
Pull Request — master (#73)
by Jérémiah
04:58
created
Status
Category
Toggle Patches
Config/EnumTypeDefinition.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -27,10 +27,10 @@
 
 block discarded – undo
27 27 
                     ->useAttributeAsKey('name')
28 28 
                     ->prototype('array')
29 29 
                         ->beforeNormalization()
30 
-                            ->ifTrue(function ($v) {
30 
+                            ->ifTrue(function($v) {
31 31 
                                 return !is_null($v) && !is_array($v);
32 32 
                             })
33 
-                            ->then(function ($v) {
33 
+                            ->then(function($v) {
34 34 
                                 return ['value' => $v];
35 35 
                             })
36 36 
                         ->end()
Please login to merge, or discard this patch.
ExpressionLanguage/ConfigExpressionProvider.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch 
@@ -22,100 +22,100 @@  discard block
 
 block discarded – undo
22 22 
         return [
23 23 
             new ExpressionFunction(
24 24 
                 'service',
25 
-                function ($value) {
25 
+                function($value) {
26 26 
                     return sprintf('$container->get(%s)', $value);
27 27 
                 },
28 
-                function () {
28 
+                function() {
29 29 
                 }
30 30 
             ),
31 31
32 32 
             new ExpressionFunction(
33 33 
                 'parameter',
34 
-                function ($value) {
34 
+                function($value) {
35 35 
                     return sprintf('$container->getParameter(%s)', $value);
36 36 
                 },
37 
-                function () {
37 
+                function() {
38 38 
                 }
39 39 
             ),
40 40
41 41 
             new ExpressionFunction(
42 42 
                 'isTypeOf',
43 
-                function ($className) {
43 
+                function($className) {
44 44 
                     return sprintf('($className = %s) && $value instanceof $className', $className);
45 45 
                 },
46 
-                function () {
46 
+                function() {
47 47 
                 }
48 48 
             ),
49 49
50 50 
             new ExpressionFunction(
51 51 
                 'resolver',
52 
-                function ($alias, $args = '[]') {
52 
+                function($alias, $args = '[]') {
53 53 
                     return sprintf('$container->get(\'overblog_graphql.resolver_resolver\')->resolve([%s, %s])', $alias, $args);
54 54 
                 },
55 
-                function () {
55 
+                function() {
56 56 
                 }
57 57 
             ),
58 58
59 59 
             new ExpressionFunction(
60 60 
                 'mutateAndGetPayloadCallback',
61 
-                function ($mutateAndGetPayload) {
61 
+                function($mutateAndGetPayload) {
62 62 
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
63 63 
                     $code .= 'return '.$mutateAndGetPayload.'; }';
64 64
65 65 
                     return $code;
66 66 
                 },
67 
-                function () {
67 
+                function() {
68 68 
                 }
69 69 
             ),
70 70
71 71 
             new ExpressionFunction(
72 72 
                 'mutateAndGetPayloadCallback',
73 
-                function ($mutateAndGetPayload) {
73 
+                function($mutateAndGetPayload) {
74 74 
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
75 75 
                     $code .= 'return '.$mutateAndGetPayload.'; }';
76 76
77 77 
                     return $code;
78 78 
                 },
79 
-                function () {
79 
+                function() {
80 80 
                 }
81 81 
             ),
82 82
83 83 
             new ExpressionFunction(
84 84 
                 'idFetcherCallback',
85 
-                function ($idFetcher) {
85 
+                function($idFetcher) {
86 86 
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
87 87 
                     $code .= 'return '.$idFetcher.'; }';
88 88
89 89 
                     return $code;
90 90 
                 },
91 
-                function () {
91 
+                function() {
92 92 
                 }
93 93 
             ),
94 94
95 95 
             new ExpressionFunction(
96 96 
                 'resolveSingleInputCallback',
97 
-                function ($resolveSingleInput) {
97 
+                function($resolveSingleInput) {
98 98 
                     $code = 'function ($value) use ('.TypeGenerator::USE_FOR_CLOSURES.', $args, $info) { ';
99 99 
                     $code .= 'return '.$resolveSingleInput.'; }';
100 100
101 101 
                     return $code;
102 102 
                 },
103 
-                function () {
103 
+                function() {
104 104 
                 }
105 105 
             ),
106 106
107 107 
             new ExpressionFunction(
108 108 
                 'mutation',
109 
-                function ($alias, $args = '[]') {
109 
+                function($alias, $args = '[]') {
110 110 
                     return sprintf('$container->get(\'overblog_graphql.mutation_resolver\')->resolve([%s, %s])', $alias, $args);
111 111 
                 },
112 
-                function () {
112 
+                function() {
113 113 
                 }
114 114 
             ),
115 115
116 116 
             new ExpressionFunction(
117 117 
                 'globalId',
118 
-                function ($id, $typeName = null) {
118 
+                function($id, $typeName = null) {
119 119 
                     $typeNameEmpty = null === $typeName || '""' === $typeName || 'null' === $typeName || 'false' === $typeName;
120 120
121 121 
                     return sprintf(
 
@@ -124,28 +124,28 @@  discard block
 
 block discarded – undo
124 124 
                         $id
125 125 
                     );
126 126 
                 },
127 
-                function () {
127 
+                function() {
128 128 
                 }
129 129 
             ),
130 130
131 131 
             new ExpressionFunction(
132 132 
                 'fromGlobalId',
133 
-                function ($globalId) {
133 
+                function($globalId) {
134 134 
                     return sprintf(
135 135 
                         '\\Overblog\\GraphQLBundle\\Relay\\Node\\GlobalId::fromGlobalId(%s)',
136 136 
                         $globalId
137 137 
                     );
138 138 
                 },
139 
-                function () {
139 
+                function() {
140 140 
                 }
141 141 
             ),
142 142
143 143 
             new ExpressionFunction(
144 144 
                 'newObject',
145 
-                function ($className, $args = '[]') {
145 
+                function($className, $args = '[]') {
146 146 
                     return sprintf('(new \ReflectionClass(%s))->newInstanceArgs(%s)', $className, $args);
147 147 
                 },
148 
-                function () {
148 
+                function() {
149 149 
                 }
150 150 
             ),
151 151 
         ];
Please login to merge, or discard this patch.
ExpressionLanguage/AuthorizationExpressionProvider.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch 
@@ -21,79 +21,79 @@
 
 block discarded – undo
21 21 
         return [
22 22 
             new ExpressionFunction(
23 23 
                 'hasRole',
24 
-                function ($role) {
24 
+                function($role) {
25 25 
                     return sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s)', $role);
26 26 
                 },
27 
-                function () {
27 
+                function() {
28 28 
                 }
29 29 
             ),
30 30
31 31 
             new ExpressionFunction(
32 32 
                 'hasAnyRole',
33 
-                function ($roles) {
33 
+                function($roles) {
34 34 
                     $code = sprintf('array_reduce(%s, function ($isGranted, $role) use ($container) { return $isGranted || $container->get(\'security.authorization_checker\')->isGranted($role); }, false)', $roles);
35 35
36 36 
                     return $code;
37 37 
                 },
38 
-                function () {
38 
+                function() {
39 39 
                 }
40 40 
             ),
41 41
42 42 
             new ExpressionFunction(
43 43 
                 'isAnonymous',
44 
-                function () {
44 
+                function() {
45 45 
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_ANONYMOUSLY\')';
46 46 
                 },
47 
-                function () {
47 
+                function() {
48 48 
                 }
49 49 
             ),
50 50
51 51 
             new ExpressionFunction(
52 52 
                 'isRememberMe',
53 
-                function () {
53 
+                function() {
54 54 
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\')';
55 55 
                 },
56 
-                function () {
56 
+                function() {
57 57 
                 }
58 58 
             ),
59 59
60 60 
             new ExpressionFunction(
61 61 
                 'isFullyAuthenticated',
62 
-                function () {
62 
+                function() {
63 63 
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
64 64 
                 },
65 
-                function () {
65 
+                function() {
66 66 
                 }
67 67 
             ),
68 68
69 69 
             new ExpressionFunction(
70 70 
                 'isAuthenticated',
71 
-                function () {
71 
+                function() {
72 72 
                     return '$container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_REMEMBERED\') || $container->get(\'security.authorization_checker\')->isGranted(\'IS_AUTHENTICATED_FULLY\')';
73 73 
                 },
74 
-                function () {
74 
+                function() {
75 75 
                 }
76 76 
             ),
77 77
78 78 
             new ExpressionFunction(
79 79 
                 'hasPermission',
80 
-                function ($object, $permission) {
80 
+                function($object, $permission) {
81 81 
                     $code = sprintf('$container->get(\'security.authorization_checker\')->isGranted(%s, %s)', $permission, $object);
82 82
83 83 
                     return $code;
84 84 
                 },
85 
-                function () {
85 
+                function() {
86 86 
                 }
87 87 
             ),
88 88
89 89 
             new ExpressionFunction(
90 90 
                 'hasAnyPermission',
91 
-                function ($object, $permissions) {
91 
+                function($object, $permissions) {
92 92 
                     $code = sprintf('array_reduce(%s, function ($isGranted, $permission) use ($container, $object) { return $isGranted || $container->get(\'security.authorization_checker\')->isGranted($permission, %s); }, false)', $permissions, $object);
93 93
94 94 
                     return $code;
95 95 
                 },
96 
-                function () {
96 
+                function() {
97 97 
                 }
98 98 
             ),
99 99 
         ];
Please login to merge, or discard this patch.
Tests/Request/ExecutorTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31 
             'fields' => [
32 32 
                 'myField' => [
33 33 
                     'type' => Type::boolean(),
34 
-                    'resolve' => function () {
34 
+                    'resolve' => function() {
35 35 
                         return false;
36 36 
                     },
37 37 
                 ],
 
@@ -46,7 +46,7 @@  discard block
 
 block discarded – undo
46 46 
      */
47 47 
     public function testInvalidExecutorReturnNotObject()
48 48 
     {
49 
-        $this->executor->setExecutor(function () {
49 
+        $this->executor->setExecutor(function() {
50 50 
             return false;
51 51 
         });
52 52 
         $this->executor->execute($this->request);
 
@@ -58,7 +58,7 @@  discard block
 
 block discarded – undo
58 58 
      */
59 59 
     public function testInvalidExecutorReturnInvalidObject()
60 60 
     {
61 
-        $this->executor->setExecutor(function () {
61 
+        $this->executor->setExecutor(function() {
62 62 
             return new \stdClass();
63 63 
         });
64 64 
         $this->executor->execute($this->request);
Please login to merge, or discard this patch.
Resolver/AccessResolver.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -39,11 +39,11 @@  discard block
 
 block discarded – undo
39 39 
     {
40 40 
         $result = call_user_func_array($resolveCallback, $resolveArgs);
41 41
42 
-        $onFulFilled = function ($result) use ($accessChecker, $resolveArgs) {
42 
+        $onFulFilled = function($result) use ($accessChecker, $resolveArgs) {
43 43 
             switch (true) {
44 44 
                 case is_array($result):
45 45 
                     $result = array_map(
46 
-                        function ($object) use ($accessChecker, $resolveArgs) {
46 
+                        function($object) use ($accessChecker, $resolveArgs) {
47 47 
                             return $this->hasAccess($accessChecker, $object, $resolveArgs) ? $object : null;
48 48 
                         },
49 49 
                         $result
 
@@ -51,7 +51,7 @@  discard block
 
 block discarded – undo
51 51 
                     break;
52 52 
                 case $result instanceof Connection:
53 53 
                     $result->edges = array_map(
54 
-                        function (Edge $edge) use ($accessChecker, $resolveArgs) {
54 
+                        function(Edge $edge) use ($accessChecker, $resolveArgs) {
55 55 
                             $edge->node = $this->hasAccess($accessChecker, $edge->node, $resolveArgs) ? $edge->node : null;
56 56
57 57 
                             return $edge;
Please login to merge, or discard this patch.