app\modules\auth\models\forms\ResetPasswordForm

Complexity

Total Complexity

9

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	16	5
A	rules()	0	7	1
A	attributeLabels()	0	6	1
A	resetPassword()	0	12	2

<?php

namespace app\modules\auth\models\forms;

use Yii;
use yii\base\Exception;
use yii\web\BadRequestHttpException;
use app\models\entity\User;
use app\modules\auth\services\Tokenizer;

class ResetPasswordForm extends \yii\base\Model
{
    /**
     * @var string
     */
    public $password;
    /**
     * @var \app\models\entity\User
     */
    private $user;
    /**
     * @var Tokenizer
     */
    private $tokenizer;

    /**
     * Creates a form model given a token.
     *
     * @param string $token
     * @param array $config name-value pairs that will be used to initialize the object properties
     * @throws \yii\base\BadRequestHttpException
     */
    public function __construct($token, Tokenizer $tokenizer, $config = [])
    {
        $this->tokenizer = $tokenizer;

        if (empty($token) || !is_string($token) || !$this->tokenizer->validate($token)) {
            throw new BadRequestHttpException(Yii::t('app', 'Invalid link for reset password'));
        }

        $this->user = User::find()->passwordResetToken($token)->one();

It seems like \app\models\entity\User::find()->passwordResetToken($token)->one() can also be of type object<yii\db\ActiveRecord> or array. However, the property $user is declared as type object<app\models\entity\User>. Maybe add an additional type check?

        if (!$this->user) {
            throw new BadRequestHttpException(Yii::t('app', 'Invalid link for reset password'));
        }

        parent::__construct($config);
    }

    /**
     * @inheritdoc
     */
    public function rules()
    {
        return [
            ['password', 'required'],
            ['password', 'string', 'min' => 6],
        ];
    }

    /**
     * @inheritdoc
     */
    public function attributeLabels()
    {
        return [
            'password' => Yii::t('app', 'Password'),
        ];
    }

    /**
     * Resets password
     *
     * @throws Exception
     */
    public function resetPassword(): void
    {
        $this->user->setPassword($this->password);
        $this->user->removePasswordResetToken();
        $this->user->updateDateLogin();

        if (!$this->user->save()) {
            throw new Exception(Yii::t('app', 'An error occurred while saving user'));
        }

        Yii::$app->user->login($this->user, 3600 * 24 * 30);
    }
}