Ronanchilvers\Orm\Features\Relationship\HasOne

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	hasOne()	0	17	3

<?php

namespace Ronanchilvers\Orm\Features\Relationship;

use ReflectionClass;
use Ronanchilvers\Orm\Orm;
use Ronanchilvers\Utility\Str;
use RuntimeException;

/**
 * Has One relationship - one to one, implies possesion
 *
 * Eg: A user has one account, a car has one engine
 *
 * @author Ronan Chilvers <ronan@d3r.com>
 */
trait HasOne
{
    /**
     * Get the data for a 'belongs to' relationship
     *
     * @param string $modelClass The model class that we 'belong to'
     * @param string $attribute The local data attribute that identifies the 'belongs to' key
     * @param string $foreignAttribute The foreign data attribute that the local field references
     * @return \Ronanchilvers\Orm\Model
     * @author Ronan Chilvers <ronan@d3r.com>
     */
    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();
    }
}