RecoverPasswordFormTrait::initEmailInput() - Code Metrics - Inspection of "add recover traits" - bytic/hello - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 22d22f...259b7d )

by Gabriel

created 2020-05-20 10:50 UTC

RecoverPasswordFormTrait::initEmailInput() A

↳ Parent: RecoverPasswordFormTrait

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	4
ccs	0
cts	3
cp	0
crap	2
rs	10
c	0
b	0
f	0

<?php

namespace ByTIC\Hello\Modules\AbstractModule\Forms\Users;

/**
 * Trait RecoverPasswordFormTrait
 * @package ByTIC\Hello\Modules\AbstractModule\Forms\Users
 */
trait RecoverPasswordFormTrait
{
    public function init()
    {
        parent::init();
        $this->removeClass('form-horizontal');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->addClass('user-recover');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $this->initEmailInput();

        $this->addButton('save', translator()->trans('submit'));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    protected function initEmailInput()
    {
        $this->addInput('email', translator()->trans('email'), true);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function processValidation()
    {
        parent::processValidation();

        $this->processValidationEmail();
    }

    public function processValidationEmail()
    {
        $email = $this->getElement('email');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        if (!$email->isError()) {
            $value = $email->getValue();
            if (!valid_email($value) && false) {
                $email->addError($this->getModelMessage('email.bad'));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            } else {
                $users = $this->getModelManager()->findByEmail($value);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
                if (count($users) == 1) {
                    $this->getModel()->email = $value;
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
                } else {
                    $email->addError($this->getModelMessage('email.dnx'));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
                }
            }
        }
    }

    public function process()
    {
        $this->getModel()->recoverPassword();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    protected function getDataFromModel()
    {
        parent::getDataFromModel();
        $this->_addModelFormMessage('no-email', 'email.empty');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1			<?php
2
3			namespace ByTIC\Hello\Modules\AbstractModule\Forms\Users;
4
5			/**
6			* Trait RecoverPasswordFormTrait
7			* @package ByTIC\Hello\Modules\AbstractModule\Forms\Users
8			*/
9			trait RecoverPasswordFormTrait
10			{
11			public function init()
12			{
13			parent::init();
14			$this->removeClass('form-horizontal');
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `removeClass()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
15			$this->addClass('user-recover');
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `addClass()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
16
17			$this->initEmailInput();
18
19			$this->addButton('save', translator()->trans('submit'));
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `addButton()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
20			}
21
22			protected function initEmailInput()
23			{
24			$this->addInput('email', translator()->trans('email'), true);
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `addInput()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
25			}
26
27			public function processValidation()
28			{
29			parent::processValidation();
30
31			$this->processValidationEmail();
32			}
33
34			public function processValidationEmail()
35			{
36			$email = $this->getElement('email');
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `getElement()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
37			if (!$email->isError()) {
38			$value = $email->getValue();
39			if (!valid_email($value) && false) {
40			$email->addError($this->getModelMessage('email.bad'));
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `getModelMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
41			} else {
42			$users = $this->getModelManager()->findByEmail($value);
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `getModelManager()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
43			if (count($users) == 1) {
44			$this->getModel()->email = $value;
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `getModel()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
45			} else {
46			$email->addError($this->getModelMessage('email.dnx'));
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `getModelMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
47			}
48			}
49			}
50			}
51
52			public function process()
53			{
54			$this->getModel()->recoverPassword();
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `getModel()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
55			}
56
57			protected function getDataFromModel()
58			{
59			parent::getDataFromModel();
60			$this->_addModelFormMessage('no-email', 'email.empty');
			0 ignored issues – show Bug introduced 2020-05-20 11:09 UTC by Report Bug Copy Issue Report It seems like `_addModelFormMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
61			}
62			}
63

bytic / hello

Push — master ( 22d22f...259b7d )

RecoverPasswordFormTrait::initEmailInput() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like