Как правильно валидировать уникальное поле?

Question

[dev400]

Пытаюсь сделать телефон уникальным, делаю по примеру других правил валидации

['phone', 'trim'],
            ['phone_repeat', 'trim'],
            ['phone', 'required'],
            ['phone_repeat', 'required'],
            ['phone', 'string', 'min' => 10],
            ['phone_repeat', 'compare', 'compareAttribute' => 'phone'],
            ['phone', 'unique', 'targetClass' => '\app\models\User', 'message' => 'Телефон занят'],

Но при попытке записи телефона, который уже есть в базе, получаю исключение..

Answer 1

[davidnum95]

Можешь свой валидатор написать. Что то типа:

public function validatePhone($attribute, $params)
    {
        if (!$this->hasErrors()) {
            $user = User::findOne(['phone' => $this->$attribute]);

            if ($user ) {
                $this->addError($attribute, 'Телефон занят');
        }
    }

В rules:
['phone', 'validatePhone'],

Comments

[dev400]

да, идея хорошая. Но почему может не работать стандартный?

[davidnum95]

dev400: не могу сказать.

Answer 2

[Максим Федоров]

Если речь идет о номере телефоне Вы сначала должны сохранять данные в базу приводя их к единому формату. Если же номере телефонов в базе уже находятся в одном формате, тогда можете использовать валидатор Unique

Comments

[dev400]

формат у них один. Пытаюсь записать номер который уже есть в базе, и получаю исключение SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '79097777777' for key 'user_phone_uindex', вместо сообщения которое я задал в правилах

[Максим Федоров]

dev400: Странно, у Вас в модели случайно сценарии и безопасные атрибуты не реализованы?

[dev400]

Максим Федоров: неа

<?php
namespace app\models;

use app\components\NewUserEvent;
use app\components\Notifications;
use yii\base\Model;
use app\models\User;

/**
 * Signup form
 */
class Register extends Model {

    const EVENT_NEW_USER = 'new-user';

    public $username;
    public $email;
    public $phone;
    public $password;
    public $password_repeat;
    public $verifyCode;


    /**
     * @inheritdoc
     */
    public function rules()
    {

        return [

            ['phone', 'trim'],
            ['phone', 'required'],
            ['phone', 'string', 'min' => 10],
            ['phone', 'unique', 'targetClass' => '\app\models\User', 'message' => 'Телефон занят'],

            ['username', 'trim'],
            ['username', 'required'],
            ['username', 'unique', 'targetClass' => '\app\models\User', 'message' => 'Логин занят'],
            ['username', 'string', 'min' => 2, 'max' => 255],

            ['email', 'trim'],
            ['email', 'required'],
            ['email', 'email'],
            ['email', 'string', 'max' => 255],
            ['email', 'unique', 'targetClass' => '\app\models\User', 'message' => 'Email адрес занят'],

            ['password', 'required'],
            ['password_repeat', 'required'],
            ['password', 'string', 'min' => 6],
            ['password_repeat', 'compare', 'compareAttribute' => 'password'],



            ['verifyCode', 'captcha', 'message' => 'Код введен неверно'],

        ];

    }


    /**
     * @param \app\models\User $user
     * @param NewUserEvent $event
     * @return \app\models\User|null
     */
    public function register( User $user, NewUserEvent $event)
    {

        if ( !$this->validate() ) {

            return null;

        }

        $user->username = $this->username;
        $user->email = $this->email;
        $user->phone = str_replace(['+', '(', ')' , ' '], '', $this->phone);
        $user->setPassword($this->password);
        $user->generateAuthKey();

        if( $user->save() ) {

            $event->username = $user->username;
            $event->phone =  $user->phone;
            $event->email = $user->email;
            $event->password = $this->password;

            $this->trigger(self::EVENT_NEW_USER, $event);

            return $user;

        }

        return null;

    }

}

[Максим Федоров]

dev400: а в каком формате вводиться номер? можно пример номера?

[dev400]

Максим Федоров: кажется я кое что понял... Спасибо

[dev400]

Максим Федоров: [phone]. Проблема кажется локализована, дело в $user->phone = str_replace(['+', '(', ')' , ' '], '', $this->phone);

[Максим Федоров]

dev400: Вы по всей видимости изменяете формат телефона после того как срабатывает валидатор....

[dev400]

Максим Федоров: да. Но тогда нужно как то до отправки в модель сделать из [phone] это 79097777777

[Максим Федоров]

Максим Федоров: тогда понятно. Вам приходит телефон [phone] валидатор ищет его в базе и не находит, соответственно считает данные валидными. Потом происходит сохранение, телефон приводиться к виду 79097777777, а телефон в таком формате уже есть... Как выход - написать свой валидатор который нормализует данные, т.е. приводит телефон к нужному виду в процессе самой валидации, перед сохранением

[Максим Федоров]

dev400: как реализовать такой "валидатор" понятно?

[dev400]

Максим Федоров: да, попробую по примеру в ответе davidnum95

[Максим Федоров]

dev400: только учтите что этот валидатор должен быть подключен до валидатора на уникальность

[dev400]

Максим Федоров: либо декорирую beforeValidate()

public function beforeValidate()
    {
       parent::beforeValidate();
        ////
    }

[dev400]

Максим Федоров: в общем то сделал так

public function validatePhone($attribute, $params)
    {

        $this->$attribute = str_replace(['+7', '(', ')' , ' '], '', $this->$attribute);

    }

Не знаю как со стороны лучших практик, но работает

[davidnum95]

dev400: со стороны лучших практик нужно на клиенте формировать тело запроса так чтобы бэкенд понял что имелось в виду