eloquentFilter\QueryFilter\Detection\DetectionFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	buildDetections()	0	8	1
A	__construct()	0	3	1

<?php

namespace eloquentFilter\QueryFilter\Detection;

use eloquentFilter\QueryFilter\Detection\Contract\DetectorFactoryContract;
use Illuminate\Database\Eloquent\Model;

/**
 * Class DetectionFactory.
 */
class DetectionFactory implements DetectorFactoryContract
{
    /**
     * @var array
     */
    public array $detections;

    /**
     * DetectionFactory constructor.
     *
     * @param array $detections
     */
    public function __construct(array $detections)
    {
        $this->detections = $detections;
    }

    /**
     * @param $field
     * @param $params
     * @param Model|null $model
     *
     * @return mixed|string|null
     * @throws \ReflectionException
     *
     */
    public function buildDetections($field, $params, $model = null): ?string
    {
        $detect = app(DetectorCondition::class, ['detector' => $this->detections]);

        /** @see DetectorCondition::detect() */
        $method = $detect->detect($field, $params, $model);

It seems like $model can also be of type Illuminate\Database\Eloquent\Model; however, parameter $model of eloquentFilter\QueryFilter\Detection\DetectorCondition::detect() does only seem to accept null, maybe add an additional type check?

        return $method;
    }
}