Connections\Module::init()

Complexity

Conditions	8
Paths	6

Size

Total Lines	91
Code Lines	54

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	public function init(AbstractionController  $c)
	{
		$config = $this->getUserConfig("Auth");
		$_config = $this->toFormConfig($config);

		# config constraints

        $components = [
            "attributes" => [
                "project_name" => [
                    "required"  => true,
                    "type"      => "text",
                    "minlength" => 2,
                    "maxlength" => 60
                ],
                "mail_checking_from" => [
                    "required"  => true,
                    "type"      => "email"
                ],
                "authentication_method" => [
                    "required"  => true,
                    "type"      => "text"
                ],
                "authentication_key" => [
                    "required"  => true,
                    "type"      => "text"
                ],
                "redirect" => [
                    "required"  => true,
                    "type"      => "text"
                ]
            ],
        ];

        $options = [
            "project" => [
                "label"      => "project -> name"
            ],
            "mail_checking_from" => [
                "label"      => "mail -> checking -> from"
            ],
            "authentication_method" => [
                "label"      => "authentication -> method",
                "validators" => [
                    "InArray"  => ["haystack" => ['_COOKIE', '_SESSION']]
                ]
            ],
            "authentication_key" => [
                "label"      => "authentication -> key",
            ],
            "redirect" => [
                "label"      => "redirect"
            ],
        ];

        $form = new Form($components);
        $form->fill($_config);

It seems like $_config can also be of type mixed; however, parameter $values of Drone\Dom\Element\Form::fill() does only seem to accept array, maybe add an additional type check?

        $validator = new FormValidator($form, $options);
        $validator->validate();

        $data["validator"] = $validator;

$data was never initialized. Although not strictly required by PHP, it is generally a good practice to add $data = array(); before regardless.

        # Validar formulario
        if (!$validator->isValid())
        {
            $data["messages"] = $validator->getMessages();
            throw new \Exception("Module config errros in user.config!", 300);
        }

        $method = $config["authentication"]["method"];
        $key    = $config["authentication"]["key"];

        switch ($method)
        {
            case '_COOKIE':

		        if (!array_key_exists($key, $_COOKIE) || empty($_COOKIE[$key]))
		            header("location: " . $c->basePath ."/public/". "Auth");

The property basePath is declared private in Drone\Mvc\AbstractionController and cannot be accessed from this context.

                break;

            case '_SESSION':

		        if (!array_key_exists($key, $_SESSION) || empty($_SESSION[$key]))
		            header("location: " . $c->basePath ."/public/". "Auth");

                break;
        }

        $this->setTranslator($c);
	}
}