Ronanchilvers\Orm\Features\Relationship\BelongsTo::belongsTo()

Complexity

Conditions	3
Paths	3

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function belongsTo(string $modelClass, string $attribute = null, string $foreignAttribute = 'id')
    {
        if (!class_exists($modelClass)) {
            throw new RuntimeException('Invalid model class in belongs_to definition');
        }
        if (is_null($attribute)) {
            $reflection = new ReflectionClass($modelClass);
            $attribute = Str::snake($reflection->getShortName());
            $attribute = strtolower($attribute);
            $attribute = static::prefix($attribute);
        }
        $foreignAttribute = $modelClass::prefix($foreignAttribute);
        $finder = Orm::finder($modelClass);

        return $finder->select()
                        ->where($foreignAttribute, $this->getAttribute($attribute))

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

                        ->one();
    }