Siblings::match() - Code Metrics - Inspection of "Add 1.0" - staudenmeir/eloquent-eager-limit-x-laravel-adjacency-list - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 31aa9e...537bf7 )

by Jonas

created 2022-03-20 14:08 UTC

Siblings::match() A

↳ Parent: Siblings

Complexity

Conditions	5
Paths	3

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	10
c	1
b	0
f	0
dl	0
loc	18
rs	9.6111
cc	5
nc	3
nop	3

<?php

namespace Staudenmeir\EloquentEagerLimitXLaravelAdjacencyList\Eloquent\Relations;

use Illuminate\Database\Eloquent\Collection;
use Staudenmeir\EloquentEagerLimit\Relations\HasLimit;
use Staudenmeir\LaravelAdjacencyList\Eloquent\Relations\Siblings as Base;

class Siblings extends Base
{
    use HasLimit {

        limit as baseLimit;
    }

    /**
     * Match the eagerly loaded results to their parents.
     *
     * @param array $models
     * @param \Illuminate\Database\Eloquent\Collection $results
     * @param string $relation
     * @return array
     */
    public function match(array $models, Collection $results, $relation)
    {
        parent::match($models, $results, $relation);

        if (!$this->parent->exists && !$this->andSelf) {
            $limit = $this->query->getQuery()->groupLimit;


            if ($limit) {
                foreach ($models as $model) {
                    $model->setRelation(
                        $relation,
                        $model->getRelation($relation)->take($limit['value'] - 1)
                    );
                }
            }
        }

        return $models;
    }

    /**
     * Set the "limit" value of the query.
     *
     * @param int $value
     * @return $this
     */
    public function limit($value)
    {
        if (!$this->parent->exists && !$this->andSelf) {
            $value++;
        }

        return $this->baseLimit($value);
    }
}


1			<?php
2
3			namespace Staudenmeir\EloquentEagerLimitXLaravelAdjacencyList\Eloquent\Relations;
4
5			use Illuminate\Database\Eloquent\Collection;
6			use Staudenmeir\EloquentEagerLimit\Relations\HasLimit;
7			use Staudenmeir\LaravelAdjacencyList\Eloquent\Relations\Siblings as Base;
8
9			class Siblings extends Base
10			{
11			use HasLimit {
			0 ignored issues – show Bug introduced 2022-03-20 14:19 UTC by Report Bug Copy Issue Report The trait `Staudenmeir\EloquentEagerLimit\Relations\HasLimit` requires the property `$exists` which is not provided by `Staudenmeir\EloquentEage...uent\Relations\Siblings`. Loading history...
12			limit as baseLimit;
13			}
14
15			/**
16			* Match the eagerly loaded results to their parents.
17			*
18			* @param array $models
19			* @param \Illuminate\Database\Eloquent\Collection $results
20			* @param string $relation
21			* @return array
22			*/
23			public function match(array $models, Collection $results, $relation)
24			{
25			parent::match($models, $results, $relation);
26
27			if (!$this->parent->exists && !$this->andSelf) {
28			$limit = $this->query->getQuery()->groupLimit;
			0 ignored issues – show Bug introduced 2022-03-20 14:19 UTC by Report Bug Copy Issue Report The property `groupLimit` does not seem to exist on `Illuminate\Database\Query\Builder`. Loading history...
29
30			if ($limit) {
31			foreach ($models as $model) {
32			$model->setRelation(
33			$relation,
34			$model->getRelation($relation)->take($limit['value'] - 1)
35			);
36			}
37			}
38			}
39
40			return $models;
41			}
42
43			/**
44			* Set the "limit" value of the query.
45			*
46			* @param int $value
47			* @return $this
48			*/
49			public function limit($value)
50			{
51			if (!$this->parent->exists && !$this->andSelf) {
52			$value++;
53			}
54
55			return $this->baseLimit($value);
56			}
57			}
58

staudenmeir / eloquent-eager-limit-x-laravel-adjacency-list

Push — main ( 31aa9e...537bf7 )

Siblings::match() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like