Yajra\Oci8\Schema\OracleBuilder

Complexity

Total Complexity

10

Size/Duplication

Total Lines	139
Duplicated Lines	23.02 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	6
Bugs	1	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	create()	14	14	1
A	table()	16	16	3
A	createBlueprint()	0	7	1
A	drop()	0	5	1
A	dropIfExists()	0	5	1
A	hasTable()	0	9	1
A	getColumnListing()	0	9	1

<?php

namespace Yajra\Oci8\Schema;

use Closure;
use Illuminate\Database\Connection;
use Illuminate\Database\Schema\Builder;

class OracleBuilder extends Builder
{
    /**
     * @var \Yajra\Oci8\Schema\OracleAutoIncrementHelper
     */
    public $helper;

    /**
     * @var \Yajra\Oci8\Schema\Comment
     */
    public $comment;

    /**
     * @param Connection $connection
     */
    public function __construct(Connection $connection)
    {
        $this->connection = $connection;
        $this->grammar    = $connection->getSchemaGrammar();
        $this->helper     = new OracleAutoIncrementHelper($connection);
        $this->comment    = new Comment($connection);
    }

    /**
     * Create a new table on the schema.
     *
     * @param  string $table
     * @param  Closure $callback
     * @return \Illuminate\Database\Schema\Blueprint
     */
    public function create($table, Closure $callback)

This method seems to be duplicated in your project.

    {
        $blueprint = $this->createBlueprint($table);

        $blueprint->create();

        $callback($blueprint);

        $this->build($blueprint);

        $this->comment->setComments($blueprint);

        $this->helper->createAutoIncrementObjects($blueprint, $table);
    }

    /**
     * Changes an existing table on the schema.
     *
     * @param  string $table
     * @param  Closure $callback
     * @return \Illuminate\Database\Schema\Blueprint
     */
    public function table($table, Closure $callback)

This method seems to be duplicated in your project.

    {
        $blueprint = $this->createBlueprint($table);

        $callback($blueprint);

        foreach ($blueprint->getCommands() as $command) {
            if ($command->get('name') == 'drop') {
                $this->helper->dropAutoIncrementObjects($table);
            }
        }

        $this->build($blueprint);

        $this->comment->setComments($blueprint);
    }

    /**
     * Create a new command set with a Closure.
     *
     * @param  string $table
     * @param  Closure $callback
     * @return \Illuminate\Database\Schema\Blueprint
     */
    protected function createBlueprint($table, Closure $callback = null)
    {
        $blueprint = new OracleBlueprint($table, $callback);
        $blueprint->setTablePrefix($this->connection->getTablePrefix());

        return $blueprint;
    }

    /**
     * Drop a table from the schema.
     *
     * @param  string $table
     * @return \Illuminate\Database\Schema\Blueprint
     */
    public function drop($table)
    {
        $this->helper->dropAutoIncrementObjects($table);
        parent::drop($table);
    }

    /**
     * Indicate that the table should be dropped if it exists.
     *
     * @return \Illuminate\Support\Fluent
     */
    public function dropIfExists($table)
    {
        $this->helper->dropAutoIncrementObjects($table);
        parent::dropIfExists($table);
    }

    /**
     * Determine if the given table exists.
     *
     * @param  string $table
     * @return bool
     */
    public function hasTable($table)
    {
        $sql = $this->grammar->compileTableExists();

It seems like you code against a specific sub-type and not the parent class Illuminate\Database\Schema\Grammars\Grammar as the method compileTableExists() does only exist in the following sub-classes of Illuminate\Database\Schema\Grammars\Grammar: Illuminate\Database\Schema\Grammars\MySqlGrammar, Illuminate\Database\Schema\Grammars\PostgresGrammar, Illuminate\Database\Schema\Grammars\SQLiteGrammar, Illuminate\Database\Schema\Grammars\SqlServerGrammar, Yajra\Oci8\Schema\Grammars\OracleGrammar. Maybe you want to instanceof check for one of these explicitly?

        $database = $this->connection->getConfig('username');
        $table    = $this->connection->getTablePrefix() . $table;

        return count($this->connection->select($sql, [$database, $table])) > 0;
    }

    /**
     * Get the column listing for a given table.
     *
     * @param  string $table
     * @return array
     */
    public function getColumnListing($table)
    {
        $database = $this->connection->getConfig('username');
        $table    = $this->connection->getTablePrefix() . $table;

        $results = $this->connection->select($this->grammar->compileColumnExists($database, $table));

It seems like you code against a specific sub-type and not the parent class Illuminate\Database\Schema\Grammars\Grammar as the method compileColumnExists() does only exist in the following sub-classes of Illuminate\Database\Schema\Grammars\Grammar: Illuminate\Database\Schema\Grammars\MySqlGrammar, Illuminate\Database\Schema\Grammars\PostgresGrammar, Illuminate\Database\Schema\Grammars\SQLiteGrammar, Illuminate\Database\Schema\Grammars\SqlServerGrammar, Yajra\Oci8\Schema\Grammars\OracleGrammar. Maybe you want to instanceof check for one of these explicitly?

        return $this->connection->getPostProcessor()->processColumnListing($results);
    }
}