Yii2 relations NULL Foreign key Как правильно делать CREATE/UPDATE?

Question

[Vladimir Klintsov]

Народ, такая беда:
Есть таблица groups
...
'file_id' => $this->integer(),
...
И есть таблица files
...

'id' => $this->primaryKey()->notNull()->unique(),
'filename' => $this->string(),

...

$this->addForeignKey(
            'file-id',
            'groups',
            'file_id',
            'files',
            'id',
            'CASCADE'
        );

Gii нагенерил:

class Groups extends \yii\db\ActiveRecord
{
...
[['file_id'], 'default', 'value' => null],
[['file_id'], 'integer'],
[['file_id'], 'exist', 'skipOnError' => true, 'targetClass' => Files::className(), 'targetAttribute' => ['file_id' => 'id']],
...
public function getFile()
{
    return $this->hasOne(Files::className(), ['id' => 'file_id']);
}
...
class Files extends \yii\db\ActiveRecord
{
...
public function getGroups()
{
    return $this->hasMany(Groups::className(), ['file_id' => 'id']);
}
...
public function actionMyaction($id)
{
...
$model = $this->findModel($id);  
...
return $this->render('Myaction', [
    'model' => $model,
]);

теперь я могу в форме Myaction при наличии отличного от NULL значения file_id таблицы Groups и соответствующего ему значения в таблице Files id получить значение filename:

<?= $form->field($model->file, 'filename')->textInput(); ?>

Беда начинается если в таблице Files нет соответствующего значеия
Call to a member function isAttributeRequired() on null

Предполагаю, что гдето должно быть New Files(), но вот только где?
Please Help!

Comments

[Дмитрий]

Добрый вечер.
file_id не обязательно к заполнению?

[Vladimir Klintsov]

Дмитрий, Добрый. Нет. Не обязательно

[Дмитрий]

Vladimir Klintsov, атрибут filename это в модели Files или Groups?

[Vladimir Klintsov]

Files

[Дмитрий]

Vladimir Klintsov, когда отвечаете на комментарий нажимайте "ответить", иначе будете долго ждать ответа)))
Решить можно по разному.
Можно так:

public function actionMyaction($id)
{
...
$model = $this->findModel($id);
$files = new Files();
...
return $this->render('Myaction', [
    'model' => $model,
    'files' => $files
]);
/************/
<?= $form->field($files, 'filename')->textInput([
    'value' => $model->file ? $model->file->filename : ''
]); ?>

Но это так себе способ, по быстрому, не особо заморачиваясь.

[Vladimir Klintsov]

Дмитрий, Да не... Это костыль. Потом начнутся проблемы как это сохранить, причём навскидку там можно наплодить кучу пустых строк в таблице Files. Лучше уже объявить в модели Groups публичную переменную $variable, затем заполнить её в форме

<?= $form->field($model, 'variable')->textInput(); ?>

и после стандартного

if ($model->load($request->post()) && $model->save(false)) {

сделать

$f = Files::findOne(['id' => $model->file_id]);
$f->filename = $model->variable;
$f->save();

И тогда совершенно незачем делать линк между таблицами - всё и так сработает. Должно быть какое-то штатное решение

[Дмитрий]

Vladimir Klintsov, так я Вам и написал, что это не лучший способ, который я предложил.
Но Вы предлагаете ещё больший костыль, гораздо костыльней)))
То что Вы делаете публичное свойство $variable(название, кстати, костыльное) - верно, но всё остальное нет.
Если хотите без костылей, то делайте отдельно модель для формы, желательно одну для create, вторую для update.
В действии контроллера передаёте данные с формы в класс, который будет создавать или изменять экземпляр модели и сохранять.
Вот тогда будет не костыльно, а очень даже приличный код, легко расширяемый и поддерживаемый)))

[Дмитрий]

Потом начнутся проблемы как это сохранить, причём навскидку там можно наплодить кучу пустых строк в таблице Files

Vladimir Klintsov, сложности будут, но если подумать, то пустых строк не будет в таблице Files. Да и с сохранением проблем никаких)))

[Vladimir Klintsov]

Дмитрий, Ну, если уже развивать дальше Ваш вариант костыля, то будет примерно так:

<?php if(isset($model->file->filename)){ ?>
<?= $form->field($model->file, 'filename')->textInput(); ?>
<?php }else{ ?>
<?= $form->field($files, 'filename')->textInput(); ?>
<?php } ?>

и сохранять:

if ($model->load($request->post())) {
$model->file_id = $files->id;
$model->save();
$files->save();

Но это всё равно громоздко. Не верится, что почти что самый древний фреймворк не содержит в себе решения такой простейшей задачи

[Vladimir Klintsov]

https://stackoverflow.com/questions/27271966/relat... нашёл что-то более-менее подходящее
Спасибо

[Дмитрий]

Vladimir Klintsov, для записи связанных данных можно использовать https://github.com/la-haute-societe/yii2-save-rela....
Подключаете в behaviors() модели и всё))). Вам достаточно будет вызвать $files->save() или что Вы будете сохранять, а связанные сохраняться автоматом.