Edyan\Neuralyzer\Helper\DB\SQLServer::loadData()

Complexity

Conditions	4
Paths	4

Size

Total Lines	19
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4.5923

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function loadData(string $table, string $fname, array $fields, string $mode): string
    {
        if (substr(gethostbyname($this->conn->getHost()), 0, 3) !== '127') {

The function Doctrine\DBAL\Connection::getHost() has been deprecated.

It seems like $this->conn->getHost() can also be of type null; however, parameter $hostname of gethostbyname() does only seem to accept string, maybe add an additional type check?

            throw new NeuralyzerException('SQL Server must be on the same host than PHP');
        }

        $sql = "BULK INSERT {$table} FROM '{$fname}' WITH (
            FIELDTERMINATOR = '|', DATAFILETYPE = 'widechar', ROWTERMINATOR = '" . PHP_EOL . "'
        )";

        if ($this->pretend === false) {
            if ($mode === 'update') {
                $this->conn->query("TRUNCATE TABLE {$table}");

The function Doctrine\DBAL\Connection::query() has been deprecated: Use {@link executeQuery()} instead.

            }

            $this->conn->query($sql);

The function Doctrine\DBAL\Connection::query() has been deprecated: Use {@link executeQuery()} instead.

        }

        return $sql;
    }