CurlerRequest::parameters_json() - Code Metrics - smi2/phpClickHouse - Measure and Improve Code Quality continuously with Scrutinizer

CurlerRequest::parameters_json() A
last analyzed 2024-07-17 07:54 UTC

↳ Parent: CurlerRequest

Complexity

Conditions	5
Paths	4

Size

Total Lines	23
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes

Metric	Value
cc	5
eloc	12
c	0
b	0
f	0
nc	4
nop	1
dl	0
loc	23
ccs	0
cts	7
cp	0
crap	30
rs	9.5555

<?php

introduced 2019-03-13 07:05 UTC by

Missing declare(strict_types=1).

namespace ClickHouseDB\Transport;

use const CURLOPT_HTTPGET;

use const CURLOPT_POST;

class CurlerRequest

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$extendinfo with single line content, use one-line comment instead.

     * @var array

introduced 2020-12-21 16:16 UTC by

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$extendinfo does not specify type hint for its items.

*/

    public $extendinfo = [];

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$parameters with single line content, use one-line comment instead.

     * @var string|array

introduced 2020-12-21 16:16 UTC by

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$parameters does not specify type hint for its items.

*/

    private $parameters = '';

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$options with single line content, use one-line comment instead.

     * @var array

introduced 2020-12-21 16:16 UTC by

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$options does not specify type hint for its items.

*/

    private $options;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$headers with single line content, use one-line comment instead.

     * @var array

introduced 2020-12-21 16:16 UTC by

@var annotation of property \ClickHouseDB\Transport\CurlerRequest::$headers does not specify type hint for its items.

*/

    private $headers; // Parsed reponse header object.

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$url with single line content, use one-line comment instead.

     * @var string

*/

    private $url;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$method with single line content, use one-line comment instead.

     * @var string

*/

    private $method;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$id with single line content, use one-line comment instead.

     * @var bool

*/

    private $id;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$handle with single line content, use one-line comment instead.

     * @var resource|null

*/

    private $handle;

    /** @var CurlerResponse */

    private $response;

    /** @var bool */

    private $_persistent = false;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_attachFiles with single line content, use one-line comment instead.

     * @var bool

*/

    private $_attachFiles = false;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_class with single line content, use one-line comment instead.

     * @var string

*/

    private $callback_class = '';

Coding Style introduced 2020-12-21 16:16 UTC by

Member variable "callback_class" is not in valid camel caps format

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_functionName with single line content, use one-line comment instead.

     * @var string

*/

    private $callback_functionName = '';

Coding Style introduced 2020-12-21 16:16 UTC by

Member variable "callback_functionName" is not in valid camel caps format

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_httpCompression with single line content, use one-line comment instead.

     * @var bool

*/

    private $_httpCompression = false;

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$callback_function with single line content, use one-line comment instead.

     * @var callable

*/

    private $callback_function = null;

Coding Style introduced 2020-12-21 16:16 UTC by

Member variable "callback_function" is not in valid camel caps format

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$infile_handle with single line content, use one-line comment instead.

     * @var bool|resource

*/

    private $infile_handle = false;

Coding Style introduced 2020-12-21 16:16 UTC by

Member variable "infile_handle" is not in valid camel caps format

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$_dns_cache with single line content, use one-line comment instead.

     * @var int

*/

    private $_dns_cache = 120;

Coding Style introduced 2020-12-21 16:16 UTC by

Member variable "_dns_cache" is not in valid camel caps format

/**

introduced 2018-07-08 20:56 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$resultFileHandle with single line content, use one-line comment instead.

     * @var resource

*/

    private $resultFileHandle = null;

/**

introduced 2019-11-28 14:43 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$sslCa with single line content, use one-line comment instead.

     * @var string

*/

    private $sslCa = null;

introduced 2019-11-28 14:43 UTC by

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

100

101

102

/**

introduced 2022-04-20 19:38 UTC by

Found multi-line comment for property \ClickHouseDB\Transport\CurlerRequest::$stdErrOut with single line content, use one-line comment instead.

103

     * @var null|resource

introduced 2022-04-20 19:38 UTC by

Null type hint should be on last position in "null|resource".

104

*/

105

    private $stdErrOut = null;

106

/**

107

     * @param bool $id

108

*/

109

    public function __construct($id = false)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::__construct() does not have native type hint for its parameter $id but it should be possible to add it based on @param annotation "bool".

Coding Style introduced 2022-04-20 19:38 UTC by

Expected 1 blank line before function; 0 found

110

111

        $this->id = $id;

112

113

        $this->header('Cache-Control', 'no-cache, no-store, must-revalidate');

114

        $this->header('Expires', '0');

115

        $this->header('Pragma', 'no-cache');

116

117

        $this->options = array(

Coding Style introduced 2018-07-08 20:56 UTC by

Short array syntax must be used to define arrays

118

            CURLOPT_SSL_VERIFYHOST => 0,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_SSL_VERIFYHOST should not be referenced via a fallback global name, but via a use statement.

119

            CURLOPT_SSL_VERIFYPEER => false,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_SSL_VERIFYPEER should not be referenced via a fallback global name, but via a use statement.

120

            CURLOPT_TIMEOUT => 10,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_TIMEOUT should not be referenced via a fallback global name, but via a use statement.

121

            CURLOPT_CONNECTTIMEOUT => 5, // Количество секунд ожидания при попытке соединения

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_CONNECTTIMEOUT should not be referenced via a fallback global name, but via a use statement.

122

            CURLOPT_HTTP_VERSION => CURL_HTTP_VERSION_1_1,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HTTP_VERSION should not be referenced via a fallback global name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

Constant CURL_HTTP_VERSION_1_1 should not be referenced via a fallback global name, but via a use statement.

123

            CURLOPT_MAXREDIRS => 10,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_MAXREDIRS should not be referenced via a fallback global name, but via a use statement.

124

            CURLOPT_HEADER => TRUE,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

125

            CURLOPT_FOLLOWLOCATION => TRUE,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_FOLLOWLOCATION should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

126

            CURLOPT_AUTOREFERER => 1, // при редиректе подставлять в «Referer:» значение из «Location:»

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_AUTOREFERER should not be referenced via a fallback global name, but via a use statement.

127

            CURLOPT_BINARYTRANSFER => 1, // передавать в binary-safe

introduced 2021-01-19 11:08 UTC by

The constant CURLOPT_BINARYTRANSFER has been deprecated: 5.1.3 ( Ignorable by Annotation )

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_BINARYTRANSFER should not be referenced via a fallback global name, but via a use statement.

128

            CURLOPT_RETURNTRANSFER => TRUE,

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_RETURNTRANSFER should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

129

            CURLOPT_USERAGENT => 'smi2/PHPClickHouse/client',

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_USERAGENT should not be referenced via a fallback global name, but via a use statement.

130

);

131

132

133

/**

introduced 2018-07-08 20:56 UTC by

Empty comment

134

135

*/

136

    public function __destruct()

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::__destruct() does not need documentation comment.

137

138

        $this->close();

139

Coding Style introduced 2020-12-21 16:16 UTC by

Expected 1 blank line after function; 2 found

140

141

142

    public function close()

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::close() does not have void return type hint.

143

144

        if ($this->handle) {

introduced 2020-12-21 16:16 UTC by

Expected 1 lines after "if", found 0.

145

            curl_close($this->handle);

introduced 2018-07-08 20:56 UTC by

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

146

147

        $this->handle = null;

148

149

150

/**

151

     * @param array $attachFiles

introduced 2020-12-21 16:16 UTC by

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not specify type hint for items of its traversable parameter $attachFiles.

152

*/

153

    public function attachFiles($attachFiles)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not have native type hint for its parameter $attachFiles but it should be possible to add it based on @param annotation "array".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::attachFiles() does not have void return type hint.

154

155

        $this->header("Content-Type", "multipart/form-data");

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal Content-Type does not require double quotes, as per coding-style, please use single quotes.

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal multipart/form-data does not require double quotes, as per coding-style, please use single quotes.

156

157

        $out = [];

158

        foreach ($attachFiles as $post_name => $file_path) {

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $post_name should be in camel caps format.

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $file_path should be in camel caps format.

159

            $out[$post_name] = new \CURLFile($file_path);

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $post_name should be in camel caps format.

introduced 2018-09-26 10:27 UTC by

Class \CURLFile should not be referenced via a fully qualified name, but via a use statement.

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $file_path should be in camel caps format.

160

161

162

        $this->_attachFiles = true;

163

        $this->parameters($out);

164

Coding Style introduced 2020-12-21 16:16 UTC by

Expected 1 blank line after function; 2 found

165

166

167

/**

168

     * @param bool $set

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

169

     * @return $this

170

*/

171

    public function id($set = false)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::id() does not have native type hint for its parameter $set but it should be possible to add it based on @param annotation "bool".

172

173

        if ($set) {

174

            $this->id = $set;

175

176

177

        return $this;

178

179

180

/**

181

     * @param array $params

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2020-12-21 16:16 UTC by

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::setRequestExtendedInfo() does not specify type hint for items of its traversable parameter $params.

182

     * @return $this

183

*/

184

    public function setRequestExtendedInfo($params)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setRequestExtendedInfo() does not have native type hint for its parameter $params but it should be possible to add it based on @param annotation "array".

185

186

        $this->extendinfo = $params;

187

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

188

189

190

/**

191

     * @param string|integer|null $key

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2018-07-08 20:56 UTC by

Expected "int" but found "integer" in @param annotation.

192

     * @return mixed

193

*/

194

    public function getRequestExtendedInfo($key = null)

195

196

        if ($key) {

197

            return isset($this->extendinfo[$key]) ? $this->extendinfo[$key] : false;

introduced 2018-07-08 20:56 UTC by

Use null coalesce operator instead of ternary operator.

198

199

200

        return $this->extendinfo;

201

202

203

/**

204

     * @return bool|resource

205

*/

206

    public function getInfileHandle()

207

208

        return $this->infile_handle;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "infile_handle" is not in valid camel caps format

209

210

211

/**

212

     * @param string $file_name

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

213

     * @return bool|resource

214

*/

215

    public function setInfile($file_name)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setInfile() does not have native type hint for its parameter $file_name but it should be possible to add it based on @param annotation "string".

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $file_name should be in camel caps format.

216

217

        $this->header('Expect', '');

218

        $this->infile_handle = fopen($file_name, 'r');

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "infile_handle" is not in valid camel caps format

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $file_name should be in camel caps format.

219

        if (is_resource($this->infile_handle)) {

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "infile_handle" is not in valid camel caps format

Coding Style introduced 2019-11-28 14:43 UTC by

Blank line found at start of control structure

220

221

            if ($this->_httpCompression) {

222

                $this->header('Content-Encoding', 'gzip');

223

                $this->header('Content-Type', 'application/x-www-form-urlencoded');

224

225

                stream_filter_append($this->infile_handle, 'zlib.deflate', STREAM_FILTER_READ, ["window" => 30]);

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "infile_handle" is not in valid camel caps format

introduced 2018-07-08 20:56 UTC by

Constant STREAM_FILTER_READ should not be referenced via a fallback global name, but via a use statement.

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal window does not require double quotes, as per coding-style, please use single quotes.

226

227

                $this->options[CURLOPT_SAFE_UPLOAD] = 1;

introduced 2021-01-19 11:08 UTC by

The constant CURLOPT_SAFE_UPLOAD has been deprecated: 7.0 Use <b>CURLFile</b> for uploads instead. ( Ignorable by Annotation )

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_SAFE_UPLOAD should not be referenced via a fallback global name, but via a use statement.

228

            } else {

229

                $this->options[CURLOPT_INFILESIZE] = filesize($file_name);

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_INFILESIZE should not be referenced via a fallback global name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $file_name should be in camel caps format.

230

231

232

            $this->options[CURLOPT_INFILE] = $this->infile_handle;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_INFILE should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "infile_handle" is not in valid camel caps format

233

234

235

        return $this->infile_handle;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "infile_handle" is not in valid camel caps format

236

237

238

/**

239

     * @param callable $callback

240

*/

241

    public function setCallbackFunction($callback)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setCallbackFunction() does not have native type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setCallbackFunction() does not have void return type hint.

242

243

        $this->callback_function = $callback;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_function" is not in valid camel caps format

244

245

246

/**

247

     * @param callable $callback

248

*/

249

    public function setWriteFunction($callback)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setWriteFunction() does not have native type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setWriteFunction() does not have void return type hint.

250

251

        $this->options[CURLOPT_WRITEFUNCTION] = $callback;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_WRITEFUNCTION should not be referenced via a fallback global name, but via a use statement.

252

253

254

/**

255

     * @param callable $callback

256

*/

257

    public function setReadFunction($callback)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setReadFunction() does not have native type hint for its parameter $callback but it should be possible to add it based on @param annotation "callable".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setReadFunction() does not have void return type hint.

258

259

        $this->options[CURLOPT_READFUNCTION] = $callback;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_READFUNCTION should not be referenced via a fallback global name, but via a use statement.

260

261

262

    public function setHeaderFunction($callback)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setHeaderFunction() does not have parameter type hint nor @param annotation for its parameter $callback.

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setHeaderFunction() does not have void return type hint.

263

264

        $this->options[CURLOPT_HEADERFUNCTION] = $callback;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HEADERFUNCTION should not be referenced via a fallback global name, but via a use statement.

265

266

267

/**

268

     * @param string $classCallBack

269

     * @param string $functionName

270

*/

271

    public function setCallback($classCallBack, $functionName)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have native type hint for its parameter $classCallBack but it should be possible to add it based on @param annotation "string".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have native type hint for its parameter $functionName but it should be possible to add it based on @param annotation "string".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setCallback() does not have void return type hint.

272

273

        $this->callback_class = $classCallBack;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_class" is not in valid camel caps format

Coding Style introduced 2018-07-08 20:56 UTC by

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

274

        $this->callback_functionName = $functionName;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_functionName" is not in valid camel caps format

275

276

277

/**

introduced 2018-07-08 20:56 UTC by

Empty comment

278

279

*/

280

    public function onCallback()

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::onCallback() does not have void return type hint.

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::onCallback() does not need documentation comment.

281

282

        if ($this->callback_function) {

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_function" is not in valid camel caps format

283

            $x = $this->callback_function;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_function" is not in valid camel caps format

284

            $x($this);

285

286

287

        if ($this->callback_class && $this->callback_functionName) {

introduced 2018-07-08 20:56 UTC by

Use early exit to reduce code nesting.

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_class" is not in valid camel caps format

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_functionName" is not in valid camel caps format

288

            $c = $this->callback_functionName;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_functionName" is not in valid camel caps format

289

            $this->callback_class->$c($this);

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "callback_class" is not in valid camel caps format

290

291

292

293

    public function getDetails(): array

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getDetails() does not have @return annotation for its traversable return value.

294

295

        return [

296

            'url'        => $this->url,

297

            'method'     => $this->method,

298

            'parameters' => $this->parameters,

299

            'headers'    => $this->headers,

300

];

301

302

303

/**

304

     * @param resource $stream

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

305

     * @return void

306

*/

307

    public function setStdErrOut($stream)

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setStdErrOut() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "void".

308

309

        if (is_resource($stream)) {

introduced 2022-04-20 19:38 UTC by

Expected 0 lines after "if", found 1.

introduced 2022-04-20 19:38 UTC by

Use early exit to reduce code nesting.

introduced 2022-04-20 19:38 UTC by

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

310

            $this->stdErrOut=$stream;

Coding Style introduced 2022-04-20 19:38 UTC by

Expected at least 1 space before "="; 0 found

Coding Style introduced 2022-04-20 19:38 UTC by

Expected at least 1 space after "="; 0 found

Coding Style introduced 2022-04-20 19:38 UTC by

Equals sign not aligned correctly; expected 1 space but found 0 spaces

311

312

313

Coding Style introduced 2022-04-20 19:38 UTC by

Function closing brace must go on the next line following the body; found 1 blank lines before brace

314

315

/**

316

     * @param bool $result

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

317

     * @return string

318

*/

319

    public function dump($result = false)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::dump() does not have native type hint for its parameter $result but it should be possible to add it based on @param annotation "bool".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::dump() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "string".

320

321

        $message = "\n------------  Request ------------\n";

Coding Style introduced 2018-07-08 20:56 UTC by

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

322

        $message .= 'URL:' . $this->url . "\n\n";

323

        $message .= 'METHOD:' . $this->method . "\n\n";

324

        $message .= 'PARAMS:' . print_r($this->parameters, true) . "\n";

Bug introduced 2021-01-19 11:08 UTC by

Are you sure print_r($this->parameters, true) of type string|true can be used in concatenation? ( Ignorable by Annotation )

introduced 2018-07-08 20:56 UTC by

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

325

        $message .= 'PARAMS:' . print_r($this->headers, true) . "\n";

Bug introduced 2021-01-19 11:08 UTC by

Are you sure print_r($this->headers, true) of type string|true can be used in concatenation? ( Ignorable by Annotation )

introduced 2018-07-08 20:56 UTC by

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

326

        $message .= "-----------------------------------\n";

327

328

        if ($result) {

329

            return $message;

330

331

332

        echo $message;

333

        return '';

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

334

335

336

/**

337

     * @return bool

338

*/

339

    public function getId()

introduced 2020-12-21 16:16 UTC by

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

340

341

        return $this->id;

342

343

344

/**

345

     * @param integer $key

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2018-07-08 20:56 UTC by

Expected "int" but found "integer" in @param annotation.

346

     * @param mixed $value

Coding Style introduced 2018-07-08 20:56 UTC by

Expected 3 spaces after parameter type; 1 found

347

     * @return $this

348

*/

349

    private function option($key, $value)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::option() does not have native type hint for its parameter $key but it should be possible to add it based on @param annotation "integer".

350

351

        $this->options[$key] = $value;

352

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

353

354

355

/**

356

     * @return $this

357

*/

358

    public function persistent()

359

360

        $this->_persistent = true;

361

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

362

363

364

/**

365

     * @return bool

366

*/

367

    public function isPersistent()

introduced 2020-12-21 16:16 UTC by

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

368

369

        return $this->_persistent;

370

371

372

/**

373

     * @param int $sec

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::keepAlive() has useless @param annotation for parameter $sec.

374

     * @return $this

375

*/

376

    public function keepAlive(int $sec = 60)

377

378

        $this->options[CURLOPT_FORBID_REUSE] = TRUE;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_FORBID_REUSE should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

TRUE, FALSE and NULL must be lowercase; expected true, but found TRUE.

379

        $this->headers['Connection'] = 'Keep-Alive';

Coding Style introduced 2018-07-08 20:56 UTC by

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

380

        $this->headers['Keep-Alive'] = $sec;

Coding Style introduced 2018-07-08 20:56 UTC by

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

381

382

        return $this;

383

384

385

/**

386

     * @param bool $flag

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::verbose() has useless @param annotation for parameter $flag.

387

     * @return $this

388

*/

389

    public function verbose(bool $flag = true)

390

391

        $this->options[CURLOPT_VERBOSE] = $flag;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_VERBOSE should not be referenced via a fallback global name, but via a use statement.

392

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

393

394

395

/**

396

     * @param string $key

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::header() has useless @param annotation for parameter $key.

397

     * @param string $value

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::header() has useless @param annotation for parameter $value.

398

     * @return $this

399

*/

400

    public function header(string $key, string $value)

401

402

        $this->headers[$key] = $value;

403

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

404

405

406

/**

407

     * @return array

introduced 2020-12-21 16:16 UTC by

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

408

*/

409

    public function getHeaders():array

introduced 2022-04-20 19:38 UTC by

There must be exactly one space between return type hint colon and return type hint.

410

411

        $head = [];

412

        foreach ($this->headers as $key => $value) {

introduced 2020-12-21 16:16 UTC by

Expected 1 lines after "foreach", found 0.

413

            $head[] = sprintf("%s: %s", $key, $value);

introduced 2018-07-08 20:56 UTC by

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

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal %s: %s does not require double quotes, as per coding-style, please use single quotes.

414

415

        return $head;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

416

417

418

/**

419

     * @param string $url

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::url() has useless @param annotation for parameter $url.

420

     * @return $this

421

*/

422

    public function url(string $url)

423

424

        $this->url = $url;

425

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

426

427

428

/**

429

     * @return string

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getUrl() has useless @return annotation.

430

*/

431

    public function getUrl():string

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getUrl() does not need documentation comment.

introduced 2022-04-20 19:38 UTC by

There must be exactly one space between return type hint colon and return type hint.

432

433

        return $this->url;

434

Coding Style introduced 2020-12-21 16:16 UTC by

Expected 1 blank line after function; 2 found

435

436

437

/**

438

     * @param string $id

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() has useless @param annotation for parameter $id.

439

     * @return string

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() has useless @return annotation.

440

*/

441

    public function getUniqHash(string $id):string

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getUniqHash() does not need documentation comment.

introduced 2022-04-20 19:38 UTC by

There must be exactly one space between return type hint colon and return type hint.

442

443

        return $id . '.' . microtime() . mt_rand(0, 1000000);

introduced 2018-07-08 20:56 UTC by

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

introduced 2018-07-08 20:56 UTC by

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

444

445

446

/**

447

     * @param bool $flag

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() has useless @param annotation for parameter $flag.

448

*/

449

    public function httpCompression(bool $flag):void

introduced 2022-04-20 19:38 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::httpCompression() does not need documentation comment.

introduced 2022-04-20 19:38 UTC by

There must be exactly one space between return type hint colon and return type hint.

450

451

        if ($flag) {

452

            $this->_httpCompression = $flag;

Coding Style introduced 2018-07-08 20:56 UTC by

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

453

            $this->options[CURLOPT_ENCODING] = 'gzip';

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_ENCODING should not be referenced via a fallback global name, but via a use statement.

454

        } else {

455

            $this->_httpCompression = false;

456

            unset($this->options[CURLOPT_ENCODING]);

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_ENCODING should not be referenced via a fallback global name, but via a use statement.

457

458

459

460

/**

461

     * @param string $username

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

462

     * @param string $password

463

     * @return $this

464

*/

465

    public function authByBasicAuth($username, $password)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::authByBasicAuth() does not have native type hint for its parameter $username but it should be possible to add it based on @param annotation "string".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::authByBasicAuth() does not have native type hint for its parameter $password but it should be possible to add it based on @param annotation "string".

466

467

        $this->options[CURLOPT_USERPWD] = sprintf("%s:%s", $username, $password);

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_USERPWD should not be referenced via a fallback global name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

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

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal %s:%s does not require double quotes, as per coding-style, please use single quotes.

468

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

469

470

471

    public function authByHeaders($username, $password)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have parameter type hint nor @param annotation for its parameter $username.

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have parameter type hint nor @param annotation for its parameter $password.

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::authByHeaders() does not have return type hint nor @return annotation for its return value.

472

473

        $this->headers['X-ClickHouse-User'] = $username;

474

        $this->headers['X-ClickHouse-Key'] = $password;

Coding Style introduced 2019-08-24 09:12 UTC by

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

475

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

476

477

478

/**

479

     * @param array|string $data

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2020-12-21 16:16 UTC by

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::parameters() does not specify type hint for items of its traversable parameter $data.

480

     * @return $this

481

*/

482

    public function parameters($data)

483

484

        $this->parameters = $data;

485

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

486

487

488

/**

489

     * The number of seconds to wait when trying to connect. Use 0 for infinite waiting.

490

491

     * @param float $seconds

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-08-03 17:13 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::connectTimeOut() has useless @param annotation for parameter $seconds.

492

     * @return $this

493

*/

494

    public function connectTimeOut(float $seconds = 1.0)

495

496

        $this->options[CURLOPT_CONNECTTIMEOUT_MS] = (int) ($seconds*1000.0);

introduced 2022-08-03 17:13 UTC by

Constant CURLOPT_CONNECTTIMEOUT_MS should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2022-08-03 17:13 UTC by

Expected at least 1 space before "*"; 0 found

Coding Style introduced 2022-08-03 17:13 UTC by

Expected at least 1 space after "*"; 0 found

497

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

498

499

500

/**

501

     * The maximum number of seconds (float) allowed to execute cURL functions.

502

503

     * @param float $seconds

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2022-08-03 17:13 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::timeOut() has useless @param annotation for parameter $seconds.

504

     * @return $this

505

*/

506

    public function timeOut(float $seconds = 10)

507

508

        return $this->timeOutMs((int) ($seconds * 1000.0));

509

510

511

/**

512

     * The maximum allowed number of milliseconds to perform cURL functions.

513

514

     * @param int $ms millisecond

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

515

     * @return $this

516

*/

517

    protected function timeOutMs(int $ms = 10000)

518

519

        $this->options[CURLOPT_TIMEOUT_MS] = $ms;

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_TIMEOUT_MS should not be referenced via a fallback global name, but via a use statement.

520

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

521

Coding Style introduced 2020-12-21 16:16 UTC by

Expected 1 blank line after function; 2 found

522

523

524

/**

525

     * @param array|mixed $data

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2020-12-21 16:16 UTC by

@param annotation of method \ClickHouseDB\Transport\CurlerRequest::parameters_json() does not specify type hint for items of its traversable parameter $data.

526

     * @return $this

527

     * @throws \ClickHouseDB\Exception\TransportException

introduced 2018-07-08 20:56 UTC by

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

528

*/

529

    public function parameters_json($data)

Coding Style introduced 2018-07-08 20:56 UTC by

Method name "CurlerRequest::parameters_json" is not in camel caps format

530

Coding Style introduced 2018-07-08 20:56 UTC by

Expected 0 blank lines after opening function brace; 1 found

531

532

        $this->header("Content-Type", "application/json, text/javascript; charset=utf-8");

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal Content-Type does not require double quotes, as per coding-style, please use single quotes.

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal application/json, text/javascript; charset=utf-8 does not require double quotes, as per coding-style, please use single quotes.

533

        $this->header("Accept", "application/json, text/javascript, */*; q=0.01");

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal Accept does not require double quotes, as per coding-style, please use single quotes.

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal application/json, text/javascript, */*; q=0.01 does not require double quotes, as per coding-style, please use single quotes.

534

535

        if ($data === null) {

536

            $this->parameters = '{}';

537

            return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

538

539

540

        if (is_string($data)) {

introduced 2018-07-08 20:56 UTC by

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

541

            $this->parameters = $data;

542

            return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

543

544

545

        $this->parameters = json_encode($data);

introduced 2018-07-08 20:56 UTC by

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

546

547

        if (!$this->parameters && $data) {

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

548

            throw new \ClickHouseDB\Exception\TransportException('Cant json_encode: ' . strval($data));

introduced 2018-07-08 20:56 UTC by

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

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

549

550

551

        return $this;

552

553

554

/**

555

     * @return resource

556

*/

557

    public function getResultFileHandle()

558

559

        return $this->resultFileHandle;

560

561

562

/**

563

     * @return bool

564

*/

565

    public function isResultFile()

introduced 2020-12-21 16:16 UTC by

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

566

567

        return ($this->resultFileHandle ? true : false);

introduced 2018-07-08 20:56 UTC by

Usage of language construct "return" with parentheses is disallowed.

introduced 2020-12-21 16:16 UTC by

Useless ternary operator.

568

569

570

/**

571

     * @param resource $h resource

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

Coding Style introduced 2018-07-08 20:56 UTC by

Expected 4 spaces after parameter name; 1 found

572

     * @param bool $zlib

Coding Style introduced 2018-07-08 20:56 UTC by

Expected 5 spaces after parameter type; 1 found

573

     * @return $this

574

*/

575

    public function setResultFileHandle($h, $zlib = false)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setResultFileHandle() does not have native type hint for its parameter $zlib but it should be possible to add it based on @param annotation "bool".

576

577

        $this->resultFileHandle = $h;

578

        if ($zlib) {

introduced 2020-12-21 16:16 UTC by

Expected 1 lines after "if", found 0.

579

            $params = array('level' => 6, 'window' => 15, 'memory' => 9);

Coding Style introduced 2018-07-08 20:56 UTC by

Short array syntax must be used to define arrays

580

            stream_filter_append($this->resultFileHandle, 'zlib.deflate', STREAM_FILTER_WRITE, $params);

introduced 2018-07-08 20:56 UTC by

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

introduced 2018-07-08 20:56 UTC by

Constant STREAM_FILTER_WRITE should not be referenced via a fallback global name, but via a use statement.

581

582

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

583

584

585

/**

586

     * @return CurlerRequest

587

*/

588

    public function PUT()

introduced 2020-12-21 16:16 UTC by

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

589

590

        return $this->execute('PUT');

591

592

593

/**

594

     * @return CurlerRequest

595

*/

596

    public function POST()

introduced 2020-12-21 16:16 UTC by

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

597

598

        return $this->execute('POST');

599

600

601

/**

602

     * @return CurlerRequest

603

*/

604

    public function OPTIONS()

introduced 2020-12-21 16:16 UTC by

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

605

606

        return $this->execute('OPTIONS');

607

608

609

/**

610

     * @return CurlerRequest

611

*/

612

    public function GET()

introduced 2020-12-21 16:16 UTC by

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

613

614

        return $this->execute('GET');

615

616

617

/**

618

     * The number of seconds that DNS records are stored in memory. By default this parameter is 120 (2 minutes).

619

620

     * @param integer $set

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

introduced 2018-07-08 20:56 UTC by

Expected "int" but found "integer" in @param annotation.

621

     * @return $this

622

*/

623

    public function setDnsCache($set)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setDnsCache() does not have native type hint for its parameter $set but it should be possible to add it based on @param annotation "integer".

624

625

        $this->_dns_cache = $set;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "_dns_cache" is not in valid camel caps format

626

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

627

628

629

/**

630

     * The number of seconds that DNS records are stored in memory. By default this parameter is 120 (2 minutes).

631

632

     * @return int

633

*/

634

    public function getDnsCache()

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::getDnsCache() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "int".

635

636

        return $this->_dns_cache;

Coding Style introduced 2021-10-08 16:39 UTC by

Member variable "_dns_cache" is not in valid camel caps format

637

638

639

/**

640

     * Sets client certificate

641

642

     * @param string $filePath

643

*/

644

    public function setSslCa($filePath)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setSslCa() does not have native type hint for its parameter $filePath but it should be possible to add it based on @param annotation "string".

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setSslCa() does not have void return type hint.

645

646

        $this->option(CURLOPT_SSL_VERIFYPEER, true);

introduced 2019-11-28 14:43 UTC by

Constant CURLOPT_SSL_VERIFYPEER should not be referenced via a fallback global name, but via a use statement.

647

        $this->option(CURLOPT_CAINFO, $filePath);

introduced 2019-11-28 14:43 UTC by

Constant CURLOPT_CAINFO should not be referenced via a fallback global name, but via a use statement.

648

649

650

/**

651

     * @param string $method

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

652

     * @return $this

653

*/

654

    private function execute($method)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::execute() does not have native type hint for its parameter $method but it should be possible to add it based on @param annotation "string".

655

656

        $this->method = $method;

657

        return $this;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

658

659

660

/**

661

     * @return CurlerResponse

introduced 2018-09-26 10:27 UTC by

Incorrect annotations group.

662

     * @throws \ClickHouseDB\Exception\TransportException

introduced 2018-07-08 20:56 UTC by

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

663

*/

664

    public function response()

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::response() does not have native return type hint for its return value but it should be possible to add it based on @return annotation "CurlerResponse".

665

666

        if (!$this->response) {

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

667

            throw new \ClickHouseDB\Exception\TransportException('Can`t fetch response - is empty');

introduced 2018-07-08 20:56 UTC by

Class \ClickHouseDB\Exception\TransportException should not be referenced via a fully qualified name, but via a use statement.

668

669

670

        return $this->response;

671

672

673

    public function isResponseExists(): bool

674

675

        return $this->response !== null;

676

677

678

    public function setResponse(CurlerResponse $response): void

679

680

        $this->response = $response;

681

682

683

/**

684

     * @return mixed

685

*/

686

    public function handle()

687

688

        $this->prepareRequest();

689

        return $this->handle;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

690

691

692

/**

693

     * @param callable $callback

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setFunctionProgress() has useless @param annotation for parameter $callback.

introduced 2022-04-20 19:38 UTC by

Incorrect annotations group.

694

     * @throws \Exception

introduced 2018-09-26 10:27 UTC by

Class \Exception should not be referenced via a fully qualified name, but via a use statement.

695

*/

696

    public function setFunctionProgress(callable $callback)

introduced 2020-12-21 16:16 UTC by

Method \ClickHouseDB\Transport\CurlerRequest::setFunctionProgress() does not have void return type hint.

697

698

        if (!is_callable($callback)) {

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

699

            throw new \Exception('setFunctionProgress not is_callable');

introduced 2018-09-26 10:27 UTC by

Class \Exception should not be referenced via a fully qualified name, but via a use statement.

700

701

702

        $this->option(CURLOPT_NOPROGRESS, false);

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_NOPROGRESS should not be referenced via a fallback global name, but via a use statement.

703

        $this->option(CURLOPT_PROGRESSFUNCTION, $callback); // version 5.5.0

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_PROGRESSFUNCTION should not be referenced via a fallback global name, but via a use statement.

704

Coding Style introduced 2020-12-21 16:16 UTC by

Expected 1 blank line after function; 2 found

705

706

707

/**

708

     * @return bool

709

*/

710

    private function prepareRequest()

introduced 2020-12-21 16:16 UTC by

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

711

712

        if (!$this->handle) {

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

713

            $this->handle = curl_init();

Documentation Bug introduced 2021-01-19 11:08 UTC by

It seems like curl_init() can also be of type CurlHandle. However, the property $handle is declared as type null|resource. Maybe add an additional type check?

introduced 2018-07-08 20:56 UTC by

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

714

715

716

        $curl_opt = $this->options;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

717

        $method = $this->method;

Coding Style introduced 2018-07-08 20:56 UTC by

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

718

719

        if ($this->_attachFiles) {

introduced 2020-12-21 16:16 UTC by

Expected 1 lines after "if", found 2.

720

            $curl_opt[CURLOPT_SAFE_UPLOAD] = true;

introduced 2021-01-19 11:08 UTC by

The constant CURLOPT_SAFE_UPLOAD has been deprecated: 7.0 Use <b>CURLFile</b> for uploads instead. ( Ignorable by Annotation )

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_SAFE_UPLOAD should not be referenced via a fallback global name, but via a use statement.

721

722

Coding Style introduced 2018-07-08 20:56 UTC by

Functions must not contain multiple empty lines in a row; found 2 empty lines

723

724

        if (strtoupper($method) == 'GET') {

introduced 2020-12-21 16:16 UTC by

Expected 1 lines after "if", found 0.

introduced 2018-07-08 20:56 UTC by

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

introduced 2020-12-21 16:16 UTC by

Operator == is disallowed, use === instead.

725

            $curl_opt[CURLOPT_HTTPGET] = true;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

Coding Style introduced 2018-07-08 20:56 UTC by

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

726

            $curl_opt[CURLOPT_CUSTOMREQUEST] = strtoupper($method);

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_CUSTOMREQUEST should not be referenced via a fallback global name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

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

727

            $curl_opt[CURLOPT_POSTFIELDS] = false;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

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

728

        } else {

729

            if (strtoupper($method) === 'POST') {

introduced 2018-07-08 20:56 UTC by

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

730

                $curl_opt[CURLOPT_POST] = true;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

731

732

733

            $curl_opt[CURLOPT_CUSTOMREQUEST] = strtoupper($method);

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_CUSTOMREQUEST should not be referenced via a fallback global name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

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

734

735

            if ($this->parameters) {

736

                $curl_opt[CURLOPT_POSTFIELDS] = $this->parameters;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.

737

738

                if (!is_array($this->parameters)) {

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

739

                    $this->header('Content-Length',  mb_strlen($this->parameters, '8bit'));

Coding Style introduced 2021-10-08 16:39 UTC by

Expected 1 space after comma in argument list; 2 found

introduced 2021-10-08 16:39 UTC by

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

740

741

742

743

        // CURLOPT_DNS_CACHE_TIMEOUT - Количество секунд, в течение которых в памяти хранятся DNS-записи.

744

        $curl_opt[CURLOPT_DNS_CACHE_TIMEOUT] = $this->getDnsCache();

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_DNS_CACHE_TIMEOUT should not be referenced via a fallback global name, but via a use statement.

745

        $curl_opt[CURLOPT_URL] = $this->url;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_URL should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

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

746

747

        if (!empty($this->headers) && sizeof($this->headers)) {

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2018-07-08 20:56 UTC by

The use of function sizeof() is forbidden; use count() instead

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

748

            $curl_opt[CURLOPT_HTTPHEADER] = [];

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HTTPHEADER should not be referenced via a fallback global name, but via a use statement.

749

750

            foreach ($this->headers as $key => $value) {

751

                $curl_opt[CURLOPT_HTTPHEADER][] = sprintf("%s: %s", $key, $value);

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HTTPHEADER should not be referenced via a fallback global name, but via a use statement.

introduced 2018-07-08 20:56 UTC by

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

Coding Style Comprehensibility introduced 2018-07-08 20:56 UTC by

The string literal %s: %s does not require double quotes, as per coding-style, please use single quotes.

752

753

754

755

        if (!empty($curl_opt[CURLOPT_INFILE])) {

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_INFILE should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

Blank line found at start of control structure

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

756

757

            $curl_opt[CURLOPT_PUT] = true;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_PUT should not be referenced via a fallback global name, but via a use statement.

758

759

760

        if (!empty($curl_opt[CURLOPT_WRITEFUNCTION])) {

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_WRITEFUNCTION should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2024-01-10 11:49 UTC by

Expected 1 space after NOT operator; 0 found

761

            $curl_opt[CURLOPT_HEADER] = false;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.

762

763

764

        if ($this->resultFileHandle) {

765

            $curl_opt[CURLOPT_FILE] = $this->resultFileHandle;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_FILE should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2018-07-08 20:56 UTC by

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

766

            $curl_opt[CURLOPT_HEADER] = false;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_HEADER should not be referenced via a fallback global name, but via a use statement.

767

768

769

        if ($this->options[CURLOPT_VERBOSE]) {

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_VERBOSE should not be referenced via a fallback global name, but via a use statement.

770

            $msg="\n-----------BODY REQUEST----------\n" . $curl_opt[CURLOPT_POSTFIELDS] . "\n------END--------\n";

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2018-07-08 20:56 UTC by

Constant CURLOPT_POSTFIELDS should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2022-04-20 19:38 UTC by

Expected at least 1 space before "="; 0 found

Coding Style introduced 2022-04-20 19:38 UTC by

Expected at least 1 space after "="; 0 found

Coding Style introduced 2022-04-20 19:38 UTC by

Equals sign not aligned correctly; expected 1 space but found 0 spaces

771

            if ($this->stdErrOut && is_resource($this->stdErrOut)) {

introduced 2022-04-20 19:38 UTC by

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

772

                fwrite($this->stdErrOut,$msg);

introduced 2022-04-20 19:38 UTC by

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

773

            } else {

774

                echo $msg;

775

776

777

778

        if ($this->stdErrOut) {

779

            if (is_resource($this->stdErrOut)) {

introduced 2022-04-20 19:38 UTC by

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

780

                $curl_opt[CURLOPT_STDERR]=$this->stdErrOut;

Coding Style introduced 2020-12-21 16:16 UTC by

The variable $curl_opt should be in camel caps format.

introduced 2022-04-20 19:38 UTC by

Constant CURLOPT_STDERR should not be referenced via a fallback global name, but via a use statement.

Coding Style introduced 2022-04-20 19:38 UTC by

Expected at least 1 space before "="; 0 found

Coding Style introduced 2022-04-20 19:38 UTC by

Expected at least 1 space after "="; 0 found

Coding Style introduced 2022-04-20 19:38 UTC by

Equals sign not aligned correctly; expected 1 space but found 0 spaces

781

782

783

784

        curl_setopt_array($this->handle, $curl_opt);

introduced 2018-07-08 20:56 UTC by

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

Coding Style introduced 2022-04-20 19:38 UTC by

The variable $curl_opt should be in camel caps format.

785

        return true;

introduced 2020-12-21 16:16 UTC by

Expected 1 lines before "return", found 0.

786

787

788

smi2 / phpClickHouse

CurlerRequest::parameters_json() A last analyzed 2024-07-17 07:54 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

CurlerRequest::parameters_json() A
last analyzed 2024-07-17 07:54 UTC