Nip\Form\Traits\HasExecutionMethodsTrait

Complexity

Total Complexity

15

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	processValidation()	0	6	3
A	process()	0	2	1
A	isValid()	0	3	2
A	validate()	0	7	2
A	submited()	0	8	3
A	processRequest()	0	9	2
A	execute()	0	7	2

<?php

namespace Nip\Form\Traits;

/**
 * Trait HasExecutionMethodsTrait
 * @package Nip\Form\Traits
 */
trait HasExecutionMethodsTrait
{

    /**
     * @return bool
     */
    public function execute()
    {
        if ($this->submited()) {
            return $this->processRequest();
        }

        return false;
    }

    /**
     * @return bool
     */
    public function submited()
    {
        $request = $this->getAttrib('method') == 'post' ? $_POST : $_GET;

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

        if (count($request)) {
            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    public function processRequest()
    {
        if ($this->validate()) {
            $this->process();

            return true;
        }

        return false;
    }

    /**
     * @return bool
     */
    public function validate()
    {
        $request = $this->getAttrib('method') == 'post' ? $_POST : $_GET;
        $this->getDataFromRequest($request);

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

        $this->processValidation();

        return $this->isValid();
    }

    public function processValidation()
    {
        $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) {
                $element->validate();
            }
        }
    }

    /**
     * @return bool
     */
    public function isValid()
    {
        return count($this->getErrors()) > 0 ? false : true;

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

    }

    public function process()
    {
    }
}