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

Completed

Push — master ( 0f5779...ff4715 )

by Georges

created 2018-09-05 20:23 UTC

Config::setTimeout() A

↳ Parent: Config

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	1
dl	0
loc	4
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;

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;

    /**
     * @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;
    }
}

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
21			class Config extends ConfigurationOption
22			{
23			/**
24			* @var string
25			*/
26			protected $host = '127.0.0.1';
27
28			/**
29			* @var int
30			*/
31			protected $port = 6379;
32
33			/**
34			* @var string
35			*/
36			protected $password = '';
37
38			/**
39			* @var int
40			*/
41			protected $database = 0;
42
43			/**
44			* @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...
45			*/
46			protected $predisClient;
47
48			/**
49			* @var string
50			*/
51			protected $optPrefix = '';
52
53			/**
54			* @var int
55			*/
56			protected $timeout = 5;
57
58			/**
59			* @return string
60			*/
61			public function getHost(): string
62			{
63			return $this->host;
64			}
65
66			/**
67			* @param string $host
68			* @return Config
69			*/
70			public function setHost(string $host): self
71			{
72			$this->host = $host;
73			return $this;
74			}
75
76			/**
77			* @return int
78			*/
79			public function getPort(): int
80			{
81			return $this->port;
82			}
83
84			/**
85			* @param int $port
86			* @return Config
87			*/
88			public function setPort(int $port): self
89			{
90			$this->port = $port;
91			return $this;
92			}
93
94			/**
95			* @return null
96			*/
97			public function getPassword()
98			{
99			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...
100			}
101
102			/**
103			* @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...
104			* @return self
105			*/
106			public function setPassword(string $password): self
107			{
108			$this->password = $password;
109			return $this;
110			}
111
112			/**
113			* @return int
114			*/
115			public function getDatabase(): int
116			{
117			return $this->database;
118			}
119
120			/**
121			* @param int $database
122			* @return Config
123			*/
124			public function setDatabase(int $database): self
125			{
126			$this->database = $database;
127			return $this;
128			}
129
130			/**
131			* @return array
132			*/
133			public function getPredisConfigArray(): array
134			{
135			return [
136			'host' => $this->getHost(),
137			'port' => $this->getPort(),
138			'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...
139			'database' => $this->getDatabase(),
140			'timeout' => $this->getTimeout(),
141			];
142			}
143
144			/**
145			* @return \Predis\Client\|null
146			*/
147			public function getPredisClient()
148			{
149			return $this->predisClient;
150			}
151
152			/**
153			* @param \Predis\Client $predisClient \|null
154			* @return Config
155			*/
156			public function setPredisClient(\Predis\Client $predisClient = null): Config
157			{
158			$this->predisClient = $predisClient;
159			return $this;
160			}
161
162			/**
163			* @return string
164			* @since 7.0.2
165			*/
166			public function getOptPrefix(): string
167			{
168			return $this->optPrefix;
169			}
170
171			/**
172			* @param string $optPrefix
173			* @return Config
174			* @since 7.0.2
175			*/
176			public function setOptPrefix(string $optPrefix): Config
177			{
178			$this->optPrefix = trim($optPrefix);
179			return $this;
180			}
181
182			/**
183			* @return int
184			*/
185			public function getTimeout(): int
186			{
187			return $this->timeout;
188			}
189
190			/**
191			* @param int $timeout
192			* @return self
193			*/
194			public function setTimeout(int $timeout): self
195			{
196			$this->timeout = $timeout;
197			return $this;
198			}
199			}

PHPSocialNetwork / phpfastcache

Push — master ( 0f5779...ff4715 )

Config::setTimeout() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like