BelongsToMany::getRelationExistenceQueryForSelfJoin() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/186-self-join..." - LaravelFreelancerNL/laravel-arangodb - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — 186-self-joins-fail-in-relatio... ( ba3e0e...5bcaaa )

by Bas

created 2025-03-30 12:03 UTC

getRelationExistenceQueryForSelfJoin() A

↳ Parent: BelongsToMany

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	4
c	0
b	0
f	0
nc	1
nop	3
dl	0
loc	11
rs	10

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Eloquent\Relations;

use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Relations\BelongsToMany as IlluminateBelongsToMany;
use LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\InteractsWithPivotTable;
use LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\IsAranguentRelation;

class BelongsToMany extends IlluminateBelongsToMany
{
    use IsAranguentRelation;
    use InteractsWithPivotTable;

    /**
     * Add the constraints for a relationship query on the same table.
     *
     * @param  \Illuminate\Database\Eloquent\Builder<TRelatedModel>  $query
     * @param  \Illuminate\Database\Eloquent\Builder<TDeclaringModel>  $parentQuery
     * @param  array|mixed  $columns
     * @return \Illuminate\Database\Eloquent\Builder<TRelatedModel>
     */
    public function getRelationExistenceQueryForSelfJoin(Builder $query, Builder $parentQuery, $columns = ['*'])
    {
        $query->select($columns);

        $query->from($this->related->getTable().' as '.$hash = $this->getRelationCountHash());


//        $this->related->setTable($hash);

        $this->performJoin($query);

        return parent::getRelationExistenceQuery($query, $parentQuery, $columns);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace LaravelFreelancerNL\Aranguent\Eloquent\Relations;
6
7			use Illuminate\Database\Eloquent\Builder;
8			use Illuminate\Database\Eloquent\Relations\BelongsToMany as IlluminateBelongsToMany;
9			use LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\InteractsWithPivotTable;
10			use LaravelFreelancerNL\Aranguent\Eloquent\Relations\Concerns\IsAranguentRelation;
11
12			class BelongsToMany extends IlluminateBelongsToMany
13			{
14			use IsAranguentRelation;
15			use InteractsWithPivotTable;
16
17			/**
18			* Add the constraints for a relationship query on the same table.
19			*
20			* @param \Illuminate\Database\Eloquent\Builder<TRelatedModel> $query
21			* @param \Illuminate\Database\Eloquent\Builder<TDeclaringModel> $parentQuery
22			* @param array\|mixed $columns
23			* @return \Illuminate\Database\Eloquent\Builder<TRelatedModel>
24			*/
25			public function getRelationExistenceQueryForSelfJoin(Builder $query, Builder $parentQuery, $columns = ['*'])
26			{
27			$query->select($columns);
28
29			$query->from($this->related->getTable().' as '.$hash = $this->getRelationCountHash());
			0 ignored issues – show Unused Code introduced 2025-03-30 12:11 UTC by Report Bug Copy Issue Report The assignment to `$hash` is dead and can be removed. Loading history...
30
31			// $this->related->setTable($hash);
32
33			$this->performJoin($query);
34
35			return parent::getRelationExistenceQuery($query, $parentQuery, $columns);
36			}
37			}
38

LaravelFreelancerNL / laravel-arangodb

Push — 186-self-joins-fail-in-relatio... ( ba3e0e...5bcaaa )

getRelationExistenceQueryForSelfJoin() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like