Staudenmeir\LaravelCte\Query\Grammars\Traits\CompilesFirebirdExpressions

Complexity

Total Complexity

1

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	compileInsertUsing()	0	5	1

<?php

namespace Staudenmeir\LaravelCte\Query\Grammars\Traits;

use Illuminate\Database\Query\Builder;

trait CompilesFirebirdExpressions
{
    use CompilesExpressions;

    /**
     * Compile an insert statement using a subquery into SQL.
     *
     * @param \Illuminate\Database\Query\Builder $query
     * @param array $columns
     * @param string $sql
     * @return string
     */
    public function compileInsertUsing(Builder $query, array $columns, string $sql)
    {
        $insert = "insert into {$this->wrapTable($query->from)} ({$this->columnize($columns)}) ";

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

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

        return "$insert{$this->compileExpressions($query, $query->expressions)} $sql";

The property expressions does not seem to exist on Illuminate\Database\Query\Builder.

    }
}