Equip\Exception\HttpException

Complexity

Total Complexity

5

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

4 Methods

Rating	Name	Duplication	Size	Complexity
A	notFound()	0	7	1
A	methodNotAllowed()	0	12	1
A	badRequest()	0	7	1
A	withResponse()	0	8	2

<?php

namespace Equip\Exception;

use RuntimeException;
use Psr\Http\Message\ResponseInterface;

class HttpException extends RuntimeException
{
    /**
     * @param string $path
     *
     * @return static
     */
    public static function notFound($path)
    {
        return new static(sprintf(
            'Cannot find any resource at `%s`',
            $path
        ), 404);
    }

    /**
     * @param string $path
     * @param string $method
     * @param array $allowed
     *
     * @return static
     */
    public static function methodNotAllowed($path, $method, array $allowed)
    {
        $error = new static(sprintf(
            'Cannot access resource `%s` using method `%s`',
            $path,
            $method
        ), 405);

        $error->allowed = $allowed;

        return $error;
    }

    /**
     * @param string $message
     *
     * @return static
     */
    public static function badRequest($message)
    {
        return new static(sprintf(
            'Cannot parse the request: %s',
            $message
        ), 400);
    }

    /**
     * @var array
     */
    private $allowed = [];

    /**
     * @param ResponseInterface $response
     *
     * @return ResponseInterface
     */
    public function withResponse(ResponseInterface $response)
    {
        if ($this->allowed) {

The expression $this->allowed 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.

            $response = $response->withHeader('Allow', implode(',', $this->allowed));
        }

        return $response;
    }
}