ConfigServiceProvider::__construct() - Code Metrics - Inspection of "Start splitting the service providers into separat..." - joomla/statistics-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 51a51b...3888f6 )

by Michael

created 2016-10-04 13:11 UTC

ConfigServiceProvider::__construct() A

↳ Parent: ConfigServiceProvider

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
dl	0
loc	10
ccs	6
cts	6
cp	1
rs	9.4285
c	2
b	0
f	0
cc	2
eloc	4
nc	2
nop	1
crap	2

<?php

namespace Stats\Providers;

use Joomla\DI\Container;
use Joomla\DI\ServiceProviderInterface;
use Joomla\Registry\Registry;

/**
 * Configuration service provider
 *
 * @since  1.0
 */
class ConfigServiceProvider implements ServiceProviderInterface
{
	/**
	 * Configuration instance
	 *
	 * @var    Registry
	 * @since  1.0
	 */
	private $config;

	/**
	 * Constructor.
	 *
	 * @param   string  $file  Path to the config file.
	 *
	 * @since   1.0
	 * @throws  \RuntimeException
	 */
	public function __construct($file)
	{
		// Verify the configuration exists and is readable.
		if (!is_readable($file))
		{
			throw new \RuntimeException('Configuration file does not exist or is unreadable.');
		}

		$this->config = (new Registry)->loadFile($file);
	}

	/**
	 * Registers the service provider with a DI container.
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  void
	 *
	 * @since   1.0
	 */
	public function register(Container $container)
	{
		$container->share('config', [$this, 'getConfigService'], true);
	}

	/**
	 * Get the `config` service
	 *
	 * @param   Container  $container  The DI container.
	 *
	 * @return  Registry
	 *
	 * @since   1.0
	 */
	public function getConfigService(Container $container)

	{
		return $this->config;
	}
}


1		<?php
2
3		namespace Stats\Providers;
4
5		use Joomla\DI\Container;
6		use Joomla\DI\ServiceProviderInterface;
7		use Joomla\Registry\Registry;
8
9		/**
10		* Configuration service provider
11		*
12		* @since 1.0
13		*/
14		class ConfigServiceProvider implements ServiceProviderInterface
15		{
16		/**
17		* Configuration instance
18		*
19		* @var Registry
20		* @since 1.0
21		*/
22		private $config;
23
24		/**
25		* Constructor.
26		*
27		* @param string $file Path to the config file.
28		*
29		* @since 1.0
30		* @throws \RuntimeException
31		*/
32	2	public function __construct($file)
33		{
34		// Verify the configuration exists and is readable.
35	2	if (!is_readable($file))
36	2	{
37	1	throw new \RuntimeException('Configuration file does not exist or is unreadable.');
38		}
39
40	1	$this->config = (new Registry)->loadFile($file);
41	1	}
42
43		/**
44		* Registers the service provider with a DI container.
45		*
46		* @param Container $container The DI container.
47		*
48		* @return void
49		*
50		* @since 1.0
51		*/
52	1	public function register(Container $container)
53		{
54	1	$container->share('config', [$this, 'getConfigService'], true);
55	1	}
56
57		/**
58		* Get the `config` service
59		*
60		* @param Container $container The DI container.
61		*
62		* @return Registry
63		*
64		* @since 1.0
65		*/
66	1	public function getConfigService(Container $container)
		0 ignored issues – show Unused Code introduced 2016-10-04 13:18 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...
67		{
68	1	return $this->config;
69		}
70		}
71

joomla / statistics-server

Push — master ( 51a51b...3888f6 )

ConfigServiceProvider::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like