MonologServiceProvider::getMonologLoggerCliService() - Code Metrics - Inspection of "Merge pull request #42 from joomla/upgrade/framewo..." - joomla/statistics-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9a3a3c...24148b )

by Michael

created 2019-03-24 20:19 UTC

getMonologLoggerCliService() A

↳ Parent: MonologServiceProvider

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	4
cts	4
cp	1
rs	9.9666
c	0
b	0
f	0
cc	1
nc	1
nop	1
crap	1

<?php
/**
 * Joomla! Statistics Server
 *
 * @copyright  Copyright (C) 2013 - 2017 Open Source Matters, Inc. All rights reserved.
 * @license    http://www.gnu.org/licenses/gpl-2.0.txt GNU General Public License Version 2 or Later
 */

namespace Joomla\StatsServer\Providers;

use Joomla\DI\Container;
use Joomla\DI\ServiceProviderInterface;
use Monolog\Handler\StreamHandler;
use Monolog\Logger;
use Monolog\Processor\PsrLogMessageProcessor;
use Monolog\Processor\WebProcessor;

/**
 * Monolog service provider
 */
class MonologServiceProvider implements ServiceProviderInterface
{
	/**
	 * Registers the service provider with a DI container.
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  void
	 */
	public function register(Container $container): void
	{
		// Register the PSR-3 processor
		$container->share('monolog.processor.psr3', [$this, 'getMonologProcessorPsr3Service']);

		// Register the web processor
		$container->share('monolog.processor.web', [$this, 'getMonologProcessorWebService']);

		// Register the web application handler
		$container->share('monolog.handler.application', [$this, 'getMonologHandlerApplicationService']);

		// Register the database handler
		$container->share('monolog.handler.database', [$this, 'getMonologHandlerDatabaseService']);

		// Register the web application Logger
		$container->share('monolog.logger.application', [$this, 'getMonologLoggerApplicationService']);

		// Register the CLI application Logger
		$container->share('monolog.logger.cli', [$this, 'getMonologLoggerCliService']);

		// Register the database Logger
		$container->share('monolog.logger.database', [$this, 'getMonologLoggerDatabaseService']);
	}

	/**
	 * Get the `monolog.processor.psr3` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  PsrLogMessageProcessor
	 */
	public function getMonologProcessorPsr3Service(Container $container): PsrLogMessageProcessor

	{
		return new PsrLogMessageProcessor;
	}

	/**
	 * Get the `monolog.processor.web` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  WebProcessor
	 */
	public function getMonologProcessorWebService(Container $container): WebProcessor

	{
		return new WebProcessor;
	}

	/**
	 * Get the `monolog.handler.application` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  StreamHandler
	 */
	public function getMonologHandlerApplicationService(Container $container): StreamHandler

	{
		/** @var \Joomla\Registry\Registry $config */
		$config = $container->get('config');

		$level = strtoupper($config->get('log.application', $config->get('log.level', 'error')));

		return new StreamHandler(APPROOT . '/logs/stats.log', \constant('\\Monolog\\Logger::' . $level));
	}

	/**
	 * Get the `monolog.handler.database` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  StreamHandler
	 */
	public function getMonologHandlerDatabaseService(Container $container): StreamHandler

	{
		/** @var \Joomla\Registry\Registry $config */
		$config = $container->get('config');

		$level = strtoupper($config->get('log.database', $config->get('log.level', 'error')));

		return new StreamHandler(APPROOT . '/logs/stats.log', \constant('\\Monolog\\Logger::' . $level));
	}

	/**
	 * Get the `monolog.logger.application` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  Logger
	 */
	public function getMonologLoggerApplicationService(Container $container): Logger
	{
		return new Logger(
			'Application',
			[
				$container->get('monolog.handler.application'),
			],
			[
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$container->get('monolog.processor.web'),
			]
		);
	}

	/**
	 * Get the `monolog.logger.cli` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  Logger
	 */
	public function getMonologLoggerCliService(Container $container): Logger
	{
		return new Logger(
			'Application',
			[
				$container->get('monolog.handler.application'),
			]
		);
	}

	/**
	 * Get the `monolog.logger.database` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  Logger
	 */
	public function getMonologLoggerDatabaseService(Container $container): Logger
	{
		return new Logger(
			'Application',
			[
				$container->get('monolog.handler.database'),
			],
			[
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
				$container->get('monolog.processor.psr3'),
				$container->get('monolog.processor.web'),
			]
		);
	}
}


1			<?php
2			/**
3			* Joomla! Statistics Server
4			*
5			* @copyright Copyright (C) 2013 - 2017 Open Source Matters, Inc. All rights reserved.
6			* @license http://www.gnu.org/licenses/gpl-2.0.txt GNU General Public License Version 2 or Later
7			*/
8
9			namespace Joomla\StatsServer\Providers;
10
11			use Joomla\DI\Container;
12			use Joomla\DI\ServiceProviderInterface;
13			use Monolog\Handler\StreamHandler;
14			use Monolog\Logger;
15			use Monolog\Processor\PsrLogMessageProcessor;
16			use Monolog\Processor\WebProcessor;
17
18			/**
19			* Monolog service provider
20			*/
21			class MonologServiceProvider implements ServiceProviderInterface
22			{
23			/**
24			* Registers the service provider with a DI container.
25			*
26			* @param Container $container The DI container.
27			*
28			* @return void
29			*/
30	11		public function register(Container $container): void
31			{
32			// Register the PSR-3 processor
33	11		$container->share('monolog.processor.psr3', [$this, 'getMonologProcessorPsr3Service']);
34
35			// Register the web processor
36	11		$container->share('monolog.processor.web', [$this, 'getMonologProcessorWebService']);
37
38			// Register the web application handler
39	11		$container->share('monolog.handler.application', [$this, 'getMonologHandlerApplicationService']);
40
41			// Register the database handler
42	11		$container->share('monolog.handler.database', [$this, 'getMonologHandlerDatabaseService']);
43
44			// Register the web application Logger
45	11		$container->share('monolog.logger.application', [$this, 'getMonologLoggerApplicationService']);
46
47			// Register the CLI application Logger
48	11		$container->share('monolog.logger.cli', [$this, 'getMonologLoggerCliService']);
49
50			// Register the database Logger
51	11		$container->share('monolog.logger.database', [$this, 'getMonologLoggerDatabaseService']);
52	11		}
53
54			/**
55			* Get the `monolog.processor.psr3` service
56			*
57			* @param Container $container The DI container.
58			*
59			* @return PsrLogMessageProcessor
60			*/
61			public function getMonologProcessorPsr3Service(Container $container): PsrLogMessageProcessor
			0 ignored issues – show Unused Code introduced 2017-08-10 23:44 UTC by Report Bug Copy Issue Report The parameter `$container` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
62			{
63			return new PsrLogMessageProcessor;
64			}
65
66			/**
67			* Get the `monolog.processor.web` service
68			*
69			* @param Container $container The DI container.
70			*
71			* @return WebProcessor
72			*/
73	6		public function getMonologProcessorWebService(Container $container): WebProcessor
			0 ignored issues – show Unused Code introduced 2017-08-10 23:44 UTC by Report Bug Copy Issue Report The parameter `$container` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
74			{
75	6		return new WebProcessor;
76			}
77
78			/**
79			* Get the `monolog.handler.application` service
80			*
81			* @param Container $container The DI container.
82			*
83			* @return StreamHandler
84			*/
85	7	View Code Duplication	public function getMonologHandlerApplicationService(Container $container): StreamHandler
			0 ignored issues – show Duplication introduced 2019-03-18 00:51 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
86			{
87			/** @var \Joomla\Registry\Registry $config */
88	7		$config = $container->get('config');
89
90	7		$level = strtoupper($config->get('log.application', $config->get('log.level', 'error')));
91
92	7		return new StreamHandler(APPROOT . '/logs/stats.log', \constant('\\Monolog\\Logger::' . $level));
93			}
94
95			/**
96			* Get the `monolog.handler.database` service
97			*
98			* @param Container $container The DI container.
99			*
100			* @return StreamHandler
101			*/
102		View Code Duplication	public function getMonologHandlerDatabaseService(Container $container): StreamHandler
			0 ignored issues – show Duplication introduced 2019-03-18 00:51 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
103			{
104			/** @var \Joomla\Registry\Registry $config */
105			$config = $container->get('config');
106
107			$level = strtoupper($config->get('log.database', $config->get('log.level', 'error')));
108
109			return new StreamHandler(APPROOT . '/logs/stats.log', \constant('\\Monolog\\Logger::' . $level));
110			}
111
112			/**
113			* Get the `monolog.logger.application` service
114			*
115			* @param Container $container The DI container.
116			*
117			* @return Logger
118			*/
119	6		public function getMonologLoggerApplicationService(Container $container): Logger
120			{
121	6		return new Logger(
122	6		'Application',
123			[
124	6		$container->get('monolog.handler.application'),
125			],
126			[
			0 ignored issues – show Documentation introduced 2017-02-28 15:23 UTC by Report Bug Copy Issue Report `array($container->get('monolog.processor.web'))` is of type `array<integer,,{"0":""}>`, but the function expects a `array<integer,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...
127	6		$container->get('monolog.processor.web'),
128			]
129			);
130			}
131
132			/**
133			* Get the `monolog.logger.cli` service
134			*
135			* @param Container $container The DI container.
136			*
137			* @return Logger
138			*/
139	1		public function getMonologLoggerCliService(Container $container): Logger
140			{
141	1		return new Logger(
142	1		'Application',
143			[
144	1		$container->get('monolog.handler.application'),
145			]
146			);
147			}
148
149			/**
150			* Get the `monolog.logger.database` service
151			*
152			* @param Container $container The DI container.
153			*
154			* @return Logger
155			*/
156			public function getMonologLoggerDatabaseService(Container $container): Logger
157			{
158			return new Logger(
159			'Application',
160			[
161			$container->get('monolog.handler.database'),
162			],
163			[
			0 ignored issues – show Documentation introduced 2016-03-15 00:30 UTC by Report Bug Copy Issue Report `array($container->get('m...onolog.processor.web'))` is of type `array<integer,,{"0":"","1":"*"}>`, but the function expects a `array<integer,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...
164			$container->get('monolog.processor.psr3'),
165			$container->get('monolog.processor.web'),
166			]
167			);
168			}
169			}
170

joomla / statistics-server

Push — master ( 9a3a3c...24148b )

getMonologLoggerCliService() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like