NilPortugues\Api\JSend\Http\Message\ErrorResponse

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	19	2

<?php

/**
 * Author: Nil Portugués Calderó <contact@nilportugues.com>
 * Date: 7/27/15
 * Time: 11:52 PM.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace NilPortugues\Api\JSend\Http\Message;

use NilPortugues\Api\Http\Message\AbstractResponse;

/**
 * Class ErrorResponse.
 */
class ErrorResponse extends AbstractResponse
{
    /**
     * @param string $message
     * @param int    $code
     * @param null   $data
     */
    public function __construct($message, $code = 500, $data = null)
    {
        $body = json_encode(
            array_filter(
                [
                    'status' => 'error',
                    'code' => $code,
                    'message' => (string) $message,
                    'data' => '',
                ]
            )
        );

        if ($data) {
            $body = str_replace('"data": ""', substr(substr($data, 1), 0, -1), $body);
        }

        $this->response = parent::instance($body, 500, $this->headers);

It seems like you call parent on a different method (instance() instead of __construct()). Are you sure this is correct? If so, you might want to change this to $this->instance().

    }
}