Staudenmeir\LaravelAdjacencyList\Eloquent\HasRecursiveRelationships

Complexity

Total Complexity

6

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	10
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	isChildOf()	0	3	2
A	depthRelatedTo()	0	10	3
A	isParentOf()	0	3	1

<?php

namespace Staudenmeir\LaravelAdjacencyList\Eloquent;

use Illuminate\Database\Eloquent\Model;
use Staudenmeir\LaravelAdjacencyList\Eloquent\Traits\HasAdjacencyList;
use Staudenmeir\LaravelCte\Eloquent\QueriesExpressions;

trait HasRecursiveRelationships
{
    use HasAdjacencyList;
    use QueriesExpressions;

    public function isParentOf(Model $model)
    {
        return $this->children->contains($model);
    }

    public function isChildOf(Model $model)
    {
        return $this->parent ? $this->parent->is($model) : false;
    }

    public function depthRelatedTo(Model $model)
    {
        $currentModel = $this->bloodline->firstWhere('id', $this->getKey());

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

        $relatedModel = $this->bloodline->firstWhere('id', $model->getKey());

        if ($currentModel && $relatedModel) {
            return $currentModel->depth - $relatedModel->depth;
        }

        return null;
    }
}