ClickHouseDB\Settings

Complexity

Total Complexity

25

Size/Duplication

Total Lines	225
Duplicated Lines	0 %

Test Coverage

Coverage

83.87%

Importance

Changes	5
Bugs	0	Features	0

21 Methods

Rating	Name	Duplication	Size	Complexity
A	isEnableHttpCompression()	0	3	1
A	enableHttpCompression()	0	4	1
A	is()	0	3	1
A	session_id()	0	4	1
A	getTimeOut()	0	3	1
A	isHttps()	0	3	1
A	https()	0	4	1
A	set()	0	4	1
A	get()	0	6	2
A	database()	0	4	1
A	readonly()	0	4	1
A	__construct()	0	12	1
A	getSetting()	0	7	2
A	max_execution_time()	0	4	1
A	apply()	0	7	2
A	isReadOnlyUser()	0	3	1
A	getSettings()	0	3	1
A	makeSessionId()	0	4	1
A	setReadOnlyUser()	0	3	1
A	getSessionId()	0	6	2
A	getDatabase()	0	3	1

<?php

Missing declare(strict_types=1).

namespace ClickHouseDB;

use ClickHouseDB\Transport\Http;

class Settings
{
    /**

Found multi-line comment for property \ClickHouseDB\Settings::$client with single line content, use one-line comment instead.

     * @var Http
     */
    private $client = null;

Class Settings contains write-only property $client.

    /**

Found multi-line comment for property \ClickHouseDB\Settings::$settings with single line content, use one-line comment instead.

     * @var array

@var annotation of property \ClickHouseDB\Settings::$settings does not specify type hint for its items.

     */
    private $settings = [];

    private $_ReadOnlyUser = false;

Property \ClickHouseDB\Settings::$_ReadOnlyUser does not have @var annotation.

    /**

Found multi-line comment for property \ClickHouseDB\Settings::$_isHttps with single line content, use one-line comment instead.

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

The private property $_isHttps is not used, and could be removed.

Class Settings contains unused property $_isHttps.

    /**
     * Settings constructor.

Documentation comment contains forbidden comment "Settings constructor.".

     * @param Http $client

Expected 1 lines between description and annotations, found 0.

Method \ClickHouseDB\Settings::__construct() has useless @param annotation for parameter $client.

     */
    public function __construct(Http $client)
    {
        $default = [
            'extremes' => false,
            'readonly' => true,
            'max_execution_time' => 20,
            'enable_http_compression' => 0,
            'https' => false,
        ];

        $this->settings = $default;
        $this->client = $client;

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

    }

    /**
     * @param string|int $key

Incorrect annotations group.

     * @return mixed
     */
    public function get($key)
    {
        if (!$this->is($key)) {

Expected 1 space(s) after NOT operator; 0 found

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

    /**
     * @param string|int $key

Incorrect annotations group.

     * @return bool
     */
    public function is($key)

Method \ClickHouseDB\Settings::is() does not have return type hint for its return value but it should be possible to add it based on @return annotation "bool".

    {
        return isset($this->settings[$key]);
    }


    /**
     * @param string|int $key

Incorrect annotations group.

     * @param mixed $value

Expected 6 spaces after parameter type; 1 found

     * @return $this
     */
    public function set($key, $value)
    {
        $this->settings[$key] = $value;
        return $this;
    }

    /**
     * @return mixed
     */
    public function getDatabase()
    {
        return $this->get('database');
    }

    /**
     * @param string $db

Incorrect annotations group.

     * @return $this
     */
    public function database($db)

Method \ClickHouseDB\Settings::database() does not have parameter type hint for its parameter $db but it should be possible to add it based on @param annotation "string".

    {
        $this->set('database', $db);
        return $this;
    }

    /**
     * @return mixed
     */
    public function getTimeOut()
    {
        return $this->get('max_execution_time');
    }

    /**
     * @return mixed|null
     */
    public function isEnableHttpCompression()
    {
        return $this->getSetting('enable_http_compression');
    }

    /**
     * @param bool|int $flag

Incorrect annotations group.

     * @return $this
     */
    public function enableHttpCompression($flag)
    {
        $this->set('enable_http_compression', intval($flag));

Function intval() should not be referenced via a fallback global name, but via a use statement.

        return $this;
    }


    public function https($flag = true)

Method \ClickHouseDB\Settings::https() does not have parameter type hint nor @param annotation for its parameter $flag.

Method \ClickHouseDB\Settings::https() does not have return type hint nor @return annotation for its return value.

    {
        $this->set('https', $flag);
        return $this;
    }

    public function isHttps()

Method \ClickHouseDB\Settings::isHttps() does not have return type hint nor @return annotation for its return value.

    {
        return $this->get('https');
    }


    /**
     * @param int|bool $flag

Incorrect annotations group.

     * @return $this
     */
    public function readonly($flag)
    {
        $this->set('readonly', $flag);
        return $this;
    }

    /**
     * @param string $session_id

Incorrect annotations group.

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

Method name "Settings::session_id" is not in camel caps format

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".

    {
        $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)));

Function sha1() should not be referenced via a fallback global name, but via a use statement.

Function uniqid() should not be referenced via a fallback global name, but via a use statement.

        return $this->getSessionId();
    }

    /**
     * @param int|float $time

Incorrect annotations group.

     * @return $this
     */
    public function max_execution_time($time)

Method name "Settings::max_execution_time" is not in camel caps format

    {
        $this->set('max_execution_time', $time);
        return $this;
    }

    /**
     * @return array

@return annotation of method \ClickHouseDB\Settings::getSettings() does not specify type hint for items of its traversable return value.

     */
    public function getSettings()

Method \ClickHouseDB\Settings::getSettings() does not have return type hint for its return value but it should be possible to add it based on @return annotation "array".

    {
        return $this->settings;
    }

    /**
     * @param array $settings_array

Incorrect annotations group.

@param annotation of method \ClickHouseDB\Settings::apply() does not specify type hint for items of its traversable parameter $settings_array.

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

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".

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

        return $this;
    }

    /**
     * @param int|bool $flag
     */
    public function setReadOnlyUser($flag)

Method \ClickHouseDB\Settings::setReadOnlyUser() does not have void return type hint.

    {
        $this->_ReadOnlyUser = $flag;
    }

    /**
     * @return bool
     */
    public function isReadOnlyUser()

Method \ClickHouseDB\Settings::isReadOnlyUser() does not have return type hint for its return value but it should be possible to add it based on @return annotation "bool".

    {
        return $this->_ReadOnlyUser;
    }

    /**
     * @param string $name

Incorrect annotations group.

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

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".

    {
        if (!isset($this->settings[$name])) {

Expected 1 space(s) after NOT operator; 0 found

            return null;
        }

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