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 (#602)
by
unknown
23:58
created

Executor::getSchemaNameOrDefault()   A

Complexity

Conditions 5
Paths 7

Size

Total Lines 16
Code Lines 8

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 9
CRAP Score 5

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 8
dl 0
loc 16
ccs 9
cts 9
cp 1
rs 9.6111
c 1
b 0
f 0
cc 5
nc 7
nop 1
crap 5
1
<?php
2
3
declare(strict_types=1);
4
5
namespace Overblog\GraphQLBundle\Request;
6
7
use GraphQL\Executor\ExecutionResult;
8
use GraphQL\Executor\Promise\PromiseAdapter;
9
use GraphQL\GraphQL;
10
use GraphQL\Type\Schema;
11
use GraphQL\Validator\DocumentValidator;
12
use GraphQL\Validator\Rules\DisableIntrospection;
13
use GraphQL\Validator\Rules\QueryComplexity;
14
use GraphQL\Validator\Rules\QueryDepth;
15
use Overblog\GraphQLBundle\Event\Events;
16
use Overblog\GraphQLBundle\Event\ExecutorArgumentsEvent;
17
use Overblog\GraphQLBundle\Event\ExecutorContextEvent;
18
use Overblog\GraphQLBundle\Event\ExecutorResultEvent;
19
use Overblog\GraphQLBundle\Executor\ExecutorInterface;
20
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
21
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
22
23
class Executor
24
{
25
    public const PROMISE_ADAPTER_SERVICE_ID = 'overblog_graphql.promise_adapter';
26
27
    private $schemas = [];
28
29
    private $dispatcher;
30
31
    private $promiseAdapter;
32
33
    private $executor;
34
35
    private $defaultFieldResolver;
36
37
    private $useExperimentalExecutor;
38
39 123
    public function __construct(
40
        ExecutorInterface $executor,
41
        PromiseAdapter $promiseAdapter,
42
        EventDispatcherInterface $dispatcher,
43
        ?callable $defaultFieldResolver = null,
44
        bool $useExperimental = false
45
    ) {
46 123
        $this->executor = $executor;
47 123
        $this->promiseAdapter = $promiseAdapter;
48 123
        $this->dispatcher = $dispatcher;
49 123
        $this->defaultFieldResolver = $defaultFieldResolver;
50 123
        $this->useExperimentalExecutor = $useExperimental;
51 123
    }
52
53
    public function setExecutor(ExecutorInterface $executor): self
54
    {
55
        $this->executor = $executor;
56
57
        return $this;
58
    }
59
60 115
    public function addSchemaBuilder(string $name, callable $builder): self
61
    {
62 115
        $this->schemas[$name] = $builder;
63
64 115
        return $this;
65
    }
66
67
    /**
68
     * @param string $name
69
     * @param Schema $schema
70
     *
71
     * @return self
72
     */
73 106
    public function addSchema(string $name, Schema $schema): self
74
    {
75 106
        $this->schemas[$name] = $schema;
76
77 106
        return $this;
78
    }
79
80
    /**
81
     * @param string $name
82
     *
83
     * @return Schema
84
     */
85 109
    public function getSchema(string $name): Schema
86
    {
87 109
        $schema = $this->schemas[$name];
88 1
        if (\is_callable($schema)) {
89
            $schema = $schema();
90
            $this->addSchema($name, $schema);
91 108
        }
92
93 102
        return $schema;
94 102
    }
95
96
    /**
97 108
     * @param string|null $name
98 1
     *
99
     * @return string
100 107
     */
101 107
    public function getSchemaNameOrDefault(?string $name)
102 107
    {
103 106
        if (empty($this->schemas)) {
104
            throw new \RuntimeException('At least one schema should be declare.');
105
        }
106 106
        if (null === $name) {
107
            // TODO(mcg-web): Replace by array_key_first PHP 7 >= 7.3.0.
108
            foreach ($this->schemas as $name => $schema) {
109 122
                break;
110
            }
111
        }
112 122
        if (!isset($this->schemas[$name])) {
113 122
            throw new NotFoundHttpException(\sprintf('Could not found "%s" schema.', $name));
114 122
        }
115
116 122
        return $name;
117
    }
118
119 122
    public function setMaxQueryDepth($maxQueryDepth): void
120 122
    {
121 122
        /** @var QueryDepth $queryDepth */
122
        $queryDepth = DocumentValidator::getRule('QueryDepth');
123 121
        $queryDepth->setMaxQueryDepth($maxQueryDepth);
124
    }
125 121
126 121
    public function setMaxQueryComplexity($maxQueryComplexity): void
127
    {
128 1
        /** @var QueryComplexity $queryComplexity */
129
        $queryComplexity = DocumentValidator::getRule('QueryComplexity');
130 1
        $queryComplexity->setMaxQueryComplexity($maxQueryComplexity);
131 1
    }
132
133
    public function enableIntrospectionQuery(): void
134
    {
135
        DocumentValidator::addRule(new DisableIntrospection(DisableIntrospection::DISABLED));
136
    }
137
138
    public function disableIntrospectionQuery(): void
139
    {
140 105
        DocumentValidator::addRule(new DisableIntrospection());
141
    }
142 105
143
    /**
144 105
     * @param string|null                    $schemaName
145 105
     * @param array                          $request
146 103
     * @param array|\ArrayObject|object|null $rootValue
147 103
     *
148 103
     * @return ExecutionResult
149 103
     */
150 103
    public function execute(?string $schemaName, array $request, $rootValue = null): ExecutionResult
151
    {
152
        $this->useExperimentalExecutor ? GraphQL::useExperimentalExecutor() : GraphQL::useReferenceExecutor();
153 103
154
        $schemaName = $this->getSchemaNameOrDefault($schemaName);
155 103
156 103
        $executorArgumentsEvent = $this->preExecute(
157 103
            $schemaName,
158 103
            $this->getSchema($schemaName),
159 103
            $request[ParserInterface::PARAM_QUERY] ?? null,
160 103
            new \ArrayObject(),
161 103
            $rootValue,
162 103
            $request[ParserInterface::PARAM_VARIABLES],
163 103
            $request[ParserInterface::PARAM_OPERATION_NAME] ?? null
164
        );
165
166 103
        $executorArgumentsEvent->getSchema()->processExtensions();
167
168 102
        $result = $this->executor->execute(
169
            $this->promiseAdapter,
170
            $executorArgumentsEvent->getSchema(),
171 103
            $executorArgumentsEvent->getRequestString(),
172
            $executorArgumentsEvent->getRootValue(),
173
            $executorArgumentsEvent->getContextValue(),
0 ignored issues
show
Bug introduced by
$executorArgumentsEvent->getContextValue() of type ArrayObject is incompatible with the type array|null expected by parameter $contextValue of Overblog\GraphQLBundle\E...torInterface::execute(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

173
            /** @scrutinizer ignore-type */ $executorArgumentsEvent->getContextValue(),
Loading history...
174
            $executorArgumentsEvent->getVariableValue(),
175
            $executorArgumentsEvent->getOperationName(),
176
            $this->defaultFieldResolver
177
        );
178
179 103
        $result = $this->postExecute($result);
180 103
181 103
        return $result;
182
    }
183
184 103
    private function preExecute(
185 103
        string $schemaName,
186 103
        Schema $schema,
187
        ?string $requestString,
188
        \ArrayObject $contextValue,
189
        $rootValue = null,
190 103
        ?array $variableValue = null,
191
        ?string $operationName = null
192 103
    ): ExecutorArgumentsEvent {
193 103
        $this->dispatcher->dispatch(
194 103
            new ExecutorContextEvent($contextValue),
195 102
            Events::EXECUTOR_CONTEXT
0 ignored issues
show
Unused Code introduced by
The call to Symfony\Contracts\EventD...erInterface::dispatch() has too many arguments starting with Overblog\GraphQLBundle\E...vents::EXECUTOR_CONTEXT. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

195
        $this->dispatcher->/** @scrutinizer ignore-call */ 
196
                           dispatch(

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
196
        );
197
198
        return $this->dispatcher->dispatch(
199
            ExecutorArgumentsEvent::create($schemaName, $schema, $requestString, $contextValue, $rootValue, $variableValue, $operationName),
200
            Events::PRE_EXECUTOR
201
        );
202
    }
203
204
    private function postExecute(ExecutionResult $result): ExecutionResult
205
    {
206
        return $this->dispatcher->dispatch(
207
            new ExecutorResultEvent($result),
208
            Events::POST_EXECUTOR
0 ignored issues
show
Unused Code introduced by
The call to Symfony\Contracts\EventD...erInterface::dispatch() has too many arguments starting with Overblog\GraphQLBundle\Event\Events::POST_EXECUTOR. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

208
        return $this->dispatcher->/** @scrutinizer ignore-call */ dispatch(

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
209
        )->getResult();
210
    }
211
}
212