Sourceout\LastFm\Http\Response::send()

Complexity

Conditions	6
Paths	6

Size

Total Lines

24

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	6

Importance

Changes

0

<?php
    public static function send(
        PsrResponseInterface $response
    ) : Collection
    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $returnValue = new Collection();

The assignment to $returnValue is dead and can be removed.

        if (strpos($response->getHeaderLine('Content-Type'), 'application/json') !== false) {
            $contents = json_decode($response->getBody()->getContents(), true);
            if (
                ($response->getStatusCode() === 200)
                && ! isset($contents['error'])
            ) {
                $returnValue = new Collection($contents);
            } else {
                $code = isset($contents['error'])?$contents['error']:0;
                $message = isset($contents['message'])?$contents['message']:'';
                throw new LastFmException($message, $code);
            }
        } else {
            $responseType = $response->getHeaderLine('Content-Type');
            throw new \InvalidArgumentException("Invalid response type {$responseType}");
        }

        return $returnValue;
    }
}