Staudenmeir\LaravelMergedRelations\Eloquent\HasMergedRelationships

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	mergedRelation()	0	3	1
A	mergedRelationWithModel()	0	9	2
A	newMergedRelation()	0	3	1

<?php

namespace Staudenmeir\LaravelMergedRelations\Eloquent;

use Illuminate\Database\Eloquent\Builder as EloquentBuilder;
use Illuminate\Database\Eloquent\Model;
use Staudenmeir\LaravelMergedRelations\Eloquent\Relations\MergedRelation;

trait HasMergedRelationships
{
    /**
     * Define a merged relationship.
     *
     * @param string $view
     * @param string|null $localKey
     * @return \Staudenmeir\LaravelMergedRelations\Eloquent\Relations\MergedRelation
     */
    public function mergedRelation($view, $localKey = null)
    {
        return $this->mergedRelationWithModel(static::class, $view, $localKey);
    }

    /**
     * Define a merged relationship with model.
     *
     * @param string $related
     * @param string $view
     * @param string|null $localKey
     * @return \Staudenmeir\LaravelMergedRelations\Eloquent\Relations\MergedRelation
     */
    public function mergedRelationWithModel($related, $view, $localKey = null)
    {
        $instance = $this->newRelatedInstance($related)->setTable($view);

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

        $query = (new Builder($instance->getConnection()->query()))->setModel($instance);

        $localKey = $localKey ?: $this->getKeyName();

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

        return $this->newMergedRelation($query, $this, $view.'.laravel_foreign_key', $localKey);

$this of type Staudenmeir\LaravelMergedRelations\Eloquent\HasMergedRelationships is incompatible with the type Illuminate\Database\Eloquent\Model expected by parameter $parent of Staudenmeir\LaravelMergedRelations\Eloquent\HasMergedRelationships::newMergedRelation().

    }

    /**
     * Instantiate a new MergedRelation relationship.
     *
     * @param \Illuminate\Database\Eloquent\Builder $query
     * @param \Illuminate\Database\Eloquent\Model $parent
     * @param string $foreignKey
     * @param string $localKey
     * @return \Staudenmeir\LaravelMergedRelations\Eloquent\Relations\MergedRelation
     */
    protected function newMergedRelation(EloquentBuilder $query, Model $parent, $foreignKey, $localKey)
    {
        return new MergedRelation($query, $parent, $foreignKey, $localKey);
    }
}