ClickHouseDB\Transport\StreamInsert::__construct()

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function __construct($source, CurlerRequest $request, $curlerRolling=null)

Incorrect spacing between argument "$curlerRolling" and equals sign; expected 1 but found 0

Incorrect spacing between default value and equals sign for argument "$curlerRolling"; expected 1 but found 0

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

    {
        if (!is_resource($source)) {

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

Expected 1 lines after "if", found 0.

Expected 1 space after NOT operator; 0 found

            throw new \InvalidArgumentException('Argument $source must be resource');

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

        }
        if ($curlerRolling instanceof CurlerRolling)

Expected 1 lines after "if", found 0.

        {
            $this->curlerRolling = $curlerRolling;
        } else {
            $this->curlerRolling = new CurlerRolling();
        }
        $this->source = $source;

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

        $this->request = $request;
    }