Как сохранять relation's в одной форме?

Question

[Vasiliy_M]

Есть сущность типа Post, есть теги к посту Tag хранящиеся через таблицу связей.

В форме поста ввел мультиселект для выбора тегов:

<?= $form->field($model, 'tags')->dropDownList(
        ArrayHelper::map(
            Tag::find()->all(), 'id', 'title'
        ), [
            'multiple' => true
        ]
    ) ?>

В модели Post:

public function afterSave($insert, $changedAttributes)
    {
        Tag2post::deleteAll(['post_id' => $this->id]);

        if (!empty($this->tags)) {
            foreach ($this->tags as $tag) {
                if (($tag = Tag::findOne($tag)) !== null) {
                    $relation = new Tag2post();
                    $relation->post_id = $this->id;
                    $relation->tag_id = $tag->id;
                    $relation->save();
                }
            }
        }

        parent::afterSave($insert, $changedAttributes);
    }

view:

<?= DetailView::widget([
        'model' => $model,
        'attributes' => [
            // ..
            [
                'attribute' => 'tags',
                'value' => function ($model) {
                    $list = [];
                    foreach ($model->getTags()->all() as $tag) {
                        $list[] = $tag->title;
                    }
                    return empty($list) ? 'теги не указаны' : implode(', ', $list);
                },
            ]
        ],
    ]) ?>

когда происходит ошибка формы при Create (например, не пропускает валидатор post-запрос), нужно сделать так, что бы в select-списке были selected все выбранные пользователем option's. Как этого добиться?

Comments

[Модератор]

Vasiliy_M дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац).
В противном случае, ваши вопросы будут удаляться по причине тег-спама.

Answer 1

[Евгений Бухарев]

Вам нужно добиться того, чтобы в $model->tags были ранее выбранные теги. Тогда дропдаун подхватит их и отметит отмеченными. Для сохранения связанных данных лучше использовать коробочный метод $model->link(). В официальной документации есть информация на этот счет. Также сохранение из afterSave лучше перенести туда где собственно сохранение происходит, в метод контроллера. Отправка запросов в цикле = зло

Comments

[Vasiliy_M]

Для сохранения связанных данных лучше использовать коробочный метод $model->link(). В официальной документации есть информация на этот счет

СПС! поправил:

public function afterSave($insert, $changedAttributes)
    {
        $this->unlinkAll('tags', true);

        if (!empty($this->tags)) {
            foreach ($this->tags as $tag) {
                if (($tag = Tag::findOne($tag)) !== null) {
                    $this->link('tags', $tag);
                }
            }
        }

        parent::afterSave($insert, $changedAttributes);
    }

Вам нужно добиться того, чтобы в $model->tags были ранее выбранные теги. Тогда дропдаун подхватит их и отметит отмеченными.

- я так понимаю, что это какой-то костыль я горожу, да?
поскольку getTags имеет следующий вид

public function getTags()
{
    return $this->hasMany(Tag::className(), ['id' => 'tag_id'])->viaTable('tag2post', ['post_id' => 'id']);
}

то я сомневаюсь, что это вообще возможно.. и нужно ли так вообще делать? Может связывать теги через аякс или в отдельном экшене?

И да - я пока учусь yii

[Евгений Бухарев]

Ситуация в следующем, свойство tags у вас отрабатывает через getTags, тоесть пользователь у вас выбрал теги, отправил на сервер, там допустим не прошла валидация, возвращается вьюшка, а там Вы запрашиваете getTags после чего летит запрос в базу и ищет те теги которые записаны в базе к данному посту, и соответственно не находит их. Такую ситуацию можно решить путем ввода свойства к модели поста, в котором будут храниться выбранные пользователем теги, допустим userSelectedTags, и в дропдауне использовать его, и далее проверять на наличие данных в нем, и соотвественно производить запись, единственно нужно будет поместить туда ранее записаные теги чтобы выводить выбраные теги из базы при генерации дропдауна