Rougin\Describe\Driver\SQLiteDriver

Complexity

Total Complexity

16

Size/Duplication

Total Lines	156
Duplicated Lines	12.18 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

87.1%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getTable()	19	19	3
A	setColumn()	0	14	1
A	showTables()	0	17	3
A	setForeignColumn()	0	16	3
A	prepareForeignColumns()	0	13	2
A	setProperties()	0	11	3

<?php

namespace Rougin\Describe\Driver;

use Rougin\Describe\Column;

/**
 * SQLite Driver
 *
 * A database driver extension for SQLite.
 *
 * @package  Describe
 * @category Driver
 * @author   Rougin Royce Gutib <rougingutib@gmail.com>
 */
class SQLiteDriver implements DriverInterface
{
    /**
     * @var array
     */
    protected $columns = [];

    /**
     * @var \PDO
     */
    protected $pdo;

    /**
     * @param PDO $pdo
     */
    public function __construct(\PDO $pdo)
    {
        $this->pdo = $pdo;
    }

    /**
     * Returns the result.
     *
     * @param  string $tableName
     * @return array
     */
    public function getTable($tableName)

This method seems to be duplicated in your project.

    {
        $this->columns = [];

        try {
            $query = $this->pdo->prepare('PRAGMA table_info("' . $tableName . '");');

            $query->execute();
            $query->setFetchMode(\PDO::FETCH_OBJ);
        } catch (\PDOException $e) {
            // Table not found
        }

        while ($row = $query->fetch()) {
            $this->setColumn($tableName, $row);
        }

        return $this->columns;
    }

    /**
     * Prepares the defined columns.
     *
     * @param  string $tableName
     * @param  mixed  $row
     * @return void
     */
    protected function setColumn($tableName, $row)
    {
        $column = new Column;

        $this->setProperties($row, $column);

        $column->setDefaultValue($row->dflt_value);
        $column->setField($row->name);
        $column->setDataType(strtolower($row->type));

        $this->setForeignColumn($tableName, $row, $column);

        array_push($this->columns, $column);
    }

    /**
     * Shows the list of tables.
     *
     * @return array
     */
    public function showTables()
    {
        $tables = [];

        $query = $this->pdo->prepare('SELECT name FROM sqlite_master WHERE type = "table";');

        $query->execute();
        $query->setFetchMode(\PDO::FETCH_OBJ);

        while ($row = $query->fetch()) {
            if ($row->name != 'sqlite_sequence') {
                array_push($tables, $row->name);
            }
        }

        return $tables;
    }

    /**
     * Sets the properties of the specified column if it does exists.
     *
     * @param  string                  $tableName
     * @param  mixed                   $row
     * @param  \Rougin\Describe\Column &$column
     * @return void
     */
    protected function setForeignColumn($tableName, $row, Column &$column)

The parameter $row is not used and could be removed.

    {
        $query = $this->pdo->prepare('PRAGMA foreign_key_list("' . $tableName . '");');

        $query->execute();
        $query->setFetchMode(\PDO::FETCH_OBJ);

        while ($row = $query->fetch()) {
            if ($column->getField() == $row->from) {
                $column->setForeign(true);

                $column->setReferencedField($row->to);
                $column->setReferencedTable($row->table);
            }
        }
    }

    /**
     * Prepares the query for getting the foreign columns.
     *
     * @param  array  $columns
     * @param  string $tableName
     * @return array
     */
    protected function prepareForeignColumns(array $columns, $tableName)
    {
        $query = $this->pdo->prepare('PRAGMA foreign_key_list("' . $tableName . '");');

        $query->execute();
        $query->setFetchMode(\PDO::FETCH_OBJ);

        while ($row = $query->fetch()) {
            $this->setForeignColumn($columns, $row);

The call to setForeignColumn() misses a required argument $column.

$columns is of type array, but the function expects a string.

        }

        return $columns;
    }

    /**
     * Sets the properties of the specified column.
     *
     * @param  mixed                   $row
     * @param  \Rougin\Describe\Column &$column
     * @return void
     */
    protected function setProperties($row, Column &$column)
    {
        if (! $row->notnull) {
            $column->setNull(true);
        }

        if ($row->pk) {
            $column->setPrimary(true);
            $column->setAutoIncrement(true);
        }
    }
}