Как лучше сделать глобальную форму?

Question

[Максим]

Хочу сделать форму в модальном окне. Она будет присутствовать на каждой странице и вызываться по id. Данные отправляются на почту через ActiveForm. Я поместил вот такой код в "layouts/main", то есть в футер страницы, таким образом у меня форма присутствует на каждой странице. Скажите, на сколько это правильное решение? Или есть более правильный метод. Поделитесь...

<?php
$model = new ContactForm();
Modal::begin([
    "size"=>"modal-sm",
    "options"=>[
        "id"=>"leads",
    ],
    'header' => '<h2>Запись на занятие</h2>',
]);?>
<div class="row">
    <div class="col-lg-12">
        <?php $form = ActiveForm::begin(['id' => 'contact-form']); ?>

        <?= $form->field($model, 'name')->textInput(['autofocus' => true]) ?>

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

        <?= $form->field($model, 'subject') ?>

        <?= $form->field($model, 'body')->textarea(['rows' => 6]) ?>

        <?= $form->field($model, 'verifyCode')->widget(Captcha::className(), [
            'template' => '<div class="row"><div class="col-lg-3">{image}</div><div class="col-lg-9">{input}</div></div>',
        ]) ?>

        <div class="form-group">
            <?= Html::submitButton( Yii::t('frontend','Submit'), ['class' => 'btn btn-primary', 'name' => 'contact-button']) ?>
        </div>

        <?php ActiveForm::end(); ?>
    </div>
</div>

<?php Modal::end();?>

Comments

[Дмитрий Ким]

У вас экшен формы не определен, а так юзабельно.

Answer 1

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

Как по мне приведенное Ваши решение не правильное, потому что:
1. Создание модели (пусть даже для формы) в представлении это ни разу не тру. Представление должно получать модель, а не создавать ее. Иначе это уже не MVC.
2. Наличие формы на каждой странице (а соответственно и всех html, js) только увеличивает размер страницы и уменьшает ее скорость загрузки. Это не оправдано с учетом того что пользователь может этой формой вообще не воспользоваться, да и открывать ее на каждой странице, в процессе работы с сайтом, он ее не будет.

Гораздо оптимальнее сделать подгрузку формы при открытии модалки (из коробки бутстраповская модалка позволяет это сделать)

Comments

[Максим]

Можете подсказать как это более правильно сделать? Не совсем понял о чём идет речь..

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

Максим Ворожцов,
1. Делаете экшен в каком-нибудь контроллере который рендерит форму
2. При открытии модалки обращаетесь к этому экшену и отображаете форму в модалке

[Дмитрий Ким]

Представление должно получать модель, а не создавать ее.

Делаете экшен в каком-нибудь контроллере который рендерит форму

Звучит довольно таки странно :-)

[Максим]

Максим Федоров, Сделал чего не хватает? Как подгужать форму из контроллера, чтобы срабатывало только при открытии?

Контроллер

public function actionContact()
    {
        $model = new ContactForm();
        if ($model->load(Yii::$app->request->post()) && $model->validate()) {
            if ($model->sendEmail(Yii::$app->params['adminEmail'])) {

                Yii::$app->session->setFlash('success', 'Спасибо, что связались с нами. Мы ответим Вам в кратчайшие сроки!');
            } else {
                Yii::$app->session->setFlash('error', 'Произошла ошибка при отправке сообщения.');
            }

            return $this->refresh();
        } else {
            return $this->render('contact', [
                'model' => $model,
            ]);
        }
    }

В представлении сделал модальное окно

<?php
Modal::begin([
    "size"=>"modal-sm",
    "options"=>[
        "id"=>"leads",
    ],
    'header' => '<h2>Запись на занятие</h2>',
]);?>


<?php Modal::end();?>

Модель

<?php

namespace frontend\models;

use Yii;
use yii\base\Model;

/**
 * ContactForm is the model behind the contact form.
 */
class ContactForm extends Model
{
    public $name;
    public $phone;
    public $subject;
    public $body;
    public $verifyCode;


    /**
     * @inheritdoc
     */
    public function rules()
    {
        return [
            [['name', 'phone'], 'required'],
        ];
    }

    /**
     * @inheritdoc
     */
    public function attributeLabels()
    {
        return [
            'name' => Yii::t('frontend','Name'),
            'phone' => Yii::t('frontend','Phone'),
        ];
    }

    /**
     * Sends an email to the specified email address using the information collected by this model.
     *
     * @param string $email the target email address
     * @return bool whether the email was sent
     */
    public function sendEmail($email)
    {
        return Yii::$app->mailer->compose()
            ->setTo($email)
            ->setSubject("Заявка с сайта")
           ->setTextBody('Имя: '.$this->name.', Телефон: '.$this->phone)
            ->send();
    }
}

Форма views

<?php

/* @var $this yii\web\View */
/* @var $form yii\bootstrap\ActiveForm */
/* @var $model \frontend\models\ContactForm */

use yii\helpers\Html;
use yii\bootstrap\ActiveForm;
use yii\captcha\Captcha;
/*
$this->title = Yii::t('frontend','Contact');
$this->params['breadcrumbs'][] = $this->title;*/
?>
<div class="site-contact">
    <selector id="contact-forms">
        <div class="container">
            <div class="row">
                <div class="col-lg-12">
                    <?php $form = ActiveForm::begin(['id' => 'contact-form']); ?>

                    <?= $form->field($model, 'name')->textInput(['autofocus' => true]) ?>

                    <?= $form->field($model, 'phone') ?>

                    <div class="form-group">
                        <?= Html::submitButton( Yii::t('frontend','Submit'), ['class' => 'btn btn-primary', 'name' => 'contact-button']) ?>
                    </div>

                    <?php ActiveForm::end(); ?>
                </div>
            </div>
        </div>
    </selector>

</div>

Answer 2

[Crash]

В целом верно, можно также сделать виджетом, чтобы не загромождать шаблон