Sco\Admin\Traits\HasSelectOptions::getOptions()

Complexity

Conditions	4
Paths	4

Size

Total Lines	27
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

0

<?php
    public function getOptions()
    {
        if ($this->options instanceof \Closure) {
            $options = ($this->options)();
        } elseif ($this->isOptionsModel()) {
            $options = $this->getOptionsFromModel();
        } elseif (is_array($this->options)) {
            $options = $this->options;
        } else {
            throw new InvalidArgumentException(
                sprintf(
                    "The %s element[%s] options must be return array(key=>value)",
                    $this->getType(),

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

                    $this->getName()

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

                )
            );
        }

        return collect($options)->mapWithKeys(function ($value, $key) {
            return [
                $key => [
                    'label' => $value,
                    'value' => (string) $key,
                ],
            ];
        })->values();
    }