Ronanchilvers\Orm\Features\Relationship\HasOne::hasOne()

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 hasOne(string $modelClass, string $foreignAttribute = null, string $attribute = 'id')
    {
        if (!class_exists($modelClass)) {
            throw new RuntimeException('Invalid model class in belongs_to definition');
        }
        if (is_null($foreignAttribute)) {
            $reflection = new ReflectionClass(get_called_class());
            $foreignAttribute = Str::snake($reflection->getShortName());
            $foreignAttribute = strtolower($foreignAttribute);
            $foreignAttribute = $modelClass::prefix($foreignAttribute);
        }
        $attribute = static::prefix($attribute);
        $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();
    }