Terranet\Administrator\Traits\Module\HasForm

Complexity

Total Complexity

7

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	form()	0	3	1
A	scaffoldForm()	0	22	3
A	scaffoldTranslatable()	0	5	3

<?php

namespace Terranet\Administrator\Traits\Module;

use Terranet\Administrator\Decorators\Grid;
use Terranet\Administrator\Form\Collection\Mutable;
use Terranet\Translatable\Translatable;

trait HasForm
{
    /**
     * Provides array of editable columns.
     *
     * @return Mutable
     */
    public function form()
    {
        return $this->scaffoldForm();
    }

    /**
     * Build editable columns based on table columns metadata.
     *
     * @return Mutable
     */
    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;
    }

    /**
     * @param $eloquent
     * @return array
     */
    protected function scaffoldTranslatable($eloquent)
    {
        return ($eloquent instanceof Translatable && method_exists($eloquent, 'getTranslatedAttributes'))
            ? $eloquent->getTranslatedAttributes()
            : [];
    }
}