Staudenmeir\EloquentEagerLimitXLaravelAdjacencyList\Eloquent\Relations\Ancestors::limit()

Complexity

Conditions	2
Paths	2

Size

Total Lines	34
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function limit($value)
    {
        if ($this->parent->exists) {
            $this->query->limit($value);
        } else {
            $grammar = $this->getEagerLimitGrammar();

			// TODO
			$sql = $grammar->compileFirstPathSegment(
				$this->related->qualifyColumn(
					$this->related->getPathName()

It seems like $this->related->getPathName() can also be of type Illuminate\Database\Eloquent\Builder; however, parameter $column of Illuminate\Database\Eloquent\Model::qualifyColumn() does only seem to accept string, maybe add an additional type check?

				)
			);

            $column = new Expression($sql);

            // TODO
            //		    if ($grammar instanceof MySqlGrammar && $grammar->useLegacyGroupLimit($this->query->getQuery())) {
            //			    $column = 'laravel_through_key';
            //		    }

            $this->query->groupLimit($value, $column);

            $this->query->getQuery()->addBinding(
                array_fill(
					0,
					substr_count($sql, '?'),
					$this->related->getPathSeparator()
                ),
                'groupBy'
            );
        }

        return $this;
    }