Валидация email на уникальность по таблице не работает?

Question

[Радость моя]

Товарищи друзья, приветы, это снова я.

Сделал в форме правило на проверку уникальности:

['email', 'unique', 'targetClass' => User::className(), 'message' => 'Эта почта уже зарегистрирована'],

но при вводе существующей почты форма не ругается. Она как это проверят? не ajax-ом? А если не ажаксом то после отправки все равно ничего не говорит (хотя как и надо - запись в базу не добавляет), либо куда-то говорит но я не вижу. Как это на фронт вывести, или еще лучше - как это сразу ajaxoм проверять при change? (ну как сейчас проверяется пустое поле или нет).

UPD:
на фронте вывожу так

<?= $form->field($formRegister, 'email')->textInput() ?>

Comments

[Дмитрий]

Добрый день.
Вообще-то должно работать без проблем. Email на уникальность так и проверяется.
Покажите полностью правила валидации и как Вы записываете в базу значение.

[Радость моя]

правила:

public function rules()
    {
        return [

            ['email', 'required'],
            ['email', 'email'],
            ['email', 'unique', 'targetClass' => User::className(), 'message' => 'Эта почта уже зарегистрирована'],
            
            ['fio', 'trim'],
            ['fio', 'required'],
            ['fio', 'string', 'min' => 1,],

            ['phone_mobile', 'trim'],
            ['phone_mobile', 'required'],
            ['phone_mobile', 'string', 'min' => 11,],

            ['position_id', 'required'],
            ['position_id', 'exist', 'targetClass' => Positions::className(), 'targetAttribute' => 'id'],

            ['pay_type', 'required'],
            ['pay_type', 'exist', 'targetClass' => PayTypes::className(), 'targetAttribute' => 'id'],

            ['pay_account', 'trim'],
            ['pay_account', 'required'],
            ['pay_account', 'string',],

        ];
    }

регистрация:

public function signup()
    {
        if (!$this->validate()) {
            return null;
        }
        
        $user = new User();
        $user->email = $this->email;
        $user->setPassword('123456');
        $user->generateAuthKey();
        
        return $user->save() ? $user : null;
    }

[Радость моя]

slo_nik, забыл маякнуть :)

[Дмитрий]

Радость моя, только что, по этому поводу, хотел дополнить свой ответ))) Так бы Вы долго ждали, если бы я случайно не глянул, что Вы комментарий оставили.

Answer 1

[Дмитрий]

Радость моя, скорей всего проблема у Вас тут:

if (!$this->validate()) {
            return null;
        }

Если не проходит валидация Вы возвращаете null, поэтому не видите сообщений.
Сделайте вот так:

public function signup()
    {
        if($this->validate()){
            $user = new Users();
            $user->email = $this->email;
            $user->setPassword($this->password);
            $user->generateAuthKey();

            if($user->save()){
               // после успешного сохранения в базу отправляете письмо или ещё что-то делаете
            }
            return $user;
        }
        return null;
    }

Comments

[Радость моя]

не, тоже самое. может какой то параметр надо передать в activeform при ее инициализации? или оно по умолчанию проверяет уник через ajax ?
сейчас у меня так:

<?php $form = ActiveForm::begin(['id' => 'form-signup']); ?>
<?= $form->field($formRegister, 'email')->textInput() ?>
...

[Дмитрий]

Радость моя, через ajax она ничего не проверяет, пока явно не укажите, что надо использовать ajax.
Вот при таких параметрах, с такими правилами и методом singup всё должно работать.
Смотрите в консоли браузера на вкладках "консоль" и "сеть", так же проверьте логи сервера и самого yii2

[Дмитрий]

Радость моя, Вы форму ajax-ом отправляете?

[Радость моя]

Вот при таких параметрах

slo_nik, каких таких? :)

[Дмитрий]

Радость моя, вот при таких. Этого достаточно, чтобы форма уходила без ajax, с перезагрузкой страницы.
$form = ActiveForm::begin(['id' => 'form-signup'])

[Радость моя]

slo_nik, а мне надо как раз чтобы уникальность почты проверялась ажаксом. может я в вопросе неправильно выразился, извиняюсь.

[Дмитрий]

Радость моя, а как Вы в контроллере используете метод signup?
Покажите действие контроллера.

[Дмитрий]

Радость моя, тогда делайте ajax валидация именно для этого поля.

<?= $form->field($formRegister, 'email', ['enableAjaxValidation'=>true, 'validateOnType' => true])->textInput() ?>

А в контролере действие, которое будет обрабатывать ajax запрос.

p.s. вроде не ошибся в параметра, но уточните в документации

[Радость моя]

slo_nik, спасибо, ajax-валидация заработала. только теперь модель сохраняется, но страница выдает ошибку: Response content must be a string or an object implementing __toString(). что то я наверно response неправильно отдаю..

а экшн сохранения вот:

public function actionCreate() {
		
		$model = new SignupHandForm();
		
		// Ajax-валидация
		if (Yii::$app->request->isAjax && $model->load(Yii::$app->request->post())) {
			Yii::$app->response->format = \yii\web\Response::FORMAT_JSON;
			return ActiveForm::validate($model);
		}
		// Сохранение
		if ($model->load(Yii::$app->request->post())) {

			if ($model->validate()) {

				return $model->signup();
				
			} else { return null; }
		}

[Дмитрий]

Радость моя, уберите проверку, Вы же выше уже провалидировали код и прекращайте возвращать null.
Луче верните какое-либо исключение, которое больше подойдёт для Вашей ситуации.

[Дмитрий]

Радость моя, и вот тут, из-за этого возможно ошибка.
return $model->signup();
Сделайте, лучше, вот так:

if($model->signup()){
                Yii::$app->session->setFlash('success', Yii::t('app','FLASH_EMAIL_CONFIRM_REQUEST'));
                return $this->goHome();
            }

В методе signup Вы что-то возвращаете и в контролере Вы опять же что-то пытаетесь вернуть. Проверяете, отработал метод - формируете flash сообщение и отправляете на какую-то страницу, например на главную.
Конечно же подставьте свои значения.

[Радость моя]

slo_nik, спасибо за подсказки. да я накосячил с двойной валидацией. теперь все норм.

[Дмитрий]

Радость моя, пожалуйста