Как в одном контроллере сделать форму для добавление в другую таблицу бд?

Question

[hollanditkzn]

У меня имеется основная где хранятся заказы и в полном просмотре заказа, хранятся в таблице Zakaz, на этой странице имеется кнопка доставка. при нажатие на нее открывается модальное окно. В модальном окне есть форма, которая должно сохранять данные в таблицу Courier.
Пример я набросал как описано в документации https://yiiframework.com.ua/ru/doc/guide/2/input-m...
Вот в контроллере как организовал

public function actionView($id)
    {
        $model = $this->findModel($id);
        $shipping = $this->findShipping($id);

        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

        if ($model->load(Yii::$app->request->post())) {
            $model->file = UploadedFile::getInstance($model, 'file');
            if(isset($model->file))
            {
            $model->file->saveAs('maket/Maket_'.$model->id_zakaz.'.'.$model->file->extension);
            $model->maket = 'Maket_'.$model->id_zakaz.'.'.$model->file->extension;
            $model->status = 4;
            }            
            $model->save();

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        } 
        return $this->render('view', [
            'model' => $this->findModel($id),
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }
protected function findModel($id)
    {
        if (($model = Zakaz::findOne($id)) !== null) {
            return $model;
        } else {
            throw new NotFoundHttpException('The requested page does not exist.');
        }
    }
    protected function findShipping($id)
    {
        if (($shipping = Courier::findOne($id)) !== null) {
            return $shipping;
        } else {
            throw new NotFoundHttpException("The requested page does not exist.");
            
        }
    }

В zakaz/view находится вот такой код

<?php Modal::begin([
            'header' => '<h2>Доставка<h2>',
            'size' => 'modal-lg',
            'toggleButton' => [
                'tag' => 'button',
                'class' => 'btn btn-info btn-sm',
                'label' => 'Доставка',
            ],
        ]);


        echo $this->render('shipping', ['shipping' => $shipping]); 

        Modal::end(); ?>

И питался в файле shipping

<?php 

use yii\helpers\Html;
use yii\widgets\ActiveForm;
?>

<div class="zakaz-shippingForm">
	<?php $f = ActiveForm::begin(); ?>

	<?= $f->field($shipping, 'id_zakaz')->textInput() ?>
	<?php ActiveForm::end(); ?>
</div>

У меня сейчас выходит страницу с ошибкой
Not Found (#404)
The requested page does not exist
Как правильно написать и где я допустил ошибку?
Простой способ конечно это на отдельную страницу вывести эту форму, но хотелось бы конечно именно в модалке сделать

Answer 1

[Максим Тимофеев]

Ад какой-то, как по коду так и описание вопроса. Давайте начнем с самого вероятного:

$model = $this->findModel($id);
$shipping = $this->findShipping($id);

Сильно сомневаюсь что и там и там одинаковый id. Как следствие одна из этих строк кидает:

throw new NotFoundHttpException("The requested page does not exist.");

Comments

[hollanditkzn]

Максим Тимофеев Это ведет вот этот код findShipping($id). Значит мне убрать $id там или как то по другому реализовать?

[Максим Тимофеев]

Я вижу куда это ведет поэтому и спрашиваю. У Вас что в Courier и в Zakaz одинаковый id? И если эти модели связаны, то почему не пользуетесь связью, а делаете два запроса?

[hollanditkzn]

Максим Тимофеев: В Courier

public function rules()
    {
        return [
            [['id_zakaz'], 'integer'],
            [['data_to', 'data_from'], 'safe'],
            [['commit'], 'string'],
            [['to', 'from'], 'string', 'max' => 50],
            [['id_zakaz'], 'exist', 'skipOnError' => true, 'targetClass' => Zakaz::className(), 'targetAttribute' => ['id_zakaz' => 'id_zakaz']],
        ];
    }
public function getIdZakaz()
    {
        return $this->hasOne(Zakaz::className(), ['id_zakaz' => 'id_zakaz']);
    }

    /**
     * @return \yii\db\ActiveQuery
     */
    public function getZakazs()
    {
        return $this->hasMany(Zakaz::className(), ['id_shipping' => 'id']);
    }

А в Zakaz

public function rules()
    {
        return [
            [['srok', 'minut', 'number', 'description', 'phone'], 'required'],
            [['id_zakaz', 'id_tovar', 'oplata', 'fact_oplata', 'number', 'status', 'action', 'id_sotrud', 'phone', 'id_shipping'], 'integer'],
            [['srok', 'minut', 'data'], 'safe'],
            [['comment'], 'string'],
            [['prioritet'], 'string', 'max' => 36],

            ['status', 'default', 'value' => self::STATUS_NEW],
            ['id_sotrud', 'default', 'value' => Yii::$app->user->getId()],
            ['data', 'default', 'value' => date('Y-m-d')],

            [['description', 'information'], 'string', 'max' => 500],
            [['email', 'name', 'img', 'maket'],'string', 'max' => 50],
            [['file'], 'file', 'skipOnEmpty' => true],
            [['id_sotrud'], 'exist', 'skipOnError' => true, 'targetClass' => User::className(), 'targetAttribute' => ['id_sotrud' => 'id']],
            [['id_tovar'], 'exist', 'skipOnError' => true, 'targetClass' => Tovar::className(), 'targetAttribute' => ['id_tovar' => 'id']],
            [['id_shipping'], 'exist', 'skipOnError' => true, 'targetClass' => Courier::className(), 'targetAttribute' => ['id_shipping' => 'id']],
        ];
    }
public function getIdShipping()
    {
        return $this->hasOne(Courier::className(), ['id' => 'id_shipping']);
    }

[hollanditkzn]

Максим Тимофеев: типо вот из этой серии связанных модели Правильная генерация ActiveForm из связанных моделей в Yii2?

[Максим Тимофеев]

Я не знаю id чего Вам приходит в экшен, но если это id для Zakaz и в нем есть связь

public function getIdShipping()
    {
        return $this->hasOne(Courier::className(), ['id' => 'id_shipping']);
    }

То вместо

$model = $this->findModel($id);
$shipping = $this->findShipping($id);

должно быть

$model = $this->findModel($id);
$shipping = $model->idShipping;

или хотя бы

$model = $this->findModel($id);
$shipping = $this->findShipping($model->idShipping->id); //но это странно, привел для понимания сущности