Ynlo\GraphQLBundle\Request\MultiPartFileUploadMiddleware::processRequest()

Complexity

Conditions	9
Paths	7

Size

Total Lines	24
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	90

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function processRequest(Request $request, ExecuteQuery $query): void
    {
        //TODO: add support for batching
        if (mb_stripos($request->headers->get('Content-Type'), 'multipart/form-data') !== false) {
            if (($operations = $request->get('operations')) && ($map = $request->get('map'))) {
                $result = json_decode($operations, true);
                $map = json_decode($map, true);
                foreach ($map as $fileKey => $locations) {
                    foreach ($locations as $location) {
                        $items = &$result;
                        foreach (explode('.', $location) as $key) {
                            if (!isset($items[$key]) || !is_array($items[$key])) {
                                $items[$key] = [];
                            }
                            $items = &$items[$key];
                        }

                        $items = $request->files->get($fileKey);
                    }
                }

                $query->setRequestString($result['query'] ?? null);

It seems like $result['query'] ?? null can also be of type null; however, parameter $requestString of Ynlo\GraphQLBundle\Request\ExecuteQuery::setRequestString() does only seem to accept string, maybe add an additional type check?

                $query->setVariables($result['variables'] ?? []);
                $query->setOperationName($result['operationName'] ?? null);
            }