Controlabs\Http\ResponseMessage

Complexity

Total Complexity

4

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	toArray()	0	10	2
A	toJson()	0	4	1

<?php

namespace Controlabs\Http;

class ResponseMessage
{
    private $code;
    private $message;
    private $data = [];

    public function __construct(int $code, ?string $message, array $data = [])
    {
        $this->code = $code;
        $this->message = $message;
        $this->data = $data;
    }

    public function toArray()
    {
        $error = [];
        $error['error'] = [
            'code' => $this->code,
            'message' => $this->message,
        ];
        ($this->data) && $error['error']['errors'] = $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.

        return $error;
    }

    public function toJson()
    {
        return json_encode($this->toArray());
    }
}