bytic / form

src/Traits/HasExecutionMethodsTrait.php

<?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()
    {
        $this->initializeIfNotInitialized();

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

        $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()
    {
        $this->initializeIfNotInitialized();

        $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()
    {
        $this->initializeIfNotInitialized();
    }
}