ErrorServiceProvider - Code Metrics - Inspection of "1.0.x dev" - starweb/starlit-app - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#14)

by Felix

created 2019-08-26 11:58 UTC

ErrorServiceProvider A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	10

Test Coverage

Coverage

71.79%

Importance

Changes

Metric	Value
wmc	6
lcom	0
cbo	10
dl	0
loc	69
ccs	28
cts	39
cp	0.7179
rs	10
c	0
b	0
f	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	register()	0	63	6

<?php declare(strict_types=1);
/**
 * Starlit App.
 *
 * @copyright Copyright (c) 2016 Starweb AB
 * @license   BSD 3-Clause
 */

namespace Starlit\App\Provider;

use Monolog\Handler\ErrorLogHandler;
use Monolog\Logger;
use Monolog\Processor\WebProcessor;
use Starlit\App\BaseApp;
use Starlit\App\ErrorHandling\UserErrorPageHandler;
use Whoops\Handler\PlainTextHandler;
use Whoops\Handler\PrettyPageHandler;

class ErrorServiceProvider implements ServiceProviderInterface
{
    /**
     * @param BaseApp $app
     */
    public function register(BaseApp $app): void
    {
        $app->alias('errorLogger', Logger::class);
        $app->set(
            Logger::class, function (BaseApp $app) {
            $logger = new Logger('errorLogger');

            $handler = new ErrorLogHandler();
            if (!$app->isCli()) {
                $handler->pushProcessor(new WebProcessor());
                $format = '%level_name%: %message% %extra.server%%extra.url%';
            } else {
                $format = '%level_name%: %message%';
            }
            $handler->setFormatter(new \Monolog\Formatter\LineFormatter($format, null, true));
            $logger->pushHandler($handler);

            return $logger;
        });

        $app->set('whoopsDebugErrorPageHandler', function (BaseApp $app) {
            $prettyPageHandler = new PrettyPageHandler();
            if ($app->getConfig()->has('editor')) {
                $prettyPageHandler->setEditor($app->getConfig()->get('editor'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            }

            return $prettyPageHandler;
        });

        $app->alias('whoopsUserErrorPageHandler', UserErrorPageHandler::class);
        $app->set(
            UserErrorPageHandler::class, function (BaseApp $app) {
            return new UserErrorPageHandler(
                $app->getConfig()->getRequired('errorPagePath')
            );
        });

        $app->set('whoopsErrorHandler', function (BaseApp $app) {
            $plainTextHandler = new PlainTextHandler();
            $plainTextHandler->setLogger($app->get(Logger::class));
            if (!$app->isCli()) {
                $plainTextHandler->loggerOnly(true);
            }

            return $plainTextHandler;
        });

        $app->set('whoops', function (BaseApp $app) {
            $whoops = new \Whoops\Run();

            if (ini_get('display_errors')) {
                $whoops->pushHandler($app->get('whoopsDebugErrorPageHandler'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            } elseif ($app->getConfig()->has('errorPagePath')) {
                $whoops->pushHandler($app->get('whoopsUserErrorPageHandler'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            }

            // Handles cli output and logging
            $whoops->pushHandler($app->get('whoopsErrorHandler'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

            return $whoops;
        });
        $app->get('whoops')->register();
    }
}


1		<?php declare(strict_types=1);
2		/**
3		* Starlit App.
4		*
5		* @copyright Copyright (c) 2016 Starweb AB
6		* @license BSD 3-Clause
7		*/
8
9		namespace Starlit\App\Provider;
10
11		use Monolog\Handler\ErrorLogHandler;
12		use Monolog\Logger;
13		use Monolog\Processor\WebProcessor;
14		use Starlit\App\BaseApp;
15		use Starlit\App\ErrorHandling\UserErrorPageHandler;
16		use Whoops\Handler\PlainTextHandler;
17		use Whoops\Handler\PrettyPageHandler;
18
19		class ErrorServiceProvider implements ServiceProviderInterface
20		{
21		/**
22		* @param BaseApp $app
23		*/
24	23	public function register(BaseApp $app): void
25		{
26	23	$app->alias('errorLogger', Logger::class);
27	23	$app->set(
28		Logger::class, function (BaseApp $app) {
29	23	$logger = new Logger('errorLogger');
30
31	23	$handler = new ErrorLogHandler();
32	23	if (!$app->isCli()) {
33		$handler->pushProcessor(new WebProcessor());
34		$format = '%level_name%: %message% %extra.server%%extra.url%';
35		} else {
36	23	$format = '%level_name%: %message%';
37		}
38	23	$handler->setFormatter(new \Monolog\Formatter\LineFormatter($format, null, true));
39	23	$logger->pushHandler($handler);
40
41	23	return $logger;
42	23	});
43
44		$app->set('whoopsDebugErrorPageHandler', function (BaseApp $app) {
45		$prettyPageHandler = new PrettyPageHandler();
46		if ($app->getConfig()->has('editor')) {
47		$prettyPageHandler->setEditor($app->getConfig()->get('editor'));
		0 ignored issues – show Documentation introduced 2016-04-04 09:15 UTC by Report Bug Copy Issue Report `$app->getConfig()->get('editor')` is of type `*`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
48		}
49
50		return $prettyPageHandler;
51	23	});
52
53	23	$app->alias('whoopsUserErrorPageHandler', UserErrorPageHandler::class);
54	23	$app->set(
55		UserErrorPageHandler::class, function (BaseApp $app) {
56		return new UserErrorPageHandler(
57		$app->getConfig()->getRequired('errorPagePath')
58		);
59	23	});
60
61		$app->set('whoopsErrorHandler', function (BaseApp $app) {
62	23	$plainTextHandler = new PlainTextHandler();
63	23	$plainTextHandler->setLogger($app->get(Logger::class));
64	23	if (!$app->isCli()) {
65		$plainTextHandler->loggerOnly(true);
66		}
67
68	23	return $plainTextHandler;
69	23	});
70
71		$app->set('whoops', function (BaseApp $app) {
72	23	$whoops = new \Whoops\Run();
73
74	23	if (ini_get('display_errors')) {
75		$whoops->pushHandler($app->get('whoopsDebugErrorPageHandler'));
		0 ignored issues – show Documentation introduced 2016-04-04 09:15 UTC by Report Bug Copy Issue Report `$app->get('whoopsDebugErrorPageHandler')` is of type `*`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
76	23	} elseif ($app->getConfig()->has('errorPagePath')) {
77		$whoops->pushHandler($app->get('whoopsUserErrorPageHandler'));
		0 ignored issues – show Documentation introduced 2017-01-22 16:06 UTC by Report Bug Copy Issue Report `$app->get('whoopsUserErrorPageHandler')` is of type `*`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
78		}
79
80		// Handles cli output and logging
81	23	$whoops->pushHandler($app->get('whoopsErrorHandler'));
		0 ignored issues – show Documentation introduced 2016-04-04 09:15 UTC by Report Bug Copy Issue Report `$app->get('whoopsErrorHandler')` is of type `*`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
82
83	23	return $whoops;
84	23	});
85	23	$app->get('whoops')->register();
86	23	}
87		}
88

starweb / starlit-app

GitHub Access Token became invalid

Pull Request — master (#14)

ErrorServiceProvider A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

1 Method

Duplication Side-by-Side

Filter issues like