SleepingOwl\Admin\Form\FormDefault

Complexity

Total Complexity

49

Size/Duplication

Total Lines	405
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

25 Methods

Rating	Name	Duplication	Size	Complexity
A	getRepository()	0	4	1
A	getView()	0	4	1
A	getAction()	0	4	1
A	getClass()	0	4	1
A	setModelClass()	0	8	2
A	setItems()	0	8	2
A	getModelConfiguration()	0	4	1
A	setView()	0	6	1
A	__construct()	0	10	1
B	initialize()	0	27	4
A	getButtons()	0	4	1
A	setButtons()	0	6	1
A	setAction()	0	10	2
A	getItems()	0	4	1
A	addItem()	0	4	1
A	setId()	0	7	4
A	getModel()	0	4	1
A	setModel()	0	10	1
B	saveForm()	0	31	6
A	saveBelongsToRelations()	0	11	4
B	saveHasOneRelations()	0	14	6
A	validateForm()	0	22	3
A	toArray()	0	10	1
A	render()	0	4	1
A	__toString()	0	4	1

<?php

namespace SleepingOwl\Admin\Form;

use Request;
use Validator;
use Illuminate\Support\Collection;
use Illuminate\Database\Eloquent\Model;
use KodiComponents\Support\HtmlAttributes;
use SleepingOwl\Admin\Form\Element\Upload;
use SleepingOwl\Admin\Contracts\FormInterface;
use SleepingOwl\Admin\Contracts\DisplayInterface;
use SleepingOwl\Admin\Contracts\RepositoryInterface;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use SleepingOwl\Admin\Contracts\FormButtonsInterface;
use SleepingOwl\Admin\Contracts\FormElementInterface;
use Illuminate\Database\Eloquent\Relations\HasOneOrMany;
use SleepingOwl\Admin\Contracts\ModelConfigurationInterface;

class FormDefault extends FormElements implements DisplayInterface, FormInterface
{
    use HtmlAttributes;

    /**
     * View to render.
     * @var string|\Illuminate\View\View
     */
    protected $view = 'form.default';

    /**
     * Form related class.
     * @var string
     */
    protected $class;

    /**
     * @var FormButtons
     */
    protected $buttons;

    /**
     * Form related repository.
     * @var RepositoryInterface
     */
    protected $repository;

    /**
     * Form action url.
     * @var string
     */
    protected $action;

    /**
     * Form related model instance.
     * @var Model
     */
    protected $model;

    /**
     * Currently loaded model id.
     * @var int
     */
    protected $id;

    /**
     * Is form already initialized?
     * @var bool
     */
    protected $initialized = false;

    /**
     * FormDefault constructor.
     *
     * @param array $elements
     */
    public function __construct(array $elements = [])
    {
        parent::__construct($elements);

        $this->setButtons(
            app(FormButtonsInterface::class)
        );

        $this->initializePackage();
    }

    /**
     * Initialize form.
     */
    public function initialize()
    {
        if ($this->initialized) {
            return;
        }

        $this->initialized = true;
        $this->repository = app(RepositoryInterface::class, [$this->class]);

        $this->setModel(app($this->class));

        parent::initialize();

        $this->getElements()->each(function ($element) {
            if ($element instanceof Upload and ! $this->hasHtmlAttribute('enctype')) {

'enctype' is of type string, but the function expects a object<KodiComponents\Support\srtring>.

Using logical operators such as and instead of && is generally not recommended.

                $this->setHtmlAttribute('enctype', 'multipart/form-data');
            }
        });

        $this->setHtmlAttribute('method', 'POST');

        $this->getButtons()->setModelConfiguration(
            $this->getModelConfiguration()
        );

        $this->includePackage();
    }

    /**
     * @return FormButtons
     */
    public function getButtons()
    {
        return $this->buttons;
    }

    /**
     * @param FormButtonsInterface $buttons
     *
     * @return $this
     */
    public function setButtons(FormButtonsInterface $buttons)
    {
        $this->buttons = $buttons;

$buttons is of type object<SleepingOwl\Admin\Contracts\FormButtonsInterface>, but the property $buttons was declared to be of type object<SleepingOwl\Admin\Form\FormButtons>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        return $this;
    }

    /**
     * @return RepositoryInterface
     */
    public function getRepository()
    {
        return $this->repository;
    }

    /**
     * @return string|\Illuminate\View\View
     */
    public function getView()
    {
        return $this->view;
    }

    /**
     * @param \Illuminate\View\View|string $view
     *
     * @return $this
     */
    public function setView($view)
    {
        $this->view = $view;

        return $this;
    }

    /**
     * @return string
     */
    public function getAction()
    {
        return $this->action;
    }

    /**
     * @param string $action
     *
     * @return $this
     */
    public function setAction($action)
    {
        if (is_null($this->action)) {
            $this->action = $action;
        }

        $this->setHtmlAttribute('action', $this->action);

        return $this;
    }

    /**
     * @return string
     */
    public function getClass()
    {
        return $this->class;
    }

    /**
     * @param string $class
     *
     * @return $this
     */
    public function setModelClass($class)
    {
        if (is_null($this->class)) {
            $this->class = $class;
        }

        return $this;
    }

    /**
     * @deprecated 4.5.0
     * @see getElements()
     *
     * @return Collection[]
     */
    public function getItems()
    {
        return $this->getElements();
    }

    /**
     * @deprecated 4.5.0
     * @see setElements()
     *
     * @param array|FormElementInterface $items
     *
     * @return $this
     */
    public function setItems($items)
    {
        if (! is_array($items)) {
            $items = func_get_args();
        }

        return $this->setElements($items);
    }

    /**
     * @deprecated 4.5.0
     * @see addElement()
     *
     * @param FormElementInterface $item
     *
     * @return $this
     */
    public function addItem($item)
    {
        return $this->addElement($item);
    }

    /**
     * Set currently loaded model id.
     *
     * @param int $id
     */
    public function setId($id)
    {
        if (is_null($this->id) and ! is_null($id) and ($model = $this->getRepository()->find($id))) {

Using logical operators such as and instead of && is generally not recommended.

            $this->id = $id;
            $this->setModel($model);
        }
    }

    /**
     * Get related form model configuration.
     * @return ModelConfigurationInterface
     */
    public function getModelConfiguration()
    {
        return app('sleeping_owl')->getModel($this->class);
    }

    /**
     * @return Model
     */
    public function getModel()
    {
        return $this->model;
    }

    /**
     * @param Model $model
     *
     * @return $this
     */
    public function setModel(Model $model)
    {
        $this->model = $model;

        parent::setModel($model);

        $this->getButtons()->setModel($this->getModel());

        return $this;
    }

    /**
     * Save instance.
     *
     * @param ModelConfigurationInterface $modelConfiguration
     *
     * @return bool
     */
    public function saveForm(ModelConfigurationInterface $modelConfiguration)
    {
        if ($modelConfiguration !== $this->getModelConfiguration()) {
            return;
        }

        parent::save();

It seems like you call parent on a different method (save() instead of saveForm()). Are you sure this is correct? If so, you might want to change this to $this->save().

        $this->saveBelongsToRelations();

        $loaded = $this->getModel()->exists;

        if ($modelConfiguration->fireEvent($loaded ? 'updating' : 'creating', true, $this->getModel()) === false) {

It seems like you code against a concrete implementation and not the interface SleepingOwl\Admin\Contracts\ModelConfigurationInterface as the method fireEvent() does only exist in the following implementations of said interface: SleepingOwl\Admin\Model\ModelConfiguration, SleepingOwl\Admin\Model\ModelConfigurationManager, SleepingOwl\Admin\Model\SectionModelConfiguration, SleepingOwl\Admin\Section.

            return false;
        }

        if ($modelConfiguration->fireEvent('saving', true, $this->getModel()) === false) {

It seems like you code against a concrete implementation and not the interface SleepingOwl\Admin\Contracts\ModelConfigurationInterface as the method fireEvent() does only exist in the following implementations of said interface: SleepingOwl\Admin\Model\ModelConfiguration, SleepingOwl\Admin\Model\ModelConfigurationManager, SleepingOwl\Admin\Model\SectionModelConfiguration, SleepingOwl\Admin\Section.

            return false;
        }

        $this->getModel()->save();

        $this->saveHasOneRelations();

        parent::afterSave();

It seems like you call parent on a different method (afterSave() instead of saveForm()). Are you sure this is correct? If so, you might want to change this to $this->afterSave().

        $modelConfiguration->fireEvent($loaded ? 'updated' : 'created', false, $this->getModel());

It seems like you code against a concrete implementation and not the interface SleepingOwl\Admin\Contracts\ModelConfigurationInterface as the method fireEvent() does only exist in the following implementations of said interface: SleepingOwl\Admin\Model\ModelConfiguration, SleepingOwl\Admin\Model\ModelConfigurationManager, SleepingOwl\Admin\Model\SectionModelConfiguration, SleepingOwl\Admin\Section.

        $modelConfiguration->fireEvent('saved', false, $this->getModel());

It seems like you code against a concrete implementation and not the interface SleepingOwl\Admin\Contracts\ModelConfigurationInterface as the method fireEvent() does only exist in the following implementations of said interface: SleepingOwl\Admin\Model\ModelConfiguration, SleepingOwl\Admin\Model\ModelConfigurationManager, SleepingOwl\Admin\Model\SectionModelConfiguration, SleepingOwl\Admin\Section.

        return true;
    }

    protected function saveBelongsToRelations()
    {
        $model = $this->getModel();

        foreach ($model->getRelations() as $name => $relation) {
            if ($model->{$name}() instanceof BelongsTo && ! is_null($relation)) {
                $relation->save();
                $model->{$name}()->associate($relation);
            }
        }
    }

    protected function saveHasOneRelations()
    {
        $model = $this->getModel();

        foreach ($model->getRelations() as $name => $relation) {
            if ($model->{$name}() instanceof HasOneOrMany && ! is_null($relation)) {
                if (is_array($relation) || $relation instanceof \Traversable) {
                    $model->{$name}()->saveMany($relation);
                } else {
                    $model->{$name}()->save($relation);
                }
            }
        }
    }

    /**
     * @param ModelConfigurationInterface $modelConfiguration
     *
     * @return \Illuminate\Contracts\Validation\Validator|null|bool
     */
    public function validateForm(ModelConfigurationInterface $modelConfiguration)
    {
        if ($modelConfiguration !== $this->getModelConfiguration()) {
            return;
        }

        $data = Request::all();

        $verifier = app('validation.presence');
        $verifier->setConnection($this->getModel()->getConnectionName());

        $validator = Validator::make(
            $data,
            $this->getValidationRules(),
            $this->getValidationMessages(),
            $this->getValidationLabels()
        );

        $validator->setPresenceVerifier($verifier);

        return $validator->fails() ? $validator : true;
    }

    /**
     * Get the instance as an array.
     *
     * @return array
     */
    public function toArray()
    {
        return [
            'items' => $this->getElements()->onlyVisible(),
            'instance' => $this->getModel(),
            'attributes' => $this->htmlAttributesToString(),
            'buttons' => $this->getButtons(),
            'backUrl' => session('_redirectBack', \URL::previous()),
        ];
    }

    /**
     * @return \Illuminate\View\View|\Illuminate\Contracts\View\Factory
     */
    public function render()
    {
        return app('sleeping_owl.template')->view($this->getView(), $this->toArray());
    }

    /**
     * @return string
     */
    public function __toString()
    {
        return (string) $this->render();
    }
}