Staudenmeir\EloquentHasManyDeep\Eloquent\Relations\Traits\HasExistenceQueries::getRelationExistenceQuery()

Complexity

Conditions	9
Paths	16

Size

Total Lines	45
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	26
CRAP Score	9

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getRelationExistenceQuery(Builder $query, Builder $parentQuery, $columns = ['*'])
    {
        foreach ($this->throughParents as $throughParent) {
            if ($throughParent->getTable() === $parentQuery->getQuery()->from) {
                if (!in_array(HasTableAlias::class, class_uses_recursive($throughParent))) {
                    $traitClass = HasTableAlias::class;
                    $parentClass = get_class($throughParent);

                    throw new Exception(
                        <<<EOT
This query requires an additional trait. Please add the $traitClass trait to $parentClass.
See https://github.com/staudenmeir/eloquent-has-many-deep/issues/137 for details.
EOT
                    );
                }

                $table = $throughParent->getTable() . ' as ' . $this->getRelationCountHash();

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

                $throughParent->setTable($table);

                break;
            }
        }

        if ($this->firstKey instanceof Closure || $this->localKey instanceof Closure) {
            $this->performJoin($query);

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

            $closureKey = $this->firstKey instanceof Closure ? $this->firstKey : $this->localKey;

            $closureKey($query, $parentQuery);

            return $query->select($columns);

The expression return $query->select($columns) also could return the type Illuminate\Database\Query\Builder which is incompatible with the documented return type Illuminate\Database\Eloquent\Builder.

        }

        $query = parent::getRelationExistenceQuery($query, $parentQuery, $columns);

        if (is_array($this->foreignKeys[0])) {
            $column = $this->throughParent->qualifyColumn($this->foreignKeys[0][0]);

            $query->where($column, '=', $this->farParent->getMorphClass());
        } elseif ($this->hasLeadingCompositeKey()) {

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

            $this->getRelationExistenceQueryWithCompositeKey($query);

The method getRelationExistenceQueryWithCompositeKey() does not exist on Staudenmeir\EloquentHasManyDeep\Eloquent\Relations\Traits\HasExistenceQueries. Did you maybe mean getRelationExistenceQuery()?

        }

        return $query;
    }