Почему в yii2 в контролере в поле куда записывается id через цикл, модель сохранение выдает ошибку что это массив?

Question

[AlexAll]

Вот такой контроле, почему то эта строчка $item->order_id = $model->id; выдает при сохранении ошибку что это array, хотя если вардампить $item->order_id или $model->id выдает обычное число int

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

public function actionOrder() {

        $model = new Order();
        $item = new OrderItem();
        $cart = $this->service->getCart();
        if ($model->load(Yii::$app->request->post())) {
            $model->status = 'on';
            if ($model->save()) {
                $items = $this->cart->getItems();
                foreach ($items as $product) {
                    $item->order_id = $model->id;
                    $item->item_id = $product->getProductId();
                    $item->quantity = $product->getQuantity();
                    $item->item_price = $product->getCost();
                      if(!$item->save()){
                        var_dump($item->errors);
                        die;
                    }
                }

            }
        }
        return $this->redirect(['index']);
    }

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

Comments

[padlyuck]

а вы где вардампили $item->order_id или $model->id? и еще, вас не смущает, что вы в итоге получите ордер с только одним ордеритемом, даже если в корзине будет несколько товаров лежать?

Answer 1

[Дмитрий]

Добрый день.
Может так будет правильней?

foreach ($items as $product) {
                    $item = new OrderItem();
                    $item->order_id = $model->id;
                    $item->item_id = $product->getProductId();
                    $item->quantity = $product->getQuantity();
                    $item->item_price = $product->getCost();
                      if(!$item->save()){
                        var_dump($item->errors);
                        die;
                    }
                }

И не мешало бы показать полный текст ошибки.

Comments

[AlexAll]

И так я пробовал, все равно выдает array(1) { ["order_id"]=> array(1) { [0]=> string(45) "Значение «Order ID» неверно." } }

[Дмитрий]

AlexAll, но так будет правильно. Почему - уже написал Максим Тимофеев.
Проверяйте, что у Вас в $model->id;

[padlyuck]

AlexAll, ну так а при чем здесь order_id array? у вас что-то не так с самим order_id на какой валидатор это сообщение навешено?

[Дмитрий]

AlexAll, эту строку из валидатора уберите

[['order_id'], 'exist', 'skipOnError' => true, 'targetClass' => Order::className(), 'targetAttribute' => ['order_id' => 'id']],

[AlexAll]

Все отбой я нашел ошибку, спасибo всем,

Ошибка вот в чем, когда я делал миграцию OrderItem я по ошибке FK сделал на item

и тут правило работало для Item

[['order_id'], 'exist', 'skipOnError' => true, 'targetClass' => Item::className(), 'targetAttribute' => ['order_id' => 'id']],

Ошибку в базе заметил и поменял FK а в валидации забыл

ПС Вы мне не раз помогали уже если есть возможность посмотрите этот вопрос, давно голову ломаю Как коректно вставить плагин \kartik\select2\Select2 в плагин unclead\multipleinput\MultipleInput в yii2? спс

[Дмитрий]

AlexAll, возможно, что плагин MultipleInput не рассчитан на подключение Select2 от kartic.
По крайней мере, в документации, я ничего не увидел.
Или Select2 подключается, но не выводятся данные?

[AlexAll]

slo_nik, сам Select2 работает, находит то что нужно, я сделал и сохранение все сохраняет, только почему то пустые поля сохраняет, это первое а второе там с выводом сохранения, танцы с бубнами, через метод init я получаю сохраненные данные из БД а точнее id моделей и получаю их title, а потом их компанную в масиив и передаю в MultipleInput и он вроде выводит, но беда в том что в value выводится текст а не id.

[Дмитрий]

AlexAll, ну тут надо смотреть как Вы формируете данные для select2.

[AlexAll]

slo_nik, вообще по идее он получает id и находит его title вот так

'initValueText' => empty($model->items) ? '' : Item::findOne($model->items)->title,

А я передаю в MultipleInput вот этот массив

public $items;
    public function init()
    {
        parent::init();

        $title = [];
        foreach ($this->itemall as $item){

           $title[]['slide'] =  empty($item['item_id']) ? '' : Item::findOne($item['item_id'])->title;
 
        }

        $this->items = $title;

    }

    public function getItemAll()
    {
        return self::find()->orderBy(['id' => SORT_ASC])->asArray()->all();
    }

В массиве уже найдены title и передаю название
А хотелась бы сюда передать id

'initValueText' => empty($model->items) ? '' : Item::findOne($model->items)->title,

но получается, так как $model->items это массив

[Дмитрий]

AlexAll, по моему Вы всё усложнили.
Сделайте статический метод и результат работы попробуйте передать в select2

public static function getAllModels()
    {
        return ArrayHelper::map(self::find()->orderBy(['name' => SORT_ASC])->all(), 'id_car_model', 'name');
    }

В выпадающий список передаёте так

<?= $form->field($model, 'model')->dropDownList(Model::getAllModels()) ?>

[Дмитрий]

AlexAll, это пример, у Вас работать не будет...

[AlexAll]

slo_nik, Я все делал по инструкции https://github.com/unclead/yii2-multiple-input

вот так у меня вывод виджета в котором selekt 2

<?php

        echo $form->field($model, 'items')->widget(MultipleInput::className(), [

            'columns' => [
                [
                    'name' => 'slide',
                    'type' => \kartik\select2\Select2::className(),
                    'options' => [
                        'options' => ['placeholder' => 'Поиск  ...'],
//                       'initValueText' => empty($model->items) ? '' : Item::findOne($model->items)->title,
                       'pluginOptions' => [   
                            'allowClear' => true,
                            'minimumInputLength' => 3,
                            'language' => 'ru',
                            'ajax' => [
                                'url' => \yii\helpers\Url::to(['item-list']),
                                'dataType' => 'json',
                                'data' => new JsExpression('function(params) { return {q:params.term}; }')
                            ],
                            'escapeMarkup' => new JsExpression('function (markup) { return markup; }'),
                            'templateResult' => new JsExpression('function(item_id) { return item_id.text; }'),
                            'templateSelection' => new JsExpression('function (item_id) { return item_id.text; }'),
                        ],
                    ],
                ],
      
            ]
        ])->label(false);
        ?>

Вот экшен сохранения

*/
    public function actionIndex()
    {
        $model = new Slider();

        if ($model->load(Yii::$app->request->post())) {

            foreach ($model->items['slide'] as $id){
                
                $model = new Slider();
                    $model->item_id = $id;
                     $model->save();


            }
       Yii::$app->session->setFlash('success', "Слайдер сохранен");
            return $this->redirect(['index']);
        }

        return $this->render('index', [
                    'model' => $model,
        ]);
    }

А вот вывод

public $items;
    public function init()
    {
        parent::init();

        $title = [];
        foreach ($this->itemall as $item){

           $title[]['slide'] =  empty($item['item_id']) ? '' : Item::findOne($item['item_id'])->title;
 
        }

        $this->items = $title;

    }

    public function getItemAll()
    {
        return self::find()->orderBy(['id' => SORT_ASC])->asArray()->all();
    }

[Дмитрий]

AlexAll, посмотрите, что Вам возвращает getItemAll(). И попробуйте сделать как я написал. Сравните результаты.

Answer 2

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

Для начала надо бы вместо

foreach ($items as $product) {
                    $item->order_id = $model->id;

сделать

foreach ($items as $product) {
                    $item = new OrderItem();
                    $item->order_id = $model->id;

так как вы одну и туже модель перезаписываете, ну и дайте код самой модели OrderItem и что выдает var_dump

Comments

[AlexAll]

Все отбой я нашел ошибку, спасибo всем,

Ошибка вот в чем, когда я делал миграцию OrderItem я по ошибке FK сделал на item

и тут правило работало для Item

[['order_id'], 'exist', 'skipOnError' => true, 'targetClass' => Item::className(), 'targetAttribute' => ['order_id' => 'id']],

Ошибку в базе заметил и поменял FK а в валидации забыл