UserConfirmation::confirm() - Code Metrics - afrittella/back-project - Measure and Improve Code Quality continuously with Scrutinizer

UserConfirmation::confirm() A
last analyzed 2018-02-11 23:44 UTC

↳ Parent: UserConfirmation

Complexity

Conditions	3
Paths	2

Size

Total Lines	11
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	7
nc	2
nop	1
dl	0
loc	11
rs	9.4285
c	0
b	0
f	0

<?php namespace Afrittella\BackProject\Traits;
/**
 * UserConfirmation Trait
 * checks if User model is confirmed
 *
 * @author Andrea Frittella <[email protected]>
*/

trait UserConfirmation
{
    protected $confirmation_field = 'confirmed';
    protected $confirmation_token_field = 'confirmation_code';

    public function confirm($code)
    {
    if ($this->{$this->confirmation_token_field} == $code && $this->isPendingConfirmation()) {
        $this->{$this->confirmation_token_field} = null;
        $this->{$this->confirmation_field} = 1;
        $this->save();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        return true;
    }

    return false;
    }

    public function isConfirmed()
    {
    return (bool) $this->{$this->confirmation_field};
    }

    public function hasConfirmationCode()
    {
    return !is_null($this->{$this->confirmation_token_field} );
    }

    public function isPendingConfirmation()
    {
    return !$this->isConfirmed() && $this->hasConfirmationCode();
    }

    public function getConfirmationField()
    {
    return $this->confirmation_field;
    }

    public function getConfirmationTokenField()
    {
    return $this->confirmation_token_field;
    }

    public function generateConfirmationCode()
    {
        return hash_hmac('sha256', str_random(40), config('app.key'));
    }
}


1			<?php namespace Afrittella\BackProject\Traits;
2			/**
3			* UserConfirmation Trait
4			* checks if User model is confirmed
5			*
6			* @author Andrea Frittella <[email protected]>
7			*/
8
9			trait UserConfirmation
10			{
11			protected $confirmation_field = 'confirmed';
12			protected $confirmation_token_field = 'confirmation_code';
13
14			public function confirm($code)
15			{
16			if ($this->{$this->confirmation_token_field} == $code && $this->isPendingConfirmation()) {
17			$this->{$this->confirmation_token_field} = null;
18			$this->{$this->confirmation_field} = 1;
19			$this->save();
			0 ignored issues – show Bug introduced 2017-04-07 07:13 UTC by Report Bug Copy Issue Report It seems like `save()` 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			return true;
21			}
22
23			return false;
24			}
25
26			public function isConfirmed()
27			{
28			return (bool) $this->{$this->confirmation_field};
29			}
30
31			public function hasConfirmationCode()
32			{
33			return !is_null($this->{$this->confirmation_token_field} );
34			}
35
36			public function isPendingConfirmation()
37			{
38			return !$this->isConfirmed() && $this->hasConfirmationCode();
39			}
40
41			public function getConfirmationField()
42			{
43			return $this->confirmation_field;
44			}
45
46			public function getConfirmationTokenField()
47			{
48			return $this->confirmation_token_field;
49			}
50
51			public function generateConfirmationCode()
52			{
53			return hash_hmac('sha256', str_random(40), config('app.key'));
54			}
55			}
56

afrittella / back-project

GitHub Access Token became invalid

UserConfirmation::confirm() A last analyzed 2018-02-11 23:44 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

UserConfirmation::confirm() A
last analyzed 2018-02-11 23:44 UTC