Config::getScheme() - Code Metrics - Inspection of "Merge pull request #708 from Geolim4/master" - PHPSocialNetwork/phpfastcache - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2cb775...11c7b1 )

by Georges

created 2019-09-15 15:11 UTC

Config::getScheme() A

↳ Parent: Config

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	0
b	0
f	0

<?php
/**
 *
 * This file is part of phpFastCache.
 *
 * @license MIT License (MIT)
 *
 * For full copyright and license information, please see the docs/CREDITS.txt file.
 *
 * @author Khoa Bui (khoaofgod)  <[email protected]> https://www.phpfastcache.com
 * @author Georges.L (Geolim4)  <[email protected]>
 *
 */

declare(strict_types=1);

namespace Phpfastcache\Drivers\Predis;

use Phpfastcache\Config\ConfigurationOption;
use Phpfastcache\Exceptions\PhpfastcacheInvalidConfigurationException;

class Config extends ConfigurationOption
{
    /**
     * @var string
     */
    protected $host = '127.0.0.1';

    /**
     * @var int
     */
    protected $port = 6379;

    /**
     * @var string
     */
    protected $password = '';

    /**
     * @var int
     */
    protected $database = 0;

    /**
     * @var \Predis\Client
filter:
    dependency_paths: ["lib/*"]
     */
    protected $predisClient;

    /**
     * @var string
     */
    protected $optPrefix = '';

    /**
     * @var int
     */
    protected $timeout = 5;

    /**
     * @var bool
     */
    protected $persistent = false;

    /**
     * @var string
     */
    protected $scheme = 'unix';

    /**
     * @return string
     */
    public function getHost(): string
    {
        return $this->host;
    }

    /**
     * @param string $host
     * @return Config
     */
    public function setHost(string $host): self
    {
        $this->host = $host;
        return $this;
    }

    /**
     * @return int
     */
    public function getPort(): int
    {
        return $this->port;
    }

    /**
     * @param int $port
     * @return Config
     */
    public function setPort(int $port): self
    {
        $this->port = $port;
        return $this;
    }

    /**
     * @return null
     */
    public function getPassword()
    {
        return $this->password;

    }

    /**
     * @param null $password

     * @return self
     */
    public function setPassword(string $password): self
    {
        $this->password = $password;
        return $this;
    }

    /**
     * @return int
     */
    public function getDatabase(): int
    {
        return $this->database;
    }

    /**
     * @param int $database
     * @return Config
     */
    public function setDatabase(int $database): self
    {
        $this->database = $database;
        return $this;
    }

    /**
     * @return array
     */
    public function getPredisConfigArray(): array
    {
        return [
            'host' => $this->getHost(),
            'port' => $this->getPort(),
            'password' => $this->getPassword() ?: null,
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            'database' => $this->getDatabase(),
            'timeout' => $this->getTimeout(),
        ];
    }

    /**
     * @return \Predis\Client|null
     */
    public function getPredisClient()
    {
        return $this->predisClient;
    }

    /**
     * @param \Predis\Client $predisClient |null
     * @return Config
     */
    public function setPredisClient(\Predis\Client $predisClient = null): Config
    {
        $this->predisClient = $predisClient;
        return $this;
    }

    /**
     * @return string
     * @since 7.0.2
     */
    public function getOptPrefix(): string
    {
        return $this->optPrefix;
    }

    /**
     * @param string $optPrefix
     * @return Config
     * @since 7.0.2
     */
    public function setOptPrefix(string $optPrefix): Config
    {
        $this->optPrefix = trim($optPrefix);
        return $this;
    }

    /**
     * @return int
     */
    public function getTimeout(): int
    {
        return $this->timeout;
    }

    /**
     * @param int $timeout
     * @return self
     */
    public function setTimeout(int $timeout): self
    {
        $this->timeout = $timeout;
        return $this;
    }

    /**
     * @return bool
     */
    public function isPersistent(): bool
    {
        return $this->persistent;
    }

    /**
     * @param bool $persistent
     * @return Config
     */
    public function setPersistent(bool $persistent): Config
    {
        $this->persistent = $persistent;
        return $this;
    }

    /**
     * @return string
     */
    public function getScheme(): string
    {
        return $this->scheme;
    }

    /**
     * @param string $scheme
     * @return Config
     * @throws PhpfastcacheInvalidConfigurationException
     */
    public function setScheme(string $scheme): Config
    {
        if(!\in_array($scheme, ['unix', 'tls'], true)){
            throw new PhpfastcacheInvalidConfigurationException('Invalid scheme: ' . $scheme);
        }
        $this->scheme = $scheme;
        return $this;
    }
}

1			<?php
2			/**
3			*
4			* This file is part of phpFastCache.
5			*
6			* @license MIT License (MIT)
7			*
8			* For full copyright and license information, please see the docs/CREDITS.txt file.
9			*
10			* @author Khoa Bui (khoaofgod) <[email protected]> https://www.phpfastcache.com
11			* @author Georges.L (Geolim4) <[email protected]>
12			*
13			*/
14
15			declare(strict_types=1);
16
17			namespace Phpfastcache\Drivers\Predis;
18
19			use Phpfastcache\Config\ConfigurationOption;
20			use Phpfastcache\Exceptions\PhpfastcacheInvalidConfigurationException;
21
22			class Config extends ConfigurationOption
23			{
24			/**
25			* @var string
26			*/
27			protected $host = '127.0.0.1';
28
29			/**
30			* @var int
31			*/
32			protected $port = 6379;
33
34			/**
35			* @var string
36			*/
37			protected $password = '';
38
39			/**
40			* @var int
41			*/
42			protected $database = 0;
43
44			/**
45			* @var \Predis\Client
			0 ignored issues – show Bug introduced 2018-05-12 01:48 UTC by Report Bug Copy Issue Report The type `Predis\Client` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
46			*/
47			protected $predisClient;
48
49			/**
50			* @var string
51			*/
52			protected $optPrefix = '';
53
54			/**
55			* @var int
56			*/
57			protected $timeout = 5;
58
59			/**
60			* @var bool
61			*/
62			protected $persistent = false;
63
64			/**
65			* @var string
66			*/
67			protected $scheme = 'unix';
68
69			/**
70			* @return string
71			*/
72			public function getHost(): string
73			{
74			return $this->host;
75			}
76
77			/**
78			* @param string $host
79			* @return Config
80			*/
81			public function setHost(string $host): self
82			{
83			$this->host = $host;
84			return $this;
85			}
86
87			/**
88			* @return int
89			*/
90			public function getPort(): int
91			{
92			return $this->port;
93			}
94
95			/**
96			* @param int $port
97			* @return Config
98			*/
99			public function setPort(int $port): self
100			{
101			$this->port = $port;
102			return $this;
103			}
104
105			/**
106			* @return null
107			*/
108			public function getPassword()
109			{
110			return $this->password;
			0 ignored issues – show Bug Best Practice introduced 2018-03-05 21:49 UTC by Report Bug Copy Issue Report The expression `return $this->password` returns the type `string` which is incompatible with the documented return type `null`. Loading history...
111			}
112
113			/**
114			* @param null $password
			0 ignored issues – show Documentation Bug introduced 2018-03-05 21:49 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$password` is correct as it would always require `null` to be passed? Loading history...
115			* @return self
116			*/
117			public function setPassword(string $password): self
118			{
119			$this->password = $password;
120			return $this;
121			}
122
123			/**
124			* @return int
125			*/
126			public function getDatabase(): int
127			{
128			return $this->database;
129			}
130
131			/**
132			* @param int $database
133			* @return Config
134			*/
135			public function setDatabase(int $database): self
136			{
137			$this->database = $database;
138			return $this;
139			}
140
141			/**
142			* @return array
143			*/
144			public function getPredisConfigArray(): array
145			{
146			return [
147			'host' => $this->getHost(),
148			'port' => $this->getPort(),
149			'password' => $this->getPassword() ?: null,
			0 ignored issues – show Bug introduced 2018-03-05 21:49 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->getPassword()` targeting `Phpfastcache\Drivers\Predis\Config::getPassword()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
150			'database' => $this->getDatabase(),
151			'timeout' => $this->getTimeout(),
152			];
153			}
154
155			/**
156			* @return \Predis\Client\|null
157			*/
158			public function getPredisClient()
159			{
160			return $this->predisClient;
161			}
162
163			/**
164			* @param \Predis\Client $predisClient \|null
165			* @return Config
166			*/
167			public function setPredisClient(\Predis\Client $predisClient = null): Config
168			{
169			$this->predisClient = $predisClient;
170			return $this;
171			}
172
173			/**
174			* @return string
175			* @since 7.0.2
176			*/
177			public function getOptPrefix(): string
178			{
179			return $this->optPrefix;
180			}
181
182			/**
183			* @param string $optPrefix
184			* @return Config
185			* @since 7.0.2
186			*/
187			public function setOptPrefix(string $optPrefix): Config
188			{
189			$this->optPrefix = trim($optPrefix);
190			return $this;
191			}
192
193			/**
194			* @return int
195			*/
196			public function getTimeout(): int
197			{
198			return $this->timeout;
199			}
200
201			/**
202			* @param int $timeout
203			* @return self
204			*/
205			public function setTimeout(int $timeout): self
206			{
207			$this->timeout = $timeout;
208			return $this;
209			}
210
211			/**
212			* @return bool
213			*/
214			public function isPersistent(): bool
215			{
216			return $this->persistent;
217			}
218
219			/**
220			* @param bool $persistent
221			* @return Config
222			*/
223			public function setPersistent(bool $persistent): Config
224			{
225			$this->persistent = $persistent;
226			return $this;
227			}
228
229			/**
230			* @return string
231			*/
232			public function getScheme(): string
233			{
234			return $this->scheme;
235			}
236
237			/**
238			* @param string $scheme
239			* @return Config
240			* @throws PhpfastcacheInvalidConfigurationException
241			*/
242			public function setScheme(string $scheme): Config
243			{
244			if(!\in_array($scheme, ['unix', 'tls'], true)){
245			throw new PhpfastcacheInvalidConfigurationException('Invalid scheme: ' . $scheme);
246			}
247			$this->scheme = $scheme;
248			return $this;
249			}
250			}

PHPSocialNetwork / phpfastcache

Push — master ( 2cb775...11c7b1 )

Config::getScheme() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like