Loevgaard\Consignor\ShipmentServer\Request\Request

Complexity

Total Complexity

11

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getBody()	0	13	3
A	getResponseClass()	0	4	1
A	getData()	0	4	1
A	setData()	0	5	1
A	getOption()	0	8	3
A	getOptions()	0	4	1
A	setOptions()	0	5	1

<?php
namespace Loevgaard\Consignor\ShipmentServer\Request;

use Loevgaard\Consignor\ShipmentServer\Exception\EncodeJsonException;
use Loevgaard\Consignor\ShipmentServer\Response\Response;
use function Loevgaard\Consignor\ShipmentServer\encodeJson;

abstract class Request implements RequestInterface
{
    /*
     * Both $data and $options are only used in some requests, but we include them here to avoid code duplication
     */
    /**
     * @var array
     */
    protected $data;

    /**
     * @var array
     */
    protected $options;

    /**
     * @return array
     * @throws EncodeJsonException
     */
    public function getBody() : array
    {
        $body = [];
        if ($this->data) {

The expression $this->data of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $body['data'] = encodeJson($this->data);
        }

        if ($this->options) {

The expression $this->options of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $body['Options'] = encodeJson($this->options);
        }

        return $body;
    }

    public function getResponseClass(): string
    {
        return Response::class;
    }

    /**
     * @return array
     */
    public function getData(): array
    {
        return (array)$this->data;
    }

    /**
     * @param array $data
     * @return Request
     */
    public function setData(array $data)
    {
        $this->data = $data;
        return $this;
    }

    public function getOption(string $option)
    {
        if (!$this->options || !array_key_exists($option, $this->options)) {

The expression $this->options of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return null;
        }

        return $this->options[$option];
    }

    /**
     * @return array
     */
    public function getOptions(): array
    {
        return (array)$this->options;
    }

    /**
     * @param array $options
     * @return Request
     */
    public function setOptions(array $options)
    {
        $this->options = $options;
        return $this;
    }
}