EloquentUserProviderWithPasswordUpdate::__construct() - Code Metrics - SamAsEnd/laravel-needs-auto-rehash - Measure and Improve Code Quality continuously with Scrutinizer

__construct() A
last analyzed 2020-09-21 20:51 UTC

↳ Parent: EloquentUserProviderWithPasswordUpdate

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

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

<?php

namespace SamAsEnd\NeedsAutoRehash\Providers;

use Illuminate\Auth\EloquentUserProvider;
use Illuminate\Contracts\Auth\Authenticatable;

class EloquentUserProviderWithPasswordUpdate extends EloquentUserProvider implements ProviderWithPasswordUpdate
{
    public function __construct(EloquentUserProvider $provider)
    {
        parent::__construct($provider->hasher, $provider->model);
    }

    public function updatePassword(Authenticatable $user, $plainPassword)
    {
        $user->password = $this->hasher->make($plainPassword);


        $timestamps = $user->timestamps;


        $user->timestamps = false;

        $user->save();

        $user->timestamps = $timestamps;
    }
}


1		<?php
2
3		namespace SamAsEnd\NeedsAutoRehash\Providers;
4
5		use Illuminate\Auth\EloquentUserProvider;
6		use Illuminate\Contracts\Auth\Authenticatable;
7
8		class EloquentUserProviderWithPasswordUpdate extends EloquentUserProvider implements ProviderWithPasswordUpdate
9		{
10	2	public function __construct(EloquentUserProvider $provider)
11		{
12	2	parent::__construct($provider->hasher, $provider->model);
13	2	}
14
15	2	public function updatePassword(Authenticatable $user, $plainPassword)
16		{
17	2	$user->password = $this->hasher->make($plainPassword);
		0 ignored issues – show Bug introduced 2020-09-21 20:34 UTC by Report Bug Copy Issue Report Accessing `password` on the interface `Illuminate\Contracts\Auth\Authenticatable` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
18
19	2	$timestamps = $user->timestamps;
		0 ignored issues – show Bug introduced 2020-09-21 20:34 UTC by Report Bug Copy Issue Report Accessing `timestamps` on the interface `Illuminate\Contracts\Auth\Authenticatable` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
20
21	2	$user->timestamps = false;
22
23	2	$user->save();
24
25	2	$user->timestamps = $timestamps;
26	2	}
27		}
28

SamAsEnd / laravel-needs-auto-rehash

__construct() A last analyzed 2020-09-21 20:51 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

__construct() A
last analyzed 2020-09-21 20:51 UTC