hiqdev\hiart\AbstractResponse

Complexity

Total Complexity

10

Size/Duplication

Total Lines	98
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

78.26%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getRequest()	0	4	1
A	getData()	0	9	2
A	decodeData()	0	18	3
	getRawData()	0	1	?
A	isRaw()	0	4	1
A	isJson()	0	9	2
	getHeader()	0	1	?
A	decodeJson()	0	4	1

<?php
/**
 * ActiveRecord for API
 *
 * @link      https://github.com/hiqdev/yii2-hiart
 * @package   yii2-hiart
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2015-2017, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\hiart;

use yii\helpers\Json;

abstract class AbstractResponse implements ResponseInterface
{
    /**
     * @var RequestInterface
     */
    protected $request;

    /**
     * @var string response data. The property contains RAW response data
     * @see decodeData()
     * @see isDecoded
     */
    protected $data;

    /**
     * @var bool whether response is already decoded
     */
    protected $isDecoded = false;

    public function getRequest()
    {
        return $this->request;
    }

    public function getData()
    {
        if (!$this->isDecoded) {
            $this->data = $this->decodeData();

It seems like $this->decodeData() can also be of type array. However, the property $data is declared as type string. Maybe add an additional type check?

            $this->isDecoded = true;
        }

        return $this->data;
    }

    public function decodeData()
    {
        $data = $this->getRawData();

        if ($this->isRaw()) {
            return $data;
        }

        if ($this->isJson()) {
            return $this->decodeJson($data);
        }

        // TODO: implement decoding for XML and other types

        // throw new ResponseDecodingException('Failed to detect response data type', $this);

62% of this comment could be valid code. Did you maybe forget this after debugging?

        // TODO: throw exception instead of returning
        return $data;
    }

    /**
     * Method returns RAW request data.
     *
     * @return string
     */
    abstract public function getRawData();

    /**
     * Whether the request is RAW and should not be decoded.
     * @return bool
     */
    public function isRaw()
    {
        return $this->request->isRaw();
    }

    /**
     * Method checks whether response is a JSON response.
     * @return bool
     */
    public function isJson()
    {
        $value = $this->getHeader('Content-Type');
        if ($value === null) {
            return false;
        }

        return !empty(preg_grep('|application/json|i', $value));
    }

    /**
     * @param $name
     * @return array
     */
    abstract public function getHeader($name);

    /**
     * @param string $data JSON data
     * @return array
     */
    protected function decodeJson($data)
    {
        return Json::decode($data);
    }
}