Staudenmeir\LaravelAdjacencyList\Eloquent\Traits\HasRecursiveRelationshipHelpers

Complexity

Total Complexity

6

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	isParentOf()	0	3	1
A	isChildOf()	0	3	2
A	getDepthRelatedTo()	0	12	3

<?php

namespace Staudenmeir\LaravelAdjacencyList\Eloquent\Traits;

use Illuminate\Database\Eloquent\Model;

trait HasRecursiveRelationshipHelpers
{
    /**
     * Determine if the model is a child of the given model.
     *
     * @param \Illuminate\Database\Eloquent\Model $model
     * @return bool
     */
    public function isChildOf(Model $model): bool
    {
        return $this->parent ? $this->parent->is($model) : false;
    }

    /**
     * Determine if the model is the parent of the given model.
     *
     * @param \Illuminate\Database\Eloquent\Model $model
     * @return bool
     */
    public function isParentOf(Model $model): bool
    {
        return $this->children->contains($model);
    }

    /**
     * Get the depth of the model related to the given model.
     *
     * @param \Illuminate\Database\Eloquent\Model $model
     * @return int|null
     */
    public function getDepthRelatedTo(Model $model): ?int
    {
        $thisModel = $this->bloodline->find($this);
        $relatedModel = $this->bloodline->find($model);

        if ($thisModel && $relatedModel) {
            $depthName = $this->getDepthName();

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

            return $thisModel->$depthName - $relatedModel->$depthName;
        }

        return null;
    }
}