ByTIC\Hello\Modules\AbstractModule\Forms\Users\LoginFormTrait::processValidation()

Complexity

Conditions	6
Paths	9

Size

Total Lines	18
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

0

<?php
    public function processValidation()
    {
        parent::processValidation();

        $email = $this->getElement('email');

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

        if (!$email->isError()) {
            $value = $email->getValue();
            if (!valid_email($value)) {

The function valid_email was not found. Maybe you did not declare it correctly or list all dependencies?

                $email->addError($this->getModelMessage('email.bad'));

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

            }
        }

        $password = $this->getElement('password');
        if (!$email->isError() && !$password->isError()) {
            $model = $this->getModel();

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

            $request = ['email' => $email->getValue(), 'password' => $password->getValue()];
            if (!$model->authenticate($request)) {
                $email->addError($this->getModelMessage('login.error'));
            }