Yii2: показать удобные для пользователя ошибки проверки при использовании try catch с транзакциями - PullRequest
0 голосов
/ 11 мая 2018

Я использую бутстрап ActiveForm.Вот моя форма:

use yii\helpers\Html;
use yii\bootstrap\ActiveForm;
use kartik\file\FileInput;

/* @var $this yii\web\View */
/* @var $model common\models\Customer */
/* @var $form yii\widgets\ActiveForm */
?>
<?php $form = ActiveForm::begin(['options' => ['enctype' => 'multipart/form-data'],
    'id' => 'customer-form',
    'enableClientValidation' => true,
    'options' => [
        'validateOnSubmit' => true,
        'class' => 'form'
    ],
    'layout' => 'horizontal',
    'fieldConfig' => [
      'horizontalCssClasses' => [
          'label' => 'col-sm-4',
         // 'offset' => 'col-sm-offset-2',
          'wrapper' => 'col-sm-8',
      ],
    ],
]); ?>
<?= $form->field($model, 'email')->textInput(['maxlength' => true]) ?>
<?php ActiveForm::end(); ?>

Вот моя модель:

class Customer extends \yii\db\ActiveRecord
{
    public $username;
    public $password;
    public $status;
    public $email;
    public $uploads;


    public function rules()
    {
        return [
            [['user_id', 'created_by', 'updated_by'], 'integer'],
            [['created_at','uploads', 'updated_at','legacy_customer_id','fax','phone_two','trn'], 'safe'],
            [['company_name','customer_name','username','password', 'tax_id'], 'string', 'max' => 200],
            [['customer_name','email','legacy_customer_id','company_name','city'], 'required'],
            [['is_deleted','status'], 'boolean'],
            [['address_line_1', 'state','phone', 'country'], 'string', 'max' => 450],
            [['address_line_2', 'city', 'zip_code'], 'string', 'max' => 45],
            [['user_id','legacy_customer_id'], 'unique'],
            ['email', 'email'],
            [['uploads'], 'file',  'maxFiles' => 10],
            [['email'], 'unique', 'skipOnError' => true, 'targetClass' => User::className(), 'targetAttribute' => ['email' => 'email'], 'message' => 'This email address has already been taken.'],
            [['user_id'], 'exist', 'skipOnError' => true, 'targetClass' => User::className(), 'targetAttribute' => ['user_id' => 'id']],
        ];
    }
}

Вот контроллер

 public function actionCreate() {
        $model = new Customer();
        if ($model->load(Yii::$app->request->post())) {
            $transaction = Yii::$app->db->beginTransaction();
            try 
            {
            $user_create = \common\models\User::customeruser($model);
            if ($user_create) {
                $model->user_id = $user_create->id;
                $auth = \Yii::$app->authManager;
                $role = $auth->getRole('customer');
                $auth->assign($role, $model->user_id);
            }
            if ($user_create && $model->save()) {

                $photo = UploadedFile::getInstances($model, 'uploads');
                if ($photo !== null) {
                    $save_images = \common\models\CustomerDocuments::save_document($model->user_id, $photo);
                }
                $transaction->commit();  
                return $this->redirect(['view', 'id' => $model->user_id]);
            }
        }catch (Exception $e) 
        {
          $transaction->rollBack();
        }
    }

        if (Yii::$app->request->isAjax) {
            return $this->renderAjax('create', [
                        'model' => $model,
            ]);
        } else {
            return $this->render('create', [
                        'model' => $model,
            ]);
        }
    }

Теперь обязательный атрибутрабочая используется в правилах.Он не позволяет отправлять форму до тех пор, пока обязательное поле не заполнено каким-либо значением, но в то же время уникальный атрибут, использующий целевой класс, не работает и позволяет отправлять форму.После нажатия на форму отправки форма не будет отправлена, но она не показывает ошибку уникальной проверки.С уважением, я использую форму в модале начальной загрузки, и я хочу, чтобы форма показала уникальную ошибку отправки перед отправкой, такую ​​же, как требуется рабочая.Я могу сделать это с помощью jQuery для функции размытия и отправить пользовательский запрос AJAX, но я хочу решение по умолчанию Yii 2.

EDIT

Это где ошибка выдается из-за не пользователяпри сохранении

public static function customeruser( $model ) {
    $user = new User();
    $user->username = $model->email;
    $user->email = $model->email;
    $user->setPassword ( $model->legacy_customer_id );
    $user->generateAuthKey ();
    if ( !$user->save () ) {
        var_dump ( $user->getErrors () );
        exit ();
    } return $user->save () ? $user : null;
}

var_dump() показывает следующее

'username' => array (size = 1) 0 => string 'Это имя пользователя уже занято.'(length = 37) 'email' => array (size = 1) 0 => string 'Этот адрес электронной почты уже занят.'(длина = 42).

Ответы [ 2 ]

0 голосов
/ 14 мая 2018

Поскольку вы используете блок try catch вместе с транзакцией, вы должны генерировать и перехватывать такие ошибки как исключение, чтобы транзакция откатывалась, и сообщение также отображалось для пользователя.

Вы не используете или не используете красоту блока try{}catch(){} с транзакциями.Вы должны всегда бросать Exception в случае, если какая-либо из моделей не сохранена и блок catch откатит транзакцию.

Например, вы сохраняете пользователя в функции customeruser(), вызывая

$user_create = \common\models\User::customeruser($model);

и возвращая объект user или null в противном случае, а затем в следующей строке вы проверяете, создан пользователь или нет.

if ($user_create) {

Вы должны просто броситьисключение из функции customeruser() в случае, если модель не была сохранена и в противном случае возвращается объект $user, вам не нужно снова проверять $user_create, чтобы убедиться, что пользователь не был сохранен, будет сгенерировано исключение, и управление будетбыть перенесены в блок catch и строки после $user_create = \common\models\User::customeruser($model); никогда не будут вызываться.

В основном я использую следующий способ, когда у меня есть несколько моделей для сохранения, и я использую блок транзакции.

$transaction = Yii::$app->db->beginTransaction ();
try {
    if ( !$modelUser->save () ) {
        throw new \Exception ( implode ( "<br />" , \yii\helpers\ArrayHelper::getColumn ( $modelUser->errors , 0 , false ) ) );
    }
    if ( !$modelProfile->save () ) {
        throw new \Exception ( implode ( "<br />" , \yii\helpers\ArrayHelper::getColumn ( $modelProfile->errors , 0 , false ) ) );
    }
    $transaction->commit();
} catch ( \Exception $ex ) {
    $transaction->rollBack();
    Yii::$app->session->setFlash ( 'error' , $ex->getMessage () );
}

Таким образом, вы можете сделать то же самое для вашего кода

public static function customeruser( $model ) {
    $user = new User();
    $user->username = $model->email;
    $user->email = $model->email;
    $user->setPassword ( $model->legacy_customer_id );
    $user->generateAuthKey ();
    if(!$user->save()){
        throw new \Exception ( implode ( "<br />" , \yii\helpers\ArrayHelper::getColumn ( $user->errors , 0 , false ) ) );
    }
    return $user;
}

изменить actionCreate на следующий

public function actionCreate() {
    $model = new Customer();
    if ( $model->load ( Yii::$app->request->post () ) ) {
        $transaction = Yii::$app->db->beginTransaction ();
        try {
            $user_create = \common\models\User::customeruser ( $model );
            $model->user_id = $user_create->id;
            $auth = \Yii::$app->authManager;
            $role = $auth->getRole ( 'customer' );
            $auth->assign ( $role , $model->user_id );

            if ( !$model->save () ) {
                throw new \Exception ( implode ( "<br />" , \yii\helpers\ArrayHelper::getColumn ( $model->errors , 0 , false ) ) );
            }

            $photo = UploadedFile::getInstances ( $model , 'uploads' );
            if ( $photo !== null ) {
                $save_images = \common\models\CustomerDocuments::save_document ( $model->user_id , $photo );
            }

            $transaction->commit ();
            return $this->redirect ( [ 'view' , 'id' => $model->user_id ] );
        } catch ( \Exception $ex ) {
            Yii::$app->session->setFlash ( 'error' , $ex->getMessage () );
            $transaction->rollBack ();
        }
    }

    if ( Yii::$app->request->isAjax ) {
        return $this->renderAjax ( 'create' , [
                    'model' => $model ,
                ] );
    }

    return $this->render ( 'create' , [
                'model' => $model ,
            ] );
}
0 голосов
/ 14 мая 2018

Есть две три вещи:

1. Вы не принимаете адрес электронной почты при вводе пользователем.

2. Будет вставлена ​​пустая запись, если в базе данных ваше поле будет взято как NULL

3. Во второй вставке будет показано сообщение об ошибке, если поле db не NUll .

Решение: 1. Сделать поле БД Не нулевым.

2. Принять электронную почту как пользовательский ввод.

3. Попробуйте напечатать ошибку

if ($model->validate()) {
    // all inputs are valid
} else {
    // validation failed: $errors is an array containing error messages
    $errors = $model->errors;
}

4. удалить 'skipOnError' => true,

Попробуйте выше Я уверен, что вы получите решение

Help Full LINK Для проверки

...