Terranet\Administrator\Field\Detectors\EnumDetector

Complexity

Total Complexity

6

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	authorize()	0	3	3
A	detect()	0	9	2
A	enumValues()	0	3	1

<?php

namespace Terranet\Administrator\Field\Detectors;

use function admin\db\connection;
use function admin\db\enum_values;
use function admin\db\translated_values;
use Doctrine\DBAL\Schema\Column;
use Doctrine\DBAL\Types\StringType;
use Illuminate\Database\Eloquent\Model;
use Terranet\Administrator\Field\Enum;
use Terranet\Administrator\Field\Link;

class EnumDetector extends AbstractDetector
{
    /** @var array */
    protected $values = [];

    /**
     * @param string $column
     * @param Model $model
     * @return mixed
     */
    protected function enumValues(string $column, Model $model)
    {
        return $this->values = enum_values($model->getTable(), $column);
    }

    /**
     * Authorize execution.
     *
     * @param string $column
     * @param Column $metadata
     * @param Model $model
     *
     * @return bool
     */
    protected function authorize(string $column, Column $metadata, Model $model): bool
    {
        return connection('mysql') && !$metadata->getLength() && !empty($this->enumValues($column, $model));

The expression $metadata->getLength() of type integer|null is loosely compared to false; this is ambiguous if the integer can be 0. You might want to explicitly use === null instead.

    }

    /**
     * Detect field class.
     *
     * @param string $column
     * @param Column $metadata
     * @param Model $model
     *
     * @return mixed
     */
    protected function detect(string $column, Column $metadata, Model $model)
    {
        $values = translated_values($this->values, app('scaffold.module')->url(), $column);

The method url() does not exist on Illuminate\Foundation\Application.

        if (!$metadata->getNotNull()) {
            $values = ['' => '----'] + $values;
        }

        return Enum::make($column, $column)->setOptions($values);
    }
}