Error - Code Metrics - refatalsakka/framework - Measure and Improve Code Quality continuously with Scrutinizer

Error A
last analyzed 2020-11-24 15:34 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	147
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

Metric	Value
dl	0
loc	147
rs	10
c	0
b	0
f	0
wmc	14
lcom	2
cbo	4

9 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	allowDisplayingError()	4	2
A	showError()	4	1
A	hideError()	4	1
A	start()	11	2
A	sendErrorToAdmin()	7	1
A	whoops()	15	2
A	displayFriendlyMessage()	10	1
A	handleErrors()	19	3

<?php

namespace System;

use Whoops\Run as Whoops;
use Whoops\Util\Misc as WhoopsMisc;
use Whoops\Handler\JsonResponseHandler;
use Whoops\Handler\PrettyPageHandler;
use DateTime;
use DateTimeZone;

class Error
{
    /**
     * Application Object
     *
     * @var \System\Application
     */
    private $app;

    /**
     * Constructor
     *
     * @param \System\Application $app
     */
    public function __construct(Application $app)
    {
        $this->app = $app;
    }

    /**
     * Check if the errors should be displayed
     *
     * @return bool
     */
    public function allowDisplayingError()
    {
        return (bool) ($_ENV['APP_ENV'] == 'dev' && $_ENV['APP_DEBUG'] == 'true');
    }

    /**
     * Show error
     *
     * @return void
     */
    private function showError()
    {
        error_reporting(E_ALL);
    }

    /**
     * Hide error
     *
     * @return void
     */
    private function hideError()
    {
        error_reporting(0);
    }

    /**
     * Show or hide errors depend on the condition
     *
     * @return mixed
     */
    public function start()
    {
        if ($this->allowDisplayingError()) {
            $this->showError();

            $this->whoops();

            return;
        }
        return $this->hideError();
    }

    /**
     * Send Email to the admin contain the Error
     * and the date
     *
     * @property object $email
     * @param string $error
     * @return void
     */
    private function sendErrorToAdmin($error)
    {
        $date = new DateTime('now', new DateTimeZone('Europe/Berlin'));
        $date = 'Error: ' . $date->format('d.m.Y H:i:s');

        $this->app->email->recipients(['admin' => $_ENV['EMAIL_ADMIN']])->content(true, $date, $error, $error)->send();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    /**
     * Run error handling of Whoops
     *
     * @return void
     */
    private function whoops()
    {
        $run = new Whoops();

        $run->prependHandler(new PrettyPageHandler());

        if (WhoopsMisc::isAjaxRequest()) {
            $jsonHandler = new JsonResponseHandler();

            $jsonHandler->setJsonApi(true);

            $run->prependHandler($jsonHandler);
        }
        $run->register();
    }

    /**
     * Display friendly error to the users
     *
     * @property object $view
     * @return void
     */
    private function displayFriendlyMessage()

    {
        $title = $this->app->msg->error('title');
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

        $context = [
            'title' => $title,
        ];
        echo $this->app->view->render('website/pages/error', $context);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        exit;
    }

    /**
     * Check for last errors
     * if there are errors than send continue to report the admin
     * and display a friendly error to the users
     *
     * @return void
     */
    public function handleErrors()
    {
        $error = error_get_last() || null;

        if (!$error) {
            return;
        }

        $type = $error['type'];
        $message = $error['message'];
        $file = $error['file'];
        $line = $error['line'];

        $error = "There is an Error type: {$type}. says: $message. in file: $file. on line: $line.";

        // $this->displayFriendlyMessage();

        $this->sendErrorToAdmin($error);
    }
}


1			<?php
2
3			namespace System;
4
5			use Whoops\Run as Whoops;
6			use Whoops\Util\Misc as WhoopsMisc;
7			use Whoops\Handler\JsonResponseHandler;
8			use Whoops\Handler\PrettyPageHandler;
9			use DateTime;
10			use DateTimeZone;
11
12			class Error
13			{
14			/**
15			* Application Object
16			*
17			* @var \System\Application
18			*/
19			private $app;
20
21			/**
22			* Constructor
23			*
24			* @param \System\Application $app
25			*/
26			public function __construct(Application $app)
27			{
28			$this->app = $app;
29			}
30
31			/**
32			* Check if the errors should be displayed
33			*
34			* @return bool
35			*/
36			public function allowDisplayingError()
37			{
38			return (bool) ($_ENV['APP_ENV'] == 'dev' && $_ENV['APP_DEBUG'] == 'true');
39			}
40
41			/**
42			* Show error
43			*
44			* @return void
45			*/
46			private function showError()
47			{
48			error_reporting(E_ALL);
49			}
50
51			/**
52			* Hide error
53			*
54			* @return void
55			*/
56			private function hideError()
57			{
58			error_reporting(0);
59			}
60
61			/**
62			* Show or hide errors depend on the condition
63			*
64			* @return mixed
65			*/
66			public function start()
67			{
68			if ($this->allowDisplayingError()) {
69			$this->showError();
70
71			$this->whoops();
72
73			return;
74			}
75			return $this->hideError();
76			}
77
78			/**
79			* Send Email to the admin contain the Error
80			* and the date
81			*
82			* @property object $email
83			* @param string $error
84			* @return void
85			*/
86			private function sendErrorToAdmin($error)
87			{
88			$date = new DateTime('now', new DateTimeZone('Europe/Berlin'));
89			$date = 'Error: ' . $date->format('d.m.Y H:i:s');
90
91			$this->app->email->recipients(['admin' => $_ENV['EMAIL_ADMIN']])->content(true, $date, $error, $error)->send();
			0 ignored issues – show Documentation introduced 2020-09-16 07:57 UTC by Report Bug Copy Issue Report The property `email` does not exist on `object<System\Application>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
92			}
93
94			/**
95			* Run error handling of Whoops
96			*
97			* @return void
98			*/
99			private function whoops()
100			{
101			$run = new Whoops();
102
103			$run->prependHandler(new PrettyPageHandler());
104
105			if (WhoopsMisc::isAjaxRequest()) {
106			$jsonHandler = new JsonResponseHandler();
107
108			$jsonHandler->setJsonApi(true);
109
110			$run->prependHandler($jsonHandler);
111			}
112			$run->register();
113			}
114
115			/**
116			* Display friendly error to the users
117			*
118			* @property object $view
119			* @return void
120			*/
121			private function displayFriendlyMessage()
			0 ignored issues – show Unused Code introduced 2020-11-24 15:34 UTC by Report Bug Copy Issue Report This method is not used, and could be removed. Loading history...
122			{
123			$title = $this->app->msg->error('title');
			0 ignored issues – show Documentation introduced 2020-11-24 15:34 UTC by Report Bug Copy Issue Report The property `msg` does not exist on `object<System\Application>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
124
125			$context = [
126			'title' => $title,
127			];
128			echo $this->app->view->render('website/pages/error', $context);
			0 ignored issues – show Documentation introduced 2020-09-16 07:57 UTC by Report Bug Copy Issue Report The property `view` does not exist on `object<System\Application>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
129			exit;
130			}
131
132			/**
133			* Check for last errors
134			* if there are errors than send continue to report the admin
135			* and display a friendly error to the users
136			*
137			* @return void
138			*/
139			public function handleErrors()
140			{
141			$error = error_get_last() \|\| null;
142
143			if (!$error) {
144			return;
145			}
146
147			$type = $error['type'];
148			$message = $error['message'];
149			$file = $error['file'];
150			$line = $error['line'];
151
152			$error = "There is an Error type: {$type}. says: $message. in file: $file. on line: $line.";
153
154			// $this->displayFriendlyMessage();
155
156			$this->sendErrorToAdmin($error);
157			}
158			}
159

refatalsakka / framework

Error A last analyzed 2020-11-24 15:34 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

9 Methods

Duplication Side-by-Side

Filter issues like

Error A
last analyzed 2020-11-24 15:34 UTC