SMartins\Exceptions\Response\AbstractResponse

Complexity

Total Complexity

4

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getErrors()	0	3	1
A	setStatus()	0	5	1
A	getStatus()	0	3	1

<?php

namespace SMartins\Exceptions\Response;

use Illuminate\Http\JsonResponse;

abstract class AbstractResponse
{
    /**
     * The HTTP status code.
     *
     * @var int
     */
    protected $status;

    /**
     * The errors on response.
     *
     * @var \SMartins\Exceptions\JsonApi\ErrorCollection
     */
    protected $errors;

    /**
     * Create new JsonApi response passing the errors.
     *
     * @param \SMartins\Exceptions\JsonApi\ErrorCollection $errors
     *
     */
    public function __construct(ErrorHandledCollectionInterface $errors)
    {
        $this->errors = $errors;

$errors is of type SMartins\Exceptions\Response\ErrorHandledCollectionInterface, but the property $errors was declared to be of type SMartins\Exceptions\JsonApi\ErrorCollection. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->setStatus((int) $this->errors->getStatusCode());
    }

    /**
     * Get the HTTP status code.
     *
     * @return  int
     */
    public function getStatus()
    {
        return $this->status;
    }

    /**
     * Set the HTTP status code.
     *
     * @param  int  $status  The HTTP status code.
     *
     * @return  self
     */
    public function setStatus(int $status)
    {
        $this->status = $status;

        return $this;
    }

    /**
     * Get the errors on response.
     *
     * @return \SMartins\Exceptions\JsonApi\ErrorCollection
     */
    public function getErrors()
    {
        return $this->errors;
    }

    /**
     * Returns JSON response.
     *
     * @return \Illuminate\Http\JsonResponse
     */
    abstract public function json(): JsonResponse;
}