Umbrellio\LTree\Traits\HasTreeRelationships::belongsToTree()

Complexity

Conditions	4
Paths	3

Size

Total Lines	24
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function belongsToTree(
        string $related,
        string $throwRelation,
        ?string $foreignKey = null,
        $ownerKey = null
    ) {
        $instance = $this->newRelatedInstance($related);

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

        if (!$instance instanceof LTreeModelInterface) {
            throw new InvalidTraitInjectionClass(sprintf(
                'A class using this trait must implement an interface %s',
                LTreeModelInterface::class
            ));
        }

        if ($foreignKey === null) {
            $foreignKey = $this
                ->{$throwRelation}()
                ->getForeignKeyName();
        }

        $ownerKey = $ownerKey ?: $instance->getKeyName();

$ownerKey is of type null, thus it always evaluated to false.

        return new BelongsToTree($instance->newQuery(), $this, $throwRelation, $foreignKey, $ownerKey);

The method newQuery() does not exist on Umbrellio\LTree\Interfaces\LTreeModelInterface.

$this of type Umbrellio\LTree\Traits\HasTreeRelationships is incompatible with the type Illuminate\Database\Eloquent\Model expected by parameter $child of Umbrellio\LTree\Relations\BelongsToTree::__construct().

    }