Ianrizky\Illuminate\Database\Concerns\UseCustomQueryBuilder::query()

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function query(): Builder
    {
        $customQueryBuilderClass = Config::get('database.custom_query_builder_class');

        if (is_null($customQueryBuilderClass)) {
            return new Builder(
                $this, $this->getQueryGrammar(), $this->getPostProcessor()

It seems like getPostProcessor() must be provided by classes using this trait. How about adding it as abstract method to this trait?

$this of type Ianrizky\Illuminate\Database\Concerns\UseCustomQueryBuilder is incompatible with the type Illuminate\Database\ConnectionInterface expected by parameter $connection of Illuminate\Database\Query\Builder::__construct().

It seems like getQueryGrammar() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            );
        }

        if (is_subclass_of($customQueryBuilderClass, Builder::class)) {
            return new $customQueryBuilderClass(
                $this, $this->getQueryGrammar(), $this->getPostProcessor()
            );
        }

        throw new InvalidArgumentException(sprintf(
            "The custom query builder class of %s must extend the base %s laravel class",
            $customQueryBuilderClass, Builder::class
        ));