Skip to content

Commit 0be554c

Browse files
authored
Merge pull request #211 from cybercog/remove-unused-settings
Remove unused Settings properties
2 parents b6d8b8e + efc094e commit 0be554c

File tree

2 files changed

+2
-16
lines changed

2 files changed

+2
-16
lines changed

src/Settings.php

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -2,32 +2,19 @@
22

33
namespace ClickHouseDB;
44

5-
use ClickHouseDB\Transport\Http;
6-
75
class Settings
86
{
9-
/**
10-
* @var Http
11-
*/
12-
private $client = null;
13-
147
/**
158
* @var array
169
*/
1710
private $settings = [];
1811

1912
private $_ReadOnlyUser = false;
2013

21-
/**
22-
* @var bool
23-
*/
24-
private $_isHttps = false;
25-
2614
/**
2715
* Settings constructor.
28-
* @param Http $client
2916
*/
30-
public function __construct(Http $client)
17+
public function __construct()
3118
{
3219
$default = [
3320
'extremes' => false,
@@ -38,7 +25,6 @@ public function __construct(Http $client)
3825
];
3926

4027
$this->settings = $default;
41-
$this->client = $client;
4228
}
4329

4430
/**

src/Transport/Http.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ public function __construct($host, $port, $username, $password, $authMethod = nu
113113
$this->_authMethod = $authMethod;
114114
}
115115

116-
$this->_settings = new Settings($this);
116+
$this->_settings = new Settings();
117117

118118
$this->setCurler();
119119
}

0 commit comments

Comments
 (0)