Function updateTables has a Cognitive Complexity of 150 (exceeds 5 allowed). Consider refactoring.

Open

    public function updateTables(Base $importer)
    {
        $this->logs .= "> start update tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        $schema = $importer->db->getSchema();

• Read up
  Read up

File Importer.php has 658 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
 * File that imports structure and data to database.
 *
 * @package App

Function addData has a Cognitive Complexity of 42 (exceeds 5 allowed). Consider refactoring.

Open

    public function addData(Base $importer)
    {
        if (!isset($importer->data)) {
            return;
        }

• Read up
  Read up

Method updateTables has 154 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function updateTables(Base $importer)
    {
        $this->logs .= "> start update tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        $schema = $importer->db->getSchema();

Function addTables has a Cognitive Complexity of 34 (exceeds 5 allowed). Consider refactoring.

Open

    public function addTables(Base $importer)
    {
        $this->logs .= "> start add tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        foreach ($importer->tables as $tableName => $table) {

• Read up
  Read up

The class Importer has 22 public methods. Consider refactoring Importer to keep number of public methods under 10.

Open

class Importer
{
    /**
     * End of line character.
     *

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The class Importer has an overall complexity of 170 which is very high. The configured complexity threshold is 50.

Open

class Importer
{
    /**
     * End of line character.
     *

• Exclude checks
  • Disable engine
  • Disable check

Importer has 26 functions (exceeds 20 allowed). Consider refactoring.

Open

class Importer
{
    /**
     * End of line character.
     *

Method addData has 67 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function addData(Base $importer)
    {
        if (!isset($importer->data)) {
            return;
        }

Method addTables has 53 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function addTables(Base $importer)
    {
        $this->logs .= "> start add tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        foreach ($importer->tables as $tableName => $table) {

Function updateForeignKey has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.

Open

    public function updateForeignKey(Base $importer)
    {
        if (!isset($importer->foreignKey)) {
            return;
        }

• Read up
  Read up

Function dropForeignKeys has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function dropForeignKeys(array $foreignKeys)
    {
        $this->logs .= "> start drop foreign keys\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

• Read up
  Read up

Method updateForeignKey has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function updateForeignKey(Base $importer)
    {
        if (!isset($importer->foreignKey)) {
            return;
        }

Function dropIndexes has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function dropIndexes(array $tables)
    {
        $this->logs .= "> start drop indexes\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

• Read up
  Read up

Function getIndexes has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function getIndexes(Base $importer, $table)
    {
        if (!isset($table['index'])) {
            return false;
        }

• Read up
  Read up

Function loadFiles has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public function loadFiles($path = false)
    {
        $dir = new \DirectoryIterator($path ?: $this->path);
        foreach ($dir as $fileinfo) {
            if ('dir' !== $fileinfo->getType() && 'php' === $fileinfo->getExtension()) {

• Read up
  Read up

Method dropForeignKeys has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function dropForeignKeys(array $foreignKeys)
    {
        $this->logs .= "> start drop foreign keys\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

Function renameColumns has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function renameColumns($columns)
    {
        $this->logs .= "> start rename columns\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

• Read up
  Read up

Function renameTables has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function renameTables($tables)
    {
        $this->logs .= "> start rename tables\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

• Read up
  Read up

Function dropColumns has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function dropColumns($columns)
    {
        $this->logs .= "> start drop columns\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

• Read up
  Read up

Function dropTable has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    public function dropTable($tables)
    {
        $this->logs .= "> start drop tables\n";
        $startMain = microtime(true);
        $db = \App\Db::getInstance();

• Read up
  Read up

Function addForeignKey has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function addForeignKey(Base $importer)
    {
        if (!isset($importer->foreignKey)) {
            return;
        }

• Read up
  Read up

The method addData() has an NPath complexity of 640. The configured NPath complexity threshold is 200.

Open

    public function addData(Base $importer)
    {
        if (!isset($importer->data)) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method updateTables() has an NPath complexity of 2057681. The configured NPath complexity threshold is 200.

Open

    public function updateTables(Base $importer)
    {
        $this->logs .= "> start update tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        $schema = $importer->db->getSchema();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method updateTables() has 157 lines of code. Current threshold is set to 100. Avoid really long methods.

Open

    public function updateTables(Base $importer)
    {
        $this->logs .= "> start update tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        $schema = $importer->db->getSchema();

• Exclude checks
  • Disable engine
  • Disable check

The method addTables() has a Cyclomatic Complexity of 14. The configured cyclomatic complexity threshold is 10.

Open

    public function addTables(Base $importer)
    {
        $this->logs .= "> start add tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        foreach ($importer->tables as $tableName => $table) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method addData() has a Cyclomatic Complexity of 18. The configured cyclomatic complexity threshold is 10.

Open

    public function addData(Base $importer)
    {
        if (!isset($importer->data)) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method updateTables() has a Cyclomatic Complexity of 48. The configured cyclomatic complexity threshold is 10.

Open

    public function updateTables(Base $importer)
    {
        $this->logs .= "> start update tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        $schema = $importer->db->getSchema();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

This function "updateTables" has 156 lines, which is greater than the 150 lines authorized. Split it into smaller functions.

Open

    public function updateTables(Base $importer)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Class "Importer" has 26 methods, which is greater than 20 authorized. Split it into smaller classes.

Open

class Importer

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Refactor this function to reduce its Cognitive Complexity from 42 to the 15 allowed.

Open

    public function addData(Base $importer)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Refactor this function to reduce its Cognitive Complexity from 39 to the 15 allowed.

Open

    public function addTables(Base $importer)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Refactor this function to reduce its Cognitive Complexity from 175 to the 15 allowed.

Open

    public function updateTables(Base $importer)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

The method loadFiles has a boolean flag argument $path, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function loadFiles($path = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '350', column '15').

Open

                    if ((new \App\Db\Query())->from($tableName)->where(array_combine($keys, $table['values'][0]))->exists($importer->db)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '78', column '14').

Open

        $dir = new \DirectoryIterator($path ?: $this->path);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '366', column '16').

Open

                    throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '325', column '16').

Open

                    throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '394', column '17').

Open

                        throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '767', column '18').

Open

                            throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '204', column '18').

Open

                            throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '221', column '18').

Open

                            throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '797', column '19').

Open

                                throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '731', column '16').

Open

                    throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method logs has a boolean flag argument $show, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function logs($show = true)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '400', column '17').

Open

                        throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '189', column '16').

Open

                    throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method updateTables uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $tableSchema = $schema->getTableSchema($tableName);
                    foreach ($this->getColumns($importer, $table) as $columnName => $column) {
                        $renameFrom = $mode = null;
                        if (\is_array($column)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\Db' in method 'renameTables'.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Db' in method 'dropForeignKeys'.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method logs uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            file_put_contents('cache/logs/Importer.log', $this->logs . '-------------  ' . date('Y-m-d H:i:s') . " ($time min)   -------------\n", LOCK_EX);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\Db' in method 'renameColumns'.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class '\App\Db' in method 'dropIndexes'.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '734', column '8').

Open

    public function updateTables(Base $importer)
    {
        $this->logs .= "> start update tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        $schema = $importer->db->getSchema();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method addData uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $this->logs .= "| Error: No values\n";
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method renameTables uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $this->logs .= " | Error - table does not exist\n";
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method dropTable uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $this->logs .= " | Info - table does not exist\n";
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method dropForeignKeys uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $this->logs .= " | Error - table does not exists\n";
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\Db' in method 'dropColumns'.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method updateTables uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                        } else {
                            $this->logs .= "  > create index: {$index[0]} ... ";
                            $start = microtime(true);
                            $dbCommand->createIndex($index[0], $tableName, $index[1], !empty($index[2]))->execute();
                            $time = round((microtime(true) - $start), 1);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method addData uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                    } else {
                        $start = microtime(true);
                        foreach ($table['values'] as $values) {
                            $importer->db->createCommand()->insert($tableName, array_combine($keys, $values))->execute();
                        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method dropColumns uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $this->logs .= " | Info - table or column does not exist\n";
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method dropForeignKeys uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $this->logs .= " | Info - foreign key not exists\n";
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method renameColumns uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $this->logs .= " | Warning - table or column does not exists\n";
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid assigning values to variables in if clauses and the like (line '192', column '8').

Open

    public function addTables(Base $importer)
    {
        $this->logs .= "> start add tables ({$importer->dbType})\n";
        $startMain = microtime(true);
        foreach ($importer->tables as $tableName => $table) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method dropIndexes uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $this->logs .= " | Info - index not exists\n";
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class '\App\Db' in method 'dropTable'.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method updateTables uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                            } else {
                                foreach ($dbIndexes[$index[0]] as $dbIndex) {
                                    if (empty($index[2]) !== empty($dbIndex['unique'])) {
                                        $update = true;
                                    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Define a constant instead of duplicating this literal "columns" 3 times.

Open

        if (empty($table['columns'])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "Importer error: " 10 times.

Open

                    throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "values" 4 times.

Open

                if (\is_array($table['values']) && isset($table['values'][0])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "Y-m-d H:i:s" 3 times.

Open

        $this->logs = '-------------   ' . date('Y-m-d H:i:s') . "   -------------\n";

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Define a constant instead of duplicating this literal "primaryKeys" 7 times.

Open

            if (isset($table['primaryKeys'])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Call to undeclared method \App\Db::getDriverName

Open

        if ('mysql' === $importer->db->getDriverName()) {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getSchema

Open

        \App\Db::getInstance()->getSchema()->getTableSchemas('', true);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                        $importer->db->createCommand()->addPrimaryKey($primaryKey[0], $tableName, $primaryKey[1])->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getDriverName

Open

        $type = $importer->db->getDriverName();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                $importer->db->createCommand()->createTable($tableName, $this->getColumns($importer, $table), $this->getOptions($importer, $table))->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getDriverName

Open

        $type = $importer->db->getDriverName();

• Exclude checks
  • Disable engine
  • Disable check

Returning type false but getIndexes() is declared to return array

Open

            return false;

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                        $importer->db->createCommand()->createIndex($index[0], $tableName, $index[1], (isset($index[2]) && $index[2]) ? true : false)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \App\Db->tablePrefix

Open

                $keyName = str_replace('#__', $db->tablePrefix, $keyName);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                        $db->createCommand()->dropForeignKey($keyName, $tableName)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = $db->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

                        if ($columnExists && $column instanceof \yii\db\ColumnSchemaBuilder && $this->compareColumns($queryBuilder, $tableSchema->columns[$columnName], $column)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to constant TYPE_BIGINT from undeclared class \yii\db\Schema (Did you mean class \App\Db\Drivers\Mysql\Schema or class \App\Db\Drivers\Pgsql\Schema)

Open

                                $column->set('type', \in_array($column->get('type'), ['ubigpk', 'bigpk']) ? \yii\db\Schema::TYPE_BIGINT : \yii\db\Schema::TYPE_INTEGER);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = $importer->db->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

Returning type ?string but getOptions() is declared to return string

Open

        return $options;

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                $importer->db->createCommand()->addForeignKey($key[0], $key[1], $key[2], $key[3], $key[4], $key[5] ?? null, $key[6] ?? null)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db\Query::from

Open

                    if ((new \App\Db\Query())->from($tableName)->where(array_combine($keys, $table['values'][0]))->exists($importer->db)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                            $importer->db->createCommand()->insert($tableName, array_combine($keys, $values))->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = $db->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getSchema

Open

        $schema = $db->getSchema();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property isUnsigned from undeclared class \yii\db\ColumnSchemaBuilder

Open

        || ($baseColumn->unsigned !== $targetColumn->isUnsigned)

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property autoIncrement from undeclared class \yii\db\ColumnSchema

Open

        || ($baseColumn->autoIncrement !== $targetColumn->autoIncrement);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getSchema

Open

        $schema = $importer->db->getSchema();

• Exclude checks
  • Disable engine
  • Disable check

Reference to constant TYPE_INTEGER from undeclared class \yii\db\Schema (Did you mean class \App\Db\Drivers\Mysql\Schema or class \App\Db\Drivers\Pgsql\Schema)

Open

                                $column->set('type', \in_array($column->get('type'), ['ubigpk', 'bigpk']) ? \yii\db\Schema::TYPE_BIGINT : \yii\db\Schema::TYPE_INTEGER);

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

                        } elseif (!($column instanceof \yii\db\ColumnSchemaBuilder)) {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

            $importer->db->createCommand()->checkIntegrity($check)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property allowNull from undeclared class \yii\db\ColumnSchema

Open

        || ($baseColumn->allowNull !== (null === $targetColumn->isNotNull))

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property defaultValue from undeclared class \yii\db\ColumnSchema

Open

        || ($baseColumn->defaultValue !== $targetColumn->default)

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getSchema

Open

        $schema = $db->getSchema();

• Exclude checks
  • Disable engine
  • Disable check

Call to method set from undeclared class \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

                                $column->set('type', \in_array($column->get('type'), ['ubigpk', 'bigpk']) ? \yii\db\Schema::TYPE_BIGINT : \yii\db\Schema::TYPE_INTEGER);

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property autoIncrement from undeclared class \yii\db\ColumnSchemaBuilder

Open

        || ($baseColumn->autoIncrement !== $targetColumn->autoIncrement);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getSchema

Open

        $schema = $importer->db->getSchema();

• Exclude checks
  • Disable engine
  • Disable check

Checking instanceof against undeclared class \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

                            if ($column instanceof \yii\db\ColumnSchemaBuilder && (\in_array($column->get('type'), ['upk', 'pk', 'ubigpk', 'bigpk']))) {

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                    $db->createCommand()->dropTable($tableName)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = $db->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

Call to method get from undeclared class \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

                            if ($column instanceof \yii\db\ColumnSchemaBuilder && (\in_array($column->get('type'), ['upk', 'pk', 'ubigpk', 'bigpk']))) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method get from undeclared class \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

                                $column->set('type', \in_array($column->get('type'), ['ubigpk', 'bigpk']) ? \yii\db\Schema::TYPE_BIGINT : \yii\db\Schema::TYPE_INTEGER);

• Exclude checks
  • Disable engine
  • Disable check

Call to method getColumnType from undeclared class \yii\db\QueryBuilder

Open

        return strtok($baseColumn->dbType, ' ') !== strtok($queryBuilder->getColumnType($targetColumn), ' ')

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property default from undeclared class \yii\db\ColumnSchemaBuilder

Open

        || ($baseColumn->defaultValue !== $targetColumn->default)

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getTableSchema

Open

            $tableSchema = $importer->db->getTableSchema($tableName);

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                    $importer->db->createCommand()->resetSequence($tableName)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

        $dbCommand = $importer->db->createCommand();

• Exclude checks
  • Disable engine
  • Disable check

Parameter $queryBuilder has undeclared type \yii\db\QueryBuilder

Open

    protected function compareColumns(\yii\db\QueryBuilder $queryBuilder, \yii\db\ColumnSchema $baseColumn, \yii\db\ColumnSchemaBuilder $targetColumn)

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property dbType from undeclared class \yii\db\ColumnSchema

Open

        return strtok($baseColumn->dbType, ' ') !== strtok($queryBuilder->getColumnType($targetColumn), ' ')

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::getTableSchema

Open

            $tableSchema = $db->getTableSchema($tableName, true);

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property unsigned from undeclared class \yii\db\ColumnSchema

Open

        || ($baseColumn->unsigned !== $targetColumn->isUnsigned)

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \App\Db::createCommand

Open

                        $db->createCommand()->dropIndex($index, $tableName)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Parameter $baseColumn has undeclared type \yii\db\ColumnSchema

Open

    protected function compareColumns(\yii\db\QueryBuilder $queryBuilder, \yii\db\ColumnSchema $baseColumn, \yii\db\ColumnSchemaBuilder $targetColumn)

• Exclude checks
  • Disable engine
  • Disable check

Parameter $targetColumn has undeclared type \yii\db\ColumnSchemaBuilder (Did you mean class \App\Db\Drivers\Mysql\ColumnSchemaBuilder or class \App\Db\Drivers\Pgsql\ColumnSchemaBuilder)

Open

    protected function compareColumns(\yii\db\QueryBuilder $queryBuilder, \yii\db\ColumnSchema $baseColumn, \yii\db\ColumnSchemaBuilder $targetColumn)

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property isNotNull from undeclared class \yii\db\ColumnSchemaBuilder

Open

        || ($baseColumn->allowNull !== (null === $targetColumn->isNotNull))

• Exclude checks
  • Disable engine
  • Disable check

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if ($db->isTableExists($table[0])) {
                $start = microtime(true);
                try {
                    $dbCommand->renameTable($table[0], $table[1])->execute();
                    $time = round((microtime(true) - $start), 1);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            if ($tableSchema && isset($tableSchema->columns[$column[1]])) {
                $start = microtime(true);
                try {
                    $dbCommand->dropColumn($column[0], $column[1])->execute();
                    $time = round((microtime(true) - $start), 1);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (isset($dbIndexes[$index])) {
                    $start = microtime(true);
                    try {
                        $db->createCommand()->dropIndex($index, $tableName)->execute();
                        $time = round((microtime(true) - $start), 1);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                if (isset($tableSchema->foreignKeys[$keyName])) {
                    $start = microtime(true);
                    try {
                        $db->createCommand()->dropForeignKey($keyName, $tableName)->execute();
                        $time = round((microtime(true) - $start), 1);

• Read up
  Read up

Similar blocks of code found in 6 locations. Consider refactoring.

Open

            try {
                $importer->db->createCommand()->createTable($tableName, $this->getColumns($importer, $table), $this->getOptions($importer, $table))->execute();
                $time = round((microtime(true) - $start), 1);
                $this->logs .= "done    ({$time}s)\n";
            } catch (\Throwable $e) {

• Read up
  Read up

Similar blocks of code found in 6 locations. Consider refactoring.

Open

                    try {
                        $importer->db->createCommand()->createIndex($index[0], $tableName, $index[1], (isset($index[2]) && $index[2]) ? true : false)->execute();
                        $time = round((microtime(true) - $start), 1);
                        $this->logs .= "done    ({$time}s)\n";
                    } catch (\Throwable $e) {

• Read up
  Read up

Similar blocks of code found in 6 locations. Consider refactoring.

Open

                    try {
                        $importer->db->createCommand()->addPrimaryKey($primaryKey[0], $tableName, $primaryKey[1])->execute();
                        $time = round((microtime(true) - $start), 1);
                        $this->logs .= "done    ({$time}s)\n";
                    } catch (\Throwable $e) {

• Read up
  Read up

Similar blocks of code found in 6 locations. Consider refactoring.

Open

                try {
                    $importer->db->createCommand()->resetSequence($tableName)->execute();
                    $time = round((microtime(true) - $start), 1);
                    $this->logs .= "done    ({$time}s)\n";
                } catch (\Throwable $e) {

• Read up
  Read up

Similar blocks of code found in 6 locations. Consider refactoring.

Open

                        try {
                            if (isset($dbPrimaryKeys[$primaryKey[0]])) {
                                $dbCommand->dropPrimaryKey($primaryKey[0], $tableName)->execute();
                            } elseif ($dbPrimaryKeys) {
                                $dbCommand->dropPrimaryKey(key($dbPrimaryKeys), $tableName)->execute();

• Read up
  Read up

Similar blocks of code found in 6 locations. Consider refactoring.

Open

            try {
                $importer->db->createCommand()->addForeignKey($key[0], $key[1], $key[2], $key[3], $key[4], $key[5] ?? null, $key[6] ?? null)->execute();
                $time = round((microtime(true) - $start), 1);
                $this->logs .= "done    ({$time}s)\n";
            } catch (\Throwable $e) {

• Read up
  Read up

Avoid variables with short names like $db. Configured minimum length is 3.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $db. Configured minimum length is 3.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $db. Configured minimum length is 3.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $db. Configured minimum length is 3.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $db. Configured minimum length is 3.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $db. Configured minimum length is 3.

Open

        $db = \App\Db::getInstance();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Spaces must be used to indent lines; tabs are not allowed

Open

     * End of line character.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @var string|float

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Array with objects to import.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                require $fileinfo->getPath() . \DIRECTORY_SEPARATOR . $fileinfo->getFilename();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @var bool

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->addTables($importer);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public function updateScheme()

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($this->importers as &$importer) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    $start = microtime(true);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 131 characters

Open

                            throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    private $importers = [];

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    $instance->scheme();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 156 characters

Open

            file_put_contents('cache/logs/Importer.log', $this->logs . '-------------  ' . date('Y-m-d H:i:s') . " ($time min)   -------------\n", LOCK_EX);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($this->importers as &$importer) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($this->importers as &$importer) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @param Base $importer

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($importer->tables as $tableName => $table) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $importer->db->createCommand()->createTable($tableName, $this->getColumns($importer, $table), $this->getOptions($importer, $table))->execute();

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 161 characters

Open

                        $importer->db->createCommand()->createIndex($index[0], $tableName, $index[1], (isset($index[2]) && $index[2]) ? true : false)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 131 characters

Open

                            throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    $this->logs .= "  > add primary key: {$primaryKey[0]} ... ";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Path to the directory with files to import.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public $dieOnError = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Import database structure.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->addForeignKey($importer);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        $this->logs .= "done    ({$time}s)\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    private $startTime;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->logs = '-------------   ' . date('Y-m-d H:i:s') . "   -------------\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $dir = new \DirectoryIterator($path ?: $this->path);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($this->importers as &$importer) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Creating tables.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        $time = round((microtime(true) - $start), 1);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        $this->logs .= "    ({$time}s) | Error(2) [{$e->getMessage()}] in  \n{$e->getTraceAsString()} !!!\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        $this->logs .= "# end add tables ($time min)\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $className = 'Importers\\' . $fileinfo->getBasename('.php');

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $this->importers[] = $instance;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public function importData()

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public function postUpdate()

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->logs .= "  > add table: $tableName ... ";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $time = round((microtime(true) - $start), 1);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    $this->logs .= "  > create index: {$index[0]} ... ";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @var string

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($dir as $fileinfo) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            echo $this->logs . '---------  ' . date('Y-m-d H:i:s') . "  ($time min)  -------------\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public function importScheme()

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Import database rows.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        foreach ($this->importers as &$importer) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $start = microtime(true);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 159 characters

Open

                $importer->db->createCommand()->createTable($tableName, $this->getColumns($importer, $table), $this->getOptions($importer, $table))->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            if (isset($table['primaryKeys'])) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public $logs;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Start time.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    $instance->data();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @param bool $show

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public function logs($show = true)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Post Process action.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        $importer->db->createCommand()->createIndex($index[0], $tableName, $index[1], (isset($index[2]) && $index[2]) ? true : false)->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        $importer->db->createCommand()->addPrimaryKey($primaryKey[0], $tableName, $primaryKey[1])->execute();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    } catch (\Throwable $e) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    public $redundantTables = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @var Base[]

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Load all files for import.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            if ('dir' !== $fileinfo->getType() && 'php' === $fileinfo->getExtension()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                if (method_exists($instance, 'data')) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            $this->drop($importer);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                    throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @var string

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 125 characters

Open

                        $this->logs .= "    ({$time}s) | Error(2) [{$e->getMessage()}] in  \n{$e->getTraceAsString()} !!!\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                            throw new \App\Exceptions\AppException('Importer error: ' . $e->getMessage(), (int) $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * @param bool|string $path

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

     * Update db scheme.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

            try {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $time = round((microtime(true) - $start), 1);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                $this->logs .= "    ({$time}s) | Error(1) [{$e->getMessage()}] in  \n{$e->getTraceAsString()} !!!\n";

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                foreach ($indexes as $index) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used to indent lines; tabs are not allowed

Open

                        $time = round((microtime(true) - $start), 1);