Sofa\Eloquence\Relations\JoinerFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	make()	0	9	2

<?php

namespace Sofa\Eloquence\Relations;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Builder as EloquentBuilder;
use Illuminate\Database\Query\Builder;
use Sofa\Eloquence\Contracts\Relations\JoinerFactory as FactoryContract;

class JoinerFactory implements FactoryContract
{
    /**
     * Create new joiner instance.
     *
     * @param EloquentBuilder|Builder $query
     * @param Model $model
     * @return Joiner
     */
    public static function make($query, Model $model = null)
    {
        if ($query instanceof EloquentBuilder) {
            $model = $query->getModel();
            $query = $query->getQuery();
        }

        return new Joiner($query, $model);

It seems like $model can also be of type null or object<Illuminate\Database\Eloquent\Builder>; however, Sofa\Eloquence\Relations\Joiner::__construct() does only seem to accept object<Illuminate\Database\Eloquent\Model>, maybe add an additional type check?

    }
}