Nip\Form\Traits\CanInitializeTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	initAction()	0	4	2
A	initialized()	0	6	2
A	initialize()	0	10	2
A	postInit()	0	2	1
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;

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

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

        $this->initAction();
        $this->postInit();
        $this->initialized(true);
        return $this;
    }

    /**
     * @param null $initialized

Are you sure the doc-type for parameter $initialized is correct as it would always require null to be passed?

     * @return bool
     */
    protected function initialized($initialized = null): bool
    {
        if (is_bool($initialized)) {

The condition is_bool($initialized) is always false.

            $this->initialized = $initialized;
        }
        return $this->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?

        }
    }

    public function postInit()
    {
    }
}