Nip\Form\Traits\CanInitializeTrait

Complexity

Total Complexity

10

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	initAction()	0	4	2
A	initialized()	0	2	1
A	initialize()	0	3	1
A	postInit()	0	3	1
A	initializeIfNotInitialized()	0	19	4
A	init()	0	3	1

<?php


namespace Nip\Form\Traits;


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

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


    public function initializeIfNotInitialized()
    {
        if ($this->initialized !== false) {
            return;
        }

        if (method_exists($this, 'init')) {
            $this->init();

The function Nip\Form\Traits\CanInitializeTrait::init() has been deprecated: use initialize()

        } else {
            $this->initialize();
        }

        if (method_exists($this, 'postInit')) {
            $this->postInit();

The function Nip\Form\Traits\CanInitializeTrait::postInit() has been deprecated: use initialize()

        } else {
            $this->initialized();
        }

        $this->initialized = true;
    }

    /**
     * @deprecated use initialize()
     */
    public function init()
    {
        $this->initialize();
    }

    public function initialize()
    {
        $this->initAction();
    }

    /**
     * @deprecated use initialize()
     */
    public function postInit()
    {
        $this->initialized();
    }

    protected function initialized()
    {
    }

    protected function initAction()
    {
        if (function_exists('current_url')) {
            $this->setAction(current_url());

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

        }
    }
}