JsonFieldCast\Json\HasMorphClassesAttributes

Complexity

Total Complexity

10

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	toMorph()	0	6	3
B	fromMorph()	0	19	7

<?php

namespace JsonFieldCast\Json;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\Relation;

trait HasMorphClassesAttributes
{
    public function toMorph(string $key, Model $value, string $idKeyName = 'id', string $classKeyName = 'class'): static
    {
        $this->setAttribute($key ? "{$key}.{$idKeyName}" : $idKeyName, $value->getKey());

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

        $this->setAttribute($key ? "{$key}.{$classKeyName}" : $classKeyName, $value->getMorphClass());

        return $this;
    }

    public function fromMorph(string $key = '', ?Model $default = null, string $idKeyName = 'id', string $classKeyName = 'class'): ?Model
    {
        $id    = $this->getAttribute($key ? "{$key}.{$idKeyName}" : $idKeyName);

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

        $class = $this->getAttribute($key ? "{$key}.{$classKeyName}" : $classKeyName);
        if (!$id || !$class) {
            return $default;
        }

        $class = Relation::getMorphedModel($class) ?? $class;
        if (!is_a($class, Model::class, true)) {
            return $default;
        }

        $model = $class::find($id);
        if (!$model) {
            return $default;
        }

        return $model;
    }
}