Settings::isSet() - Code Metrics - Inspection of "Cleanup transport" - smi2/phpClickHouse - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#102)

by Šimon

created 2018-10-23 09:34 UTC

Settings::isSet() A

↳ Parent: Settings

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

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

<?php


namespace ClickHouseDB;

/**
 * @see https://clickhouse.yandex/docs/en/operations/settings/
 */
class Settings
{
    /** @var mixed[] */
    private $settings = [];

    /** @var bool */
    private $httpCompressionEnabled = false;

    /**
     * @return mixed|null
     */
    public function get(string $key)
    {
        if (! $this->isSet($key)) {
            return null;
        }

        return $this->settings[$key];
    }

    public function isSet(string $key) : bool
    {
        return isset($this->settings[$key]);
    }

    /**
     * @param mixed $value
     */
    public function set(string $key, $value) : void
    {
        $this->settings[$key] = $value;
    }

    /**
     * @return mixed|null
     */
    public function isHttpCompressionEnabled()
    {
        return $this->httpCompressionEnabled;
    }

    public function setHttpCompression(bool $enable) : self
    {
        $this->httpCompressionEnabled = $enable;

        return $this;
    }

    public function readonly(int $flag) : self
    {
        $this->set('readonly', $flag);

        return $this;
    }

    /**
     * @param string $session_id

     * @return $this
     */
    public function session_id($session_id)

    {
        $this->set('session_id', $session_id);

        return $this;
    }

    /**
     * @return mixed|bool
     */
    public function getSessionId()
    {
        if (empty($this->settings['session_id'])) {
            return false;
        }

        return $this->get('session_id');
    }

    /**
     * @return string|bool
     */
    public function makeSessionId()
    {
        $this->session_id(sha1(uniqid('', true)));


        return $this->getSessionId();
    }

    /**
     * @param mixed[] $forcedSettings
     *
     * @return mixed[]
     */
    public function getQueryableSettings(array $forcedSettings) : array
    {
        $settings = $this->settings;
        if (! empty($forcedSettings)) {
            $settings = $forcedSettings + $this->settings;
        }

        return $settings;
    }

    /**
     * @param array $settings_array

     * @return $this
     */
    public function apply($settings_array)

    {
        foreach ($settings_array as $key => $value) {
            $this->set($key, $value);
        }

        return $this;
    }

    /**
     * @param string $name

     * @return mixed|null
     */
    public function getSetting($name)

    {
        if (! isset($this->settings[$name])) {
            return null;
        }

        return $this->get($name);
    }
}


1		<?php
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Missing declare(strict_types = 1). Loading history...
2
3		namespace ClickHouseDB;
4
5		/**
6		* @see https://clickhouse.yandex/docs/en/operations/settings/
7		*/
8		class Settings
9		{
10		/** @var mixed[] */
11		private $settings = [];
12
13		/** @var bool */
14		private $httpCompressionEnabled = false;
15
16		/**
17		* @return mixed\|null
18		*/
19	3	public function get(string $key)
20		{
21	3	if (! $this->isSet($key)) {
22		return null;
23		}
24
25	3	return $this->settings[$key];
26		}
27
28	4	public function isSet(string $key) : bool
29		{
30	4	return isset($this->settings[$key]);
31		}
32
33		/**
34		* @param mixed $value
35		*/
36	7	public function set(string $key, $value) : void
37		{
38	7	$this->settings[$key] = $value;
39	7	}
40
41		/**
42		* @return mixed\|null
43		*/
44	63	public function isHttpCompressionEnabled()
45		{
46	63	return $this->httpCompressionEnabled;
47		}
48
49	63	public function setHttpCompression(bool $enable) : self
50		{
51	63	$this->httpCompressionEnabled = $enable;
52
53	63	return $this;
54		}
55
56		public function readonly(int $flag) : self
57		{
58		$this->set('readonly', $flag);
59
60		return $this;
61		}
62
63		/**
64		* @param string $session_id
		0 ignored issues – show introduced 2018-09-26 10:27 UTC by Report Bug Copy Issue Report Incorrect annotations group. Loading history...
65		* @return $this
66		*/
67	2	public function session_id($session_id)
		0 ignored issues – show Coding Style introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method name "Settings::session_id" is not in camel caps format Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Settings::session_id() does not have parameter type hint for its parameter $session_id but it should be possible to add it based on @param annotation "string". Loading history...
68		{
69	2	$this->set('session_id', $session_id);
70
71	2	return $this;
72		}
73
74		/**
75		* @return mixed\|bool
76		*/
77	63	public function getSessionId()
78		{
79	63	if (empty($this->settings['session_id'])) {
80	63	return false;
81		}
82
83	2	return $this->get('session_id');
84		}
85
86		/**
87		* @return string\|bool
88		*/
89	2	public function makeSessionId()
90		{
91	2	$this->session_id(sha1(uniqid('', true)));
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Function sha1() should not be referenced via a fallback global name, but via a use statement. Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Function uniqid() should not be referenced via a fallback global name, but via a use statement. Loading history...
92
93	2	return $this->getSessionId();
94		}
95
96		/**
97		* @param mixed[] $forcedSettings
98		*
99		* @return mixed[]
100		*/
101	63	public function getQueryableSettings(array $forcedSettings) : array
102		{
103	63	$settings = $this->settings;
104	63	if (! empty($forcedSettings)) {
105	11	$settings = $forcedSettings + $this->settings;
106		}
107
108	63	return $settings;
109		}
110
111		/**
112		* @param array $settings_array
		0 ignored issues – show introduced 2018-09-26 10:27 UTC by Report Bug Copy Issue Report Incorrect annotations group. Loading history... introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report @param annotation of method \ClickHouseDB\Settings::apply() does not specify type hint for items of its traversable parameter $settings_array. Loading history...
113		* @return $this
114		*/
115	1	public function apply($settings_array)
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Settings::apply() does not have parameter type hint for its parameter $settings_array but it should be possible to add it based on @param annotation "array". Loading history...
116		{
117	1	foreach ($settings_array as $key => $value) {
118	1	$this->set($key, $value);
119		}
120
121	1	return $this;
122		}
123
124		/**
125		* @param string $name
		0 ignored issues – show introduced 2018-09-26 10:27 UTC by Report Bug Copy Issue Report Incorrect annotations group. Loading history...
126		* @return mixed\|null
127		*/
128	1	public function getSetting($name)
		0 ignored issues – show introduced 2018-07-08 20:56 UTC by Report Bug Copy Issue Report Method \ClickHouseDB\Settings::getSetting() does not have parameter type hint for its parameter $name but it should be possible to add it based on @param annotation "string". Loading history...
129		{
130	1	if (! isset($this->settings[$name])) {
131		return null;
132		}
133
134	1	return $this->get($name);
135		}
136		}
137

smi2 / phpClickHouse

Pull Request — master (#102)

Settings::isSet() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like