Nip\Form\Traits\HasModelTrait

Complexity

Total Complexity

18

Size/Duplication

Total Lines	116
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	saveModel()	0	3	1
A	getModelMessage()	0	3	1
A	getDataFromModel()	0	7	4
A	getModelManager()	0	3	1
A	addModelError()	0	3	1
A	addInputModelError()	0	3	1
A	getModel()	0	3	1
A	initialized()	0	4	1
A	process()	0	4	1
A	getModelLabel()	0	3	1
A	_addModelFormMessage()	0	5	1
A	setModel()	0	4	1
A	saveToModel()	0	6	3

<?php

namespace Nip\Form\Traits;

use Nip\Records\AbstractModels\Record;

/**
 * Trait HasModelTrait
 * @package Nip\Form\Traits
 */
trait HasModelTrait
{
    /**
     * @var Record
     */
    protected $model;

    public function initialized()
    {
        parent::initialized();
        $this->getDataFromModel();
    }

    /**
     * @param $name
     * @return $this
     */
    public function addModelError($name)
    {
        return $this->addError($this->getModelMessage($name));

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

    }

    /**
     * @param $name
     * @param array $variables
     * @return mixed
     */
    public function getModelMessage($name, $variables = [])
    {
        return $this->getModelManager()->getMessage('form.'.$name, $variables);
    }

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

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

    protected function getDataFromModel()
    {
        $elements = $this->getElements();

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

        if (is_array($elements)) {
            foreach ($elements as $name => $element) {
                if (isset($this->getModel()->{$name})) {
                    $element->getData($this->getModel()->{$name}, 'model');
                }
            }
        }
    }

    /**
     * @param $input
     * @param $name
     * @param array $variables
     * @return $this
     */
    public function addInputModelError($input, $name, $variables = [])
    {
        return $this->getElement($input)->addError($this->getModelMessage($name, $variables));

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

    }

    /**
     * @param $name
     * @return mixed
     */
    public function getModelLabel($name)
    {
        return $this->getModelManager()->getLabel($name);
    }

    /**
     * @return \Nip\Records\RecordManager
     */
    public function getModelManager()
    {
        return $this->getModel()->getManager();
    }

    public function process()
    {
        $this->saveToModel();
        $this->saveModel();
    }

    public function saveToModel()
    {
        $elements = $this->getElements();
        if (is_array($elements)) {
            foreach ($elements as $name => $element) {
                $this->getModel()->{$name} = $element->getValue('model');
            }
        }
    }

    public function saveModel()
    {
        $this->getModel()->save();
    }

    protected function _addModelFormMessage($form, $model)
    {
        $this->_messageTemplates[$form] = $this->getModelMessage($model);

The property _messageTemplates does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this;
    }
}