SamAsEnd\NeedsAutoRehash\Providers\EloquentUserProviderWithPasswordUpdate

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	updatePassword()	0	11	1
A	__construct()	0	3	1

<?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);

Accessing password on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $timestamps = $user->timestamps;

Accessing timestamps on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        $user->timestamps = false;

        $user->save();

        $user->timestamps = $timestamps;
    }
}