ExceptionRenderer::devRender() - Code Metrics - Inspection of "Add Debug tests" - abterphp/framework - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 22ee9f...77fae7 )

by Peter

created 2021-03-08 21:44 UTC

ExceptionRenderer::devRender() B

↳ Parent: ExceptionRenderer

Complexity

Conditions	6
Paths	14

Size

Total Lines	70
Code Lines	54

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	6
eloc	54
c	0
b	0
f	0
nc	14
nop	1
dl	0
loc	70
rs	8.3814

How to fix Long Method

<?php

namespace AbterPhp\Framework\Debug\Exceptions\Handlers\Whoops;

use Exception;
use Opulence\Framework\Debug\Exceptions\Handlers\Http;
use Opulence\Http\HttpException;
use Throwable;
use Whoops\RunInterface;

/**
 * @SuppressWarnings(PHPMD)
 */
class ExceptionRenderer extends Http\ExceptionRenderer implements Http\IExceptionRenderer
{
    /** @var RunInterface */
    protected $run;

    /**
     * @return bool
     */
    public function isInDevelopmentEnvironment(): bool
    {
        return $this->inDevelopmentEnvironment;
    }

    /**
     * @param bool $inDevelopmentEnvironment
     *
     * @return $this
     */
    public function setInDevelopmentEnvironment(bool $inDevelopmentEnvironment): self
    {
        $this->inDevelopmentEnvironment = $inDevelopmentEnvironment;

        return $this;
    }

    /**
     * WhoopsRenderer constructor.
     *
     * @param RunInterface $run
     */
    public function __construct(RunInterface $run, bool $inDevelopmentEnvironment = false)
    {
        $this->run = $run;

        parent::__construct($inDevelopmentEnvironment);
    }

    /**
     * @return RunInterface
     */
    public function getRun()
    {
        return $this->run;
    }

    /**
     * Renders an exception
     *
     * @param Throwable|Exception $ex The thrown exception
     */
    public function render($ex)
    {
        if (!$this->inDevelopmentEnvironment) {
            $this->run->writeToOutput(false);

            $this->run->unregister();

            $this->devRender($ex);

            return;
        }

        $this->run->handleException($ex);
    }

    public function devRender($ex)
    {
        // Add support for HTTP library without having to necessarily depend on it
        if ($ex instanceof HttpException) {
            $statusCode = $ex->getStatusCode();
            $headers    = $ex->getHeaders();
        } else {
            $statusCode = 500;
            $headers    = [];
        }

        // Always get the content, even if headers are sent, so that we can unit test this
        $content = $ex->getMessage();

        if (!headers_sent()) {
            header("HTTP/1.1 $statusCode", true, $statusCode);

            switch ($this->getRequestFormat()) {
                case 'json':
                    $headers['Content-Type'] = 'application/json';
                    break;
                default:
                    $content                 = sprintf(
                        '<!DOCTYPE html>
<html>
    <head>
        <meta name="viewport" content="initial-scale=1"/>
        <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.4.1/css/bootstrap.min.css"
            integrity="sha384-Vkoo8x4CGsO3+Hhxv8T/Q5PaXtkKtu6ug5TOeNV6gBiFeWPGFN9MuhOf23Q9Ifjh"
            crossorigin="anonymous">
    </head>
    <body>
        <main class="main">
            <div class="container">
                <div class="row">
                    <div class="col-sm">
                        <div class="alert alert-danger" role="alert">
                            <h4 class="alert-heading">Exception: %s</h4>
                            <p>%s</p>
                            <hr>
                            <p class="mb-0">
                                <a href="#" class="btn btn-danger btn-lg active" role="button" aria-pressed="true"
                                onclick="location.reload(); return false;">Try again</a>
                                <a href="#" class="btn btn-primary btn-lg active" role="button" aria-pressed="true"
                                onclick="window.history.back(); return false;">Back to previous page</a>
                            </p>
                        </div>
                    </div>
                </div>
            </div>
        </main>
    </body>
</html>',
                        get_class($ex),
                        $ex->getMessage()
                    );
                    $headers['Content-Type'] = 'text/html';
            }

            foreach ($headers as $name => $values) {
                $values = (array)$values;

                foreach ($values as $value) {
                    header("$name:$value", false, $statusCode);
                }
            }

            echo $content;
            // To prevent any potential output buffering, let's flush
            flush();
        }
    }
}


1			<?php
2
3			namespace AbterPhp\Framework\Debug\Exceptions\Handlers\Whoops;
4
5			use Exception;
6			use Opulence\Framework\Debug\Exceptions\Handlers\Http;
7			use Opulence\Http\HttpException;
8			use Throwable;
9			use Whoops\RunInterface;
10
11			/**
12			* @SuppressWarnings(PHPMD)
13			*/
14			class ExceptionRenderer extends Http\ExceptionRenderer implements Http\IExceptionRenderer
15			{
16			/** @var RunInterface */
17			protected $run;
18
19			/**
20			* @return bool
21			*/
22			public function isInDevelopmentEnvironment(): bool
23			{
24			return $this->inDevelopmentEnvironment;
25			}
26
27			/**
28			* @param bool $inDevelopmentEnvironment
29			*
30			* @return $this
31			*/
32			public function setInDevelopmentEnvironment(bool $inDevelopmentEnvironment): self
33			{
34			$this->inDevelopmentEnvironment = $inDevelopmentEnvironment;
35
36			return $this;
37			}
38
39			/**
40			* WhoopsRenderer constructor.
41			*
42			* @param RunInterface $run
43			*/
44			public function __construct(RunInterface $run, bool $inDevelopmentEnvironment = false)
45			{
46			$this->run = $run;
47
48			parent::__construct($inDevelopmentEnvironment);
49			}
50
51			/**
52			* @return RunInterface
53			*/
54			public function getRun()
55			{
56			return $this->run;
57			}
58
59			/**
60			* Renders an exception
61			*
62			* @param Throwable\|Exception $ex The thrown exception
63			*/
64			public function render($ex)
65			{
66			if (!$this->inDevelopmentEnvironment) {
67			$this->run->writeToOutput(false);
68
69			$this->run->unregister();
70
71			$this->devRender($ex);
72
73			return;
74			}
75
76			$this->run->handleException($ex);
77			}
78
79			public function devRender($ex)
80			{
81			// Add support for HTTP library without having to necessarily depend on it
82			if ($ex instanceof HttpException) {
83			$statusCode = $ex->getStatusCode();
84			$headers = $ex->getHeaders();
85			} else {
86			$statusCode = 500;
87			$headers = [];
88			}
89
90			// Always get the content, even if headers are sent, so that we can unit test this
91			$content = $ex->getMessage();
92
93			if (!headers_sent()) {
94			header("HTTP/1.1 $statusCode", true, $statusCode);
95
96			switch ($this->getRequestFormat()) {
97			case 'json':
98			$headers['Content-Type'] = 'application/json';
99			break;
100			default:
101			$content = sprintf(
102			'<!DOCTYPE html>
103			<html>
104			<head>
105			<meta name="viewport" content="initial-scale=1"/>
106			<link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.4.1/css/bootstrap.min.css"
107			integrity="sha384-Vkoo8x4CGsO3+Hhxv8T/Q5PaXtkKtu6ug5TOeNV6gBiFeWPGFN9MuhOf23Q9Ifjh"
108			crossorigin="anonymous">
109			</head>
110			<body>
111			<main class="main">
112			<div class="container">
113			<div class="row">
114			<div class="col-sm">
115			<div class="alert alert-danger" role="alert">
116			<h4 class="alert-heading">Exception: %s</h4>
117			<p>%s</p>
118			<hr>
119			<p class="mb-0">
120			<a href="#" class="btn btn-danger btn-lg active" role="button" aria-pressed="true"
121			onclick="location.reload(); return false;">Try again</a>
122			<a href="#" class="btn btn-primary btn-lg active" role="button" aria-pressed="true"
123			onclick="window.history.back(); return false;">Back to previous page</a>
124			</p>
125			</div>
126			</div>
127			</div>
128			</div>
129			</main>
130			</body>
131			</html>',
132			get_class($ex),
133			$ex->getMessage()
134			);
135			$headers['Content-Type'] = 'text/html';
136			}
137
138			foreach ($headers as $name => $values) {
139			$values = (array)$values;
140
141			foreach ($values as $value) {
142			header("$name:$value", false, $statusCode);
143			}
144			}
145
146			echo $content;
147			// To prevent any potential output buffering, let's flush
148			flush();
149			}
150			}
151			}
152

abterphp / framework

Push — main ( 22ee9f...77fae7 )

ExceptionRenderer::devRender() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like