Loevgaard\Consignor\ShipmentServer\Request\Request

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getHeaders()	0	4	1
A	getBody()	0	13	3

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

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

abstract class Request implements RequestInterface
{
    /**
     * @var array
     */
    protected $data;

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

    public function getHeaders() : array
    {
        return [];
    }

    /**
     * @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;
    }
}