LaravelFreelancerNL\Aranguent\Eloquent\Concerns\QueriesAranguentRelationships

Complexity

Total Complexity

17

Size/Duplication

Total Lines	151
Duplicated Lines	0 %

Test Coverage

Coverage

87.5%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	mergeConstraintsFrom()	0	19	2
A	addWhereCountQuery()	0	12	2
C	withAggregate()	0	89	13

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Eloquent\Concerns;

use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Builder as IlluminateEloquentBuilder;
use Illuminate\Database\Query\Builder as IlluminateQueryBuilder;
use Illuminate\Database\Query\Expression;
use Illuminate\Support\Str;

trait QueriesAranguentRelationships
{
    /**
     * Add a sub-query count clause to this query.
     *
     * @param  \Illuminate\Database\Query\Builder  $query
     * @param  string  $operator
     * @param  int  $count
     * @param  string  $boolean
     * @return $this
     */
    protected function addWhereCountQuery(IlluminateQueryBuilder $query, $operator = '>=', $count = 1, $boolean = 'and')
    {

        $this->getQuery()->exchangeTableAliases($query);

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

        $this->getQuery()->importBindings($query);

        return $this->where(

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

            new Expression('LENGTH((' . $query->toSql() . '))'),
            $operator,
            is_numeric($count) ? new Expression($count) : $count,

The condition is_numeric($count) is always true.

            $boolean
        );
    }

    /**
     * Merge the where constraints from another query to the current query.
     *
     * @param IlluminateEloquentBuilder $from
     * @return IlluminateEloquentBuilder|static
     */
    public function mergeConstraintsFrom(Builder $from)
    {
        $whereBindings = $this->getQuery()->getBindings() ?? [];

        $wheres = $from->getQuery()->from !== $this->getQuery()->from
            ? $this->requalifyWhereTables(

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

                $from->getQuery()->wheres,
                $from->getQuery()->grammar->getValue($from->getQuery()->from),
                $this->getModel()->getTable()

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

            ) : $from->getQuery()->wheres;

        // Here we have some other query that we want to merge the where constraints from. We will
        // copy over any where constraints on the query as well as remove any global scopes the
        // query might have removed. Then we will return ourselves with the finished merging.
        return $this->withoutGlobalScopes(

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

            $from->removedScopes()
        )->mergeWheres(
            $wheres,
            $whereBindings
        );
    }

    /**
     * Add subselect queries to include an aggregate value for a relationship.
     *
     * @param  mixed  $relations
     * @param  string  $column
     * @param  string  $function
     * @return $this
     */
    public function withAggregate($relations, $column, $function = null)
    {
        if (empty($relations)) {
            return $this;
        }

        if (is_null($this->query->columns)) {
            $this->query->select([$this->query->from . '.*']);
        }

        $relations = is_array($relations) ? $relations : [$relations];

        foreach ($this->parseWithRelations($relations) as $name => $constraints) {

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

            // First we will determine if the name has been aliased using an "as" clause on the name
            // and if it has we will extract the actual relationship name and the desired name of
            // the resulting column. This allows multiple aggregates on the same relationships.
            $segments = explode(' ', $name);

            unset($alias);

            if (count($segments) === 3 && Str::lower($segments[1]) === 'as') {
                [$name, $alias] = [$segments[0], $segments[2]];
            }

            $relation = $this->getRelationWithoutConstraints($name);

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

            if ($function) {
                $hashedColumn = $this->getRelationHashedColumn($column, $relation);

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

                $wrappedColumn = $this->getQuery()->getGrammar()->wrap(
                    $column === '*' ? $column : $relation->getRelated()->qualifyColumn($hashedColumn)
                );

                $expression = $function === 'exists' ? $wrappedColumn : sprintf('%s(%s)', $function, $wrappedColumn);
            } else {
                $expression = $column;
            }

            // Here, we will grab the relationship sub-query and prepare to add it to the main query
            // as a sub-select. First, we'll get the "has" query and use that to get the relation
            // sub-query. We'll format this relationship name and append this column if needed.
            $query = $relation->getRelationExistenceQuery(
                $relation->getRelated()->newQuery(),
                $this,
                new Expression($expression)
            )->setBindings([], 'select');

            $query->callScope($constraints);

            $query = $query->mergeConstraintsFrom($relation->getQuery())->toBase();

            // If the query contains certain elements like orderings / more than one column selected
            // then we will remove those elements from the query so that it will execute properly
            // when given to the database. Otherwise, we may receive SQL errors or poor syntax.
            $query->orders = null;
            $query->setBindings([], 'order');

            if (count($query->columns) > 1) {
                $query->columns = [$query->columns[0]];
                $query->bindings['select'] = [];
            }

            // Finally, we will make the proper column alias to the query and run this sub-select on
            // the query builder. Then, we will return the builder instance back to the developer
            // for further constraint chaining that needs to take place on the query as needed.
            $alias ??= Str::snake(

The variable $alias seems to be defined later in this foreach loop on line 96. Are you sure it is defined here?

                preg_replace('/[^[:alnum:][:space:]_]/u', '', "$name $function $column")
            );

            if ($function === 'exists') {
                //FIXME: test for results from the subquery -> whereHas?
                $this->selectRaw(

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

                    sprintf('exists(%s) as %s', $query->toSql(), $this->getQuery()->grammar->wrap($alias)),
                    $query->getBindings()
                )->withCasts([$alias => 'bool']);
            } else {
                if ($function === null) {
                    $query->limit(1);
                }

                $this->getQuery()->exchangeTableAliases($query);
                $this->getQuery()->importBindings($query);

                $this->set($alias, new Expression(strtoupper($function) . '((' . $query->toSql() . '))'), 'postIterationVariables');

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

                $this->addSelect($alias);

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

            }
        }

        return $this;
    }
}