Acelaya\ExpressiveErrorHandler\ErrorHandler\ContentBasedErrorResponseGenerator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	__invoke()	0	7	1
B	resolveErrorHandlerFromAcceptHeader()	0	27	5

<?php
namespace Acelaya\ExpressiveErrorHandler\ErrorHandler;

use Acelaya\ExpressiveErrorHandler\Exception\InvalidArgumentException;
use Acelaya\ExpressiveErrorHandler\Log\LogMessageBuilderInterface;
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface as Request;
use Psr\Log\LoggerInterface;

class ContentBasedErrorResponseGenerator implements ErrorResponseGeneratorInterface
{
    /**
     * @deprecated Inject the default content type to be used while creating this class
     */
    const DEFAULT_CONTENT = 'text/html';

    /**
     * @var ErrorResponseGeneratorManagerInterface
     */
    private $errorHandlerManager;
    /**
     * @var LoggerInterface
     */
    private $logger;
    /**
     * @var LogMessageBuilderInterface
     */
    private $logMessageBuilder;
    /**
     * @var string
     */
    private $defaultContentType;

    /**
     * ContentBasedErrorResponseGenerator constructor.
     * @param ErrorResponseGeneratorManagerInterface|ErrorResponseGeneratorManager $errorHandlerManager
     * @param LoggerInterface $logger
     * @param LogMessageBuilderInterface $logMessageBuilder
     * @param string $defaultContentType
     */
    public function __construct(
        ErrorResponseGeneratorManagerInterface $errorHandlerManager,
        LoggerInterface $logger,
        LogMessageBuilderInterface $logMessageBuilder,
        $defaultContentType = 'text/html'
    ) {
        $this->errorHandlerManager = $errorHandlerManager;
        $this->logger = $logger;
        $this->logMessageBuilder = $logMessageBuilder;
        $this->defaultContentType = $defaultContentType;
    }

    /**
     * Final handler for an application.
     *
     * @param \Throwable|\Exception $e
     * @param Request $request
     * @param Response $response
     * @return Response
     */
    public function __invoke($e, Request $request, Response $response)
    {
        // Try to get an error handler for provided request accepted type
        $errorHandler = $this->resolveErrorHandlerFromAcceptHeader($request);
        $this->logger->error($this->logMessageBuilder->buildMessage($request, $response, $e));

It seems like $e defined by parameter $e on line 61 can also be of type object<Exception>; however, Acelaya\ExpressiveErrorHandler\Log\LogMessageBuilderInterface::buildMessage() does only seem to accept null, maybe add an additional type check?

        return $errorHandler($e, $request, $response);
    }

    /**
     * Tries to resolve
     *
     * @param Request $request
     * @return callable
     * @throws InvalidArgumentException
     */
    protected function resolveErrorHandlerFromAcceptHeader(Request $request)
    {
        // Try to find an error handler for one of the accepted content types
        $accepts = $request->hasHeader('Accept') ? $request->getHeaderLine('Accept') : $this->defaultContentType;
        /** @var array $accepts */
        $accepts = explode(',', $accepts);
        foreach ($accepts as $accept) {
            if (! $this->errorHandlerManager->has($accept)) {
                continue;
            }

            return $this->errorHandlerManager->get($accept);
        }

        // If it wasn't possible to find an error handler for accepted content type, use default one if registered
        if ($this->errorHandlerManager->has($this->defaultContentType)) {
            return $this->errorHandlerManager->get($this->defaultContentType);
        }

        // It wasn't possible to find an error handler
        throw new InvalidArgumentException(sprintf(
            'It wasn\'t possible to find an error handler for ["%s"] content types. '
            . 'Make sure you have registered at least the default "%s" content type',
            implode('", "', $accepts),
            $this->defaultContentType
        ));
    }
}