Terranet\Administrator\Traits\Module\HasForm::scaffoldForm()

Complexity

Conditions	3
Paths	2

Size

Total Lines	22
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes	2
Bugs	1	Features	0

<?php
    protected function scaffoldForm()
    {
        $editable = new Mutable();

        if ($eloquent = $this->model()) {

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

            $editable = $editable
                ->merge([$eloquent->getKeyName()])
                ->merge($translatable = $this->scaffoldTranslatable($eloquent))
                ->merge($eloquent->getFillable());

            return $editable->build(
                new Grid($eloquent)
            )->map(function ($element) use ($translatable) {
                if (\in_array($element->id(), $translatable, true)) {
                    return $element->translatable();
                }

                return $element;
            });
        }

        return $editable;
    }