LogServiceProvider::boot() - Code Metrics - Inspection of "abstract QueueServiceProvider, various fixes, LogS..." - fondbot/framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1e74ba...85d930 )

by Vladimir

created 2017-05-16 21:23 UTC

LogServiceProvider::boot() B

↳ Parent: LogServiceProvider

Complexity

Conditions	3
Paths	2

Size

Total Lines	24
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

Metric	Value
cc	3
eloc	13
nc	2
nop	0
dl	0
loc	24
ccs	0
cts	9
cp	0
crap	12
rs	8.9713
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace FondBot\Application;

use Throwable;
use Whoops\Run;
use Monolog\Logger;
use Psr\Log\LoggerInterface;
use Monolog\Handler\HandlerInterface;
use Whoops\Handler\PrettyPageHandler;
use League\Container\ServiceProvider\AbstractServiceProvider;
use League\Container\ServiceProvider\BootableServiceProviderInterface;

abstract class LogServiceProvider extends AbstractServiceProvider implements BootableServiceProviderInterface
{
    protected $provides = [
        LoggerInterface::class,
        'application_log',
    ];

    /**
     * Define handlers.
     *
     * @return HandlerInterface[]
     */
    abstract public function handlers(): array;

    /**
     * Method will be invoked on registration of a service provider implementing
     * this interface. Provides ability for eager loading of Service Providers.
     *
     * @return void
     * @throws \Psr\Container\NotFoundExceptionInterface
     *
     * @throws \Psr\Container\ContainerExceptionInterface
     * @throws \Exception
     * @throws \InvalidArgumentException
     */
    public function boot(): void
    {
        $this->container->share(LoggerInterface::class, function () {
            $logger = new Logger('FondBot');

            foreach ($this->handlers() as $handler) {
                $logger->pushHandler($handler);
            }

            return $logger;
        });

        if (PHP_SAPI !== 'cli') {
            $whoops = new Run;
            $whoops->pushHandler(new PrettyPageHandler);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $whoops->pushHandler(function (Throwable $exception) {
                /** @var LoggerInterface $logger */
                $logger = $this->container->get(LoggerInterface::class);

                $logger->error($exception);
            });
            $whoops->register();
        }
    }

    /**
     * Use the register method to register items with the container via the
     * protected $this->container property or the `getContainer` method
     * from the ContainerAwareTrait.
     *
     * @return void
     */
    public function register(): void
    {
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace FondBot\Application;
6
7			use Throwable;
8			use Whoops\Run;
9			use Monolog\Logger;
10			use Psr\Log\LoggerInterface;
11			use Monolog\Handler\HandlerInterface;
12			use Whoops\Handler\PrettyPageHandler;
13			use League\Container\ServiceProvider\AbstractServiceProvider;
14			use League\Container\ServiceProvider\BootableServiceProviderInterface;
15
16			abstract class LogServiceProvider extends AbstractServiceProvider implements BootableServiceProviderInterface
17			{
18			protected $provides = [
19			LoggerInterface::class,
20			'application_log',
21			];
22
23			/**
24			* Define handlers.
25			*
26			* @return HandlerInterface[]
27			*/
28			abstract public function handlers(): array;
29
30			/**
31			* Method will be invoked on registration of a service provider implementing
32			* this interface. Provides ability for eager loading of Service Providers.
33			*
34			* @return void
35			* @throws \Psr\Container\NotFoundExceptionInterface
36			*
37			* @throws \Psr\Container\ContainerExceptionInterface
38			* @throws \Exception
39			* @throws \InvalidArgumentException
40			*/
41			public function boot(): void
42			{
43			$this->container->share(LoggerInterface::class, function () {
44			$logger = new Logger('FondBot');
45
46			foreach ($this->handlers() as $handler) {
47			$logger->pushHandler($handler);
48			}
49
50			return $logger;
51			});
52
53			if (PHP_SAPI !== 'cli') {
54			$whoops = new Run;
55			$whoops->pushHandler(new PrettyPageHandler);
			0 ignored issues – show Documentation introduced 2017-05-10 21:42 UTC by Report Bug Copy Issue Report `new \Whoops\Handler\PrettyPageHandler()` is of type `object<Whoops\Handler\PrettyPageHandler>`, 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...
56			$whoops->pushHandler(function (Throwable $exception) {
57			/** @var LoggerInterface $logger */
58			$logger = $this->container->get(LoggerInterface::class);
59
60			$logger->error($exception);
61			});
62			$whoops->register();
63			}
64			}
65
66			/**
67			* Use the register method to register items with the container via the
68			* protected $this->container property or the `getContainer` method
69			* from the ContainerAwareTrait.
70			*
71			* @return void
72			*/
73			public function register(): void
74			{
75			}
76			}
77

fondbot / framework

Push — master ( 1e74ba...85d930 )

LogServiceProvider::boot() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like