Loevgaard\Consignor\ShipmentServer\Request\Request

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

71.43%

Importance

Changes

0

1 Method

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

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

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

abstract class Request implements RequestInterface
{
    /**
     * @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;
    }
}