Yii пользовательская проверка и проверка на стороне сервера?

Question

Я новичок в Yii. Я делаю страницу смены пароля для пользователей. Когда я дошел до проверки, у меня возникли проблемы.

Это представление :

<div class="form">
<?php $form=$this->beginWidget('CActiveForm', array(
    'id'=>'changepassword-form',
    'enableClientValidation'=>true,
    'clientOptions'=>array(
    'validateOnSubmit'=>true,
    ),
)); ?>

<p class="note">Fields with <span class="required">*</span> are required.</p>
<?php echo $form->errorSummary($model); ?>
<div class="row">
    <?php echo $form->labelEx($model,'old_pwd'); ?>
    <?php echo $form->textField($model,'old_pwd'); ?>
    <?php echo $form->error($model,'old_pwd'); ?>
</div>

<div class="row">
    <?php echo $form->labelEx($model,'pwd'); ?>
    <?php echo $form->textField($model,'pwd'); ?>
    <?php echo $form->error($model,'pwd'); ?>
</div>

<div class="row">
    <?php echo $form->labelEx($model,'pwd_repeat'); ?>
    <?php echo $form->passwordField($model, 'pwd_repeat'); ?>
    <?php echo $form->error($model,'pwd_repeat'); ?>
    <p class="hint">
        Passwords must be minimum 6 characters and can contain alphabets, numbers and special characters.
    </p>
</div>


<div class="row buttons">
    <?php echo CHtml::submitButton('Change Password'); ?>
</div>

<?php $this->endWidget(); ?>
</div><!-- form -->

Модель :

/**
 * @return array validation rules for model attributes.
 */
public function rules()
{
    // NOTE: you should only define rules for those attributes that
    // will receive user inputs.
    return array(
        array('user, pwd, status, old_pwd, pwd_repeat', 'required'),
        array('user', 'length', 'max'=>50),
        array('pwd', 'length', 'max'=>30),
        array('status', 'length', 'max'=>10),
        array('pwd','compare'),
        array('old_pwd','checkOldPassword'),
        // The following rule is used by search().
        // Please remove those attributes that should not be searched.
        array('id, user, pwd, status', 'safe', 'on'=>'search'),
    );
}

// FOR CHECKING IF THE PASSWORD IS VALID
public function checkOldPassword($attribute,$params)
{
    $record=Admin::model()->findByAttributes(array('pwd'=>$this->attribute));

    if($record===null){
        $this->addError($attribute, 'Invalid password');
    }
}

Контроллер

/**
 * Change Password for users
 */
public function actionChangePassword()
{
    // renders the view file 'protected/views/site/index.php'
    // using the default layout 'protected/views/layouts/main.php'
    if(Yii::app()->user->isGuest){
        // If the user is guest or not logged in redirect to the login form
        $this->redirect(array('site/login'));
    }
    else{
        $model = new Admin;
        if(isset($_POST['Admin'])){

            $model->attributes=$_POST['Admin'];

            $this->render('changepassword',array('model'=>$model));
        }
        else{
            $this->render('changepassword',array('model'=>$model));
        }
    }
}

Я использую специальный валидатор ' checkOldPassword ' для проверки текущего пароля пользователя. Теперь проблема в том, что пользовательская проверка не работает . Кроме того, если я отключаю JavaScript , я обнаружил, что ошибки проверки на стороне сервера также не отображаются . Я новичок в Yii. Прошу прощения за любые глупые ошибки. Пожалуйста, помогите.

Answer 1

public function rules()
{
    // NOTE: you should only define rules for those attributes that
    // will receive user inputs.
    public $pwd_repeat;
    public $old_pwd;

    return array(
    array('old_pwd,pwd_repeat,pwd','safe'),
        array('user, pwd, status, old_pwd, pwd_repeat', 'required'),
        array('user', 'length', 'max'=>50),
        array('pwd', 'length', 'max'=>30),
        array('status', 'length', 'max'=>10),
        array('pwd','compare'),
        array('old_pwd','checkOldPassword'),
        // The following rule is used by search().
        // Please remove those attributes that should not be searched.
        array('id, user, pwd, status', 'safe', 'on'=>'search'),
    );
}

// FOR CHECKING IF THE PASSWORD IS VALID
public function checkOldPassword($attribute,$params)
{
    $record=Admin::model()->findByAttributes(array('pwd'=>$this->attributes['old_pwd']));

    if($record===null){
        $this->addError($attribute, 'Invalid password');
    }
}

public function actionChangePassword()
{
    // renders the view file 'protected/views/site/index.php'
    // using the default layout 'protected/views/layouts/main.php'
    if(Yii::app()->user->isGuest){
        // If the user is guest or not logged in redirect to the login form
        $this->redirect(array('site/login'));
    }
    else{
        $model = new Admin;
        if(isset($_POST['Admin'])){

            $model->attributes=$_POST['Admin'];
            $model->save();
            $this->render('changepassword',array('model'=>$model));
        }
        else{
            $this->render('changepassword',array('model'=>$model));
        }
    }
}

Хотя, тем не менее, ваш подход не очень хорош ... Но сначала запустите его ..

Answer 2

Сначала я считаю, что clientValidation не будет работать, потому что вы проверяете старый пароль, поэтому вам нужна проверка на стороне сервера (Ajax Validation). clientValidation подходит для проверки длины пароля и т. д.

, поэтому поставьте эту строку: 'EnableAjaxValidation' => верно,

Во-вторых, я бы изменил ваш код на что-то вроде этого:

public function checkOldPassword($attribute,$params)
{
    $record = $this->find(array(
                'select' => 'old_password',
                'condition' => 'username=:username',
                'params' => array(':username' => $username//or user or w/e ur gonna find user),
                    ));

if($record===null){
    $this->addError($attribute, 'Invalid password');
}
else {

 if($this->old_password != $record->old_password) {

 $this->addError('old_password', "Invalid Password");
}
 }
}

Answer 3

Попробуйте:

// FOR CHECKING IF THE PASSWORD IS VALID
public function checkOldPassword($attribute,$params)
{
    $record=Admin::model()->findByAttributes(array('pwd'=>$this->pwd));

    if($record===null){
        $this->addError($attribute, 'Invalid password');
    }
}

заменить $ this-> атрибут на $ this-> pwd