Bone\Mvc\Router\Decorator\ExceptionDecorator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	87
Duplicated Lines	9.2 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	setView()	0	4	1
B	process()	0	28	6
A	getResponseWithBodyAndStatus()	8	8	1

<?php

namespace Bone\Mvc\Router\Decorator;

use Bone\Mvc\View\ViewEngine;
use Bone\Traits\HasLayoutTrait;
use Exception;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Server\MiddlewareInterface;
use Psr\Http\Server\RequestHandlerInterface;
use Zend\Diactoros\Response\HtmlResponse;
use Zend\Diactoros\Stream;

class ExceptionDecorator implements MiddlewareInterface
{

    use HasLayoutTrait;

    /** @var ViewEngine  */
    protected $viewEngine;

    /** @var string $view */
    protected $view;

    /**
     * @var array $templates
     */
    protected $templates;

    /**
     * ExceptionDecorator constructor.
     * @param ViewEngine $viewEngine
     * @param array $templates
     */
    public function __construct(ViewEngine $viewEngine, array $templates)
    {
        $this->viewEngine = $viewEngine;
        $this->templates = $templates;
        $this->view = 'error/error';
    }

    /**
     * @param string $view
     */
    protected function setView(string $view)
    {
        $this->view = $view;
    }

    /**
     * @param ServerRequestInterface $request
     * @param RequestHandlerInterface $handler
     * @return ResponseInterface
     */
    public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
    {
        try {
            return $handler->handle($request);
        } catch (Exception $e) {

            $template = 'error/error';
            $code = $e->getCode();

            if (array_key_exists($code, $this->templates)) {
                $template = $this->templates[$code];
            } elseif (array_key_exists('exception', $this->templates)) {
                $template = $this->templates['exception'];
            }

            $body = $this->viewEngine->render($template, [
                'message' => $e->getMessage(),
                'code' => $e->getCode(),
                'trace' => $e->getTrace(),
            ]);
            $body = $this->viewEngine->render($this->layout, [
                'content' => $body,
            ]);
            $status = ($e->getCode() >= 100 && $e->getCode() < 600) ? $e->getCode() : 500;

            return $this->getResponseWithBodyAndStatus(new HtmlResponse(), $body, $status);

The call to HtmlResponse::__construct() misses a required argument $html.

        }
    }



    /**
     * @param ResponseInterface $response
     * @param string $body
     * @param int $status
     * @return \Psr\Http\Message\MessageInterface|Response
     */
    protected function getResponseWithBodyAndStatus(ResponseInterface $response, string $body, int $status = 200)

This method seems to be duplicated in your project.

    {
        $stream = new Stream('php://memory', 'r+');
        $stream->write($body);
        $response = $response->withStatus($status)->withBody($stream);

        return $response;
    }
}