Phprest\Util\Mime::processMime()

Complexity

Conditions	5
Paths	5

Size

Total Lines	32
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	17
CRAP Score	5.0042

Importance

Changes	4
Bugs	0	Features	0

<?php
    protected function processMime(string $mime): MimeProcessResult
    {
        $vendor = (string)$this->getContainer()->get(Application::CONTAINER_ID_VENDOR);
        $apiVersion = (string)$this->getContainer()->get(Application::CONTAINER_ID_API_VERSION);
        $apiVersionRegExp = Application::API_VERSION_REG_EXP;
        $format = null;

        if (
            preg_match(
                '#application/vnd\.' . $vendor . '-v' . $apiVersionRegExp . '\+(xml|json)#',
                $mime,
                $matches
            )
        ) {
            [$mime, $apiVersion, $format] = $matches;
        } elseif (
            preg_match(
                '#application/vnd\.' . $vendor . '\+(xml|json).*?version=' . $apiVersionRegExp . '#',
                $mime,
                $matches
            )
        ) {
            [$mime, $format, $apiVersion] = $matches;
        } elseif ('application/json' === $mime) {
            $format = 'json';
            $mime = 'application/vnd.' . $vendor . '-v' . $apiVersion . '+json';
        } elseif ('application/xml' === $mime) {
            $format = 'xml';
            $mime = 'application/vnd.' . $vendor . '-v' . $apiVersion . '+xml';
        }

        return new MimeProcessResult($mime, $vendor, $apiVersion, $format);

It seems like $format can also be of type null; however, parameter $format of Phprest\Util\DataStructure\MimeProcessResult::__construct() does only seem to accept string, maybe add an additional type check?

    }