PackageManager - Code Metrics - Vinelab/url-shortener - Measure and Improve Code Quality continuously with Scrutinizer

PackageManager A
last analyzed 2015-12-23 10:31 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
wmc	7
c	2
b	0
f	0
lcom	0
cbo	1
dl	0
loc	82
rs	10

7 Methods

Rating	Name	Size	Complexity
A	__construct()	6	1
A	getDriverName()	4	1
A	getDriverParameters()	4	1
A	getHttpClient()	4	1
A	setDriverName()	4	1
A	setDriverParameters()	4	1
A	setHttpClient()	4	1

<?php

namespace Vinelab\UrlShortener\Base;

/**
 * Class PackageManager is the heart of the package,
 * It' main job is to initialize the config file.
 *
 * @category Manager
 *
 * @author   Mahmoud Zalt <[email protected]>
 */
class PackageManager
{
    /**
     * default driver name.
     *
     * @var string
     */
    protected $driverName;

    /**
     * configurations of the default driver.
     *
     * @var array
     */
    protected $driverParameters;

    /**
     * Http client.
     *
     * @var object
     */
    private $httpClient;

    /**
     * @param \Vinelab\UrlShortener\Base\ConfigManager $config
     * @param null                                     $httpClient The http Client 'will be injected here (mocked) for testing'
     */
    public function __construct(ConfigManager $config, $httpClient = null)
    {
        $this->setDriverName($config->driverName());
        $this->setDriverParameters($config->driverParameters($this->getDriverName()));
        $this->setHttpClient($httpClient);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

    /**
     * @return string
     */
    public function getDriverName()
    {
        return $this->driverName;
    }

    /**
     * @return array
     */
    public function getDriverParameters()
    {
        return $this->driverParameters;
    }

    /**
     * @return object
     */
    public function getHttpClient()
    {
        return $this->httpClient;
    }

    /**
     * @param string $driverName
     */
    private function setDriverName($driverName)
    {
        $this->driverName = $driverName;
    }

    /**
     * @param array $driverParameters
     */
    private function setDriverParameters($driverParameters)
    {
        $this->driverParameters = $driverParameters;
    }

    /**
     * @param object $httpClient
     */
    private function setHttpClient($httpClient)
    {
        $this->httpClient = $httpClient;
    }
}


1			<?php
2
3			namespace Vinelab\UrlShortener\Base;
4
5			/**
6			* Class PackageManager is the heart of the package,
7			* It' main job is to initialize the config file.
8			*
9			* @category Manager
10			*
11			* @author Mahmoud Zalt <[email protected]>
12			*/
13			class PackageManager
14			{
15			/**
16			* default driver name.
17			*
18			* @var string
19			*/
20			protected $driverName;
21
22			/**
23			* configurations of the default driver.
24			*
25			* @var array
26			*/
27			protected $driverParameters;
28
29			/**
30			* Http client.
31			*
32			* @var object
33			*/
34			private $httpClient;
35
36			/**
37			* @param \Vinelab\UrlShortener\Base\ConfigManager $config
38			* @param null $httpClient The http Client 'will be injected here (mocked) for testing'
39			*/
40			public function __construct(ConfigManager $config, $httpClient = null)
41			{
42			$this->setDriverName($config->driverName());
43			$this->setDriverParameters($config->driverParameters($this->getDriverName()));
44			$this->setHttpClient($httpClient);
			0 ignored issues – show Documentation introduced 2015-12-23 10:31 UTC by Report Bug Copy Issue Report `$httpClient` is of type `null`, but the function expects a `object`. 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...
45			}
46
47			/**
48			* @return string
49			*/
50			public function getDriverName()
51			{
52			return $this->driverName;
53			}
54
55			/**
56			* @return array
57			*/
58			public function getDriverParameters()
59			{
60			return $this->driverParameters;
61			}
62
63			/**
64			* @return object
65			*/
66			public function getHttpClient()
67			{
68			return $this->httpClient;
69			}
70
71			/**
72			* @param string $driverName
73			*/
74			private function setDriverName($driverName)
75			{
76			$this->driverName = $driverName;
77			}
78
79			/**
80			* @param array $driverParameters
81			*/
82			private function setDriverParameters($driverParameters)
83			{
84			$this->driverParameters = $driverParameters;
85			}
86
87			/**
88			* @param object $httpClient
89			*/
90			private function setHttpClient($httpClient)
91			{
92			$this->httpClient = $httpClient;
93			}
94			}
95

Vinelab / url-shortener

PackageManager A last analyzed 2015-12-23 10:31 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

7 Methods

Duplication Side-by-Side

Filter issues like

PackageManager A
last analyzed 2015-12-23 10:31 UTC