Jasny\ErrorHandler\Middleware

Complexity

Total Complexity

7

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	__invoke()	0	24	5
A	errorResponse()	0	7	1

<?php

namespace Jasny\ErrorHandler;

use Jasny\ErrorHandler;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Message\ResponseInterface;

/**
 * Use error handler as middleware
 */
class Middleware
{
    /**
     * @var ErrorHandler
     */
    protected $errorHandler;
    
    /**
     * Class constructor
     * 
     * @param ErrorHandler $errorHandler
     */
    public function __construct(ErrorHandler $errorHandler)
    {
        $this->errorHandler = $errorHandler;
    }
    
    /**
     * Run middleware action
     *
     * @param ServerRequestInterface $request
     * @param ResponseInterface      $response
     * @param callback               $next
     * @return ResponseInterface
     */
    public function __invoke(ServerRequestInterface $request, ResponseInterface $response, $next)
    {
        if (!is_callable($next)) {
            throw new \InvalidArgumentException("'next' should be a callback");            
        }

        try {
            $nextResponse = $next($request, $response);
            $error = null;
        } catch (\Error $e) {

The class Error does not exist. Did you forget a USE statement, or did you not list all dependencies?

            $error = $e;
        } catch (\Exception $e) {
            $error = $e;
        }
        
        $this->errorHandler->setError($error);
        
        if ($error) {
            $this->errorHandler->log($error);
            $nextResponse = $this->errorResponse($request, $response);
        }
        
        return $nextResponse;
    }

    /**
     * Handle caught error
     *
     * @param ServerRequestInterface $request
     * @param ResponseInterface      $response
     * @return ResponseInterface
     */
    protected function errorResponse(ServerRequestInterface $request, ResponseInterface $response)

The parameter $request is not used and could be removed.

    {
        $errorResponse = $response->withStatus(500);
        $errorResponse->getBody()->write('Unexpected error');

        return $errorResponse;
    }
}