LaravelFreelancerNL\Aranguent\Query\Concerns\CompilesUnions::compileUnions()

Complexity

Conditions	6
Paths	24

Size

Total Lines	33
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	18
CRAP Score	6

Importance

Changes	2
Bugs	0	Features	0

<?php
    protected function compileUnions(IlluminateBuilder $query, $firstQuery = '')
    {
        $unionResultsId = 'union' . $query->getQueryId() . 'Results';
        $unionDocId = 'union' . $query->getQueryId() . 'Result';

        $query->registerTableAlias($unionResultsId, $unionDocId);

        $firstQuery = $this->wrapSubquery($firstQuery);

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

        $unions = '';
        foreach ($query->unions as $union) {
            $prefix = ($unions !== '') ? $unions : $firstQuery;
            $unions = $this->compileUnion($union, $prefix);
        }

        $aql = 'LET ' . $unionResultsId . ' = ' . $unions
            . ' FOR ' . $unionDocId . ' IN ' . $unionResultsId;

        // Union groups

        if (!empty($query->unionOrders)) {
            $aql .= ' ' . $this->compileOrders($query, $query->unionOrders, $unionResultsId);

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

        }

        if ($query->unionOffset) {
            $aql .= ' ' . $this->compileOffset($query, $query->unionOffset);

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

        }

        if ($query->unionLimit) {
            $aql .= ' ' . $this->compileLimit($query, $query->unionLimit);

The method compileLimit() does not exist on LaravelFreelancerNL\Aranguent\Query\Concerns\CompilesUnions. Did you maybe mean compileUnion()?

        }

        // Union aggregates?
        return $aql . ' RETURN ' . $unionDocId;
    }