Почему так сохраняется в базе?

Question

[Сергей]

есть таблица Product с одним из столбцов tied предполагается что туда будут заносится связаные товары вот таким образом установлено его заполнение в виде

<?php 
        $product=Product::find()->where('category_id='.$cat->id)->all();
                foreach($product as $prod){ ?>
                    <?= $form->field($model, 'tied['.$prod->title.']')->checkbox(['value' => $prod->title, 'label' => $prod->title])
                    }
            ?>

вот часть экшена который отвечает за сохранение в базу

if ($model->load(Yii::$app->request->post())) {
                $post=Yii::$app->request->post();
                foreach($post ["Product"]["tied"] as $tied){
                    $model->tied .=$tied."%";
                }
                $model->save();
           }

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

Answer 1

[Антон Натаров]

Я так понимаю, что у вас один класс используется т.е. только Продукт?
Если да, то массовая работа и обновления проводятся так.

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

public function actionUpdate()
    {
        // Тут получаете свой cat либо передаете каким либо образом. 
        $products = Product::find()->where('category_id'= $cat->id)->indexBy('id')->all();

        if (Model::loadMultiple($products, Yii::$app->request->post()) && Model::validateMultiple($products)) {
            foreach ($products as $product) {
                $product->save(false);
            }
            return $this->redirect('index');
        }
        return $this->render('update', ['products' => $products]);
    }

В виде

$form = ActiveForm::begin();

foreach ($products as $index => $product) {
    echo $form->field($product, "[$index]value")->checkbox(['value' => $prod->title, 'label' => $prod->title]);
}

ActiveForm::end();

Если вы используете не одну модель и связанные вещи т.е. к пример как User и Profile, то вам нужна Работа с несколькими моделями

К тому же, если вы работаете со свзями, можно использовать жадную загрузку. подгружая и связи таким образом

//получаете продукты и связи за один запрос.
$products = Product::find()->with('category', 'someTied')->all();

//или с условием
$products = Product::find()->with([
    'categpry' => function ($query) {
        $query->andWhere(['category_id' => 15]);
    },
])->all();

Если вам нужно сохранить или обновить именно связные Id у разных объектов, есть метод link.

Comments

[Сергей]

сразу несколько вопросов

[Сергей]

1 зачем это у меня форма не подрозумевает загрузку файлов Model::loadMultiple и если ставлю то выдает ошибку reset() expects parameter 1 to be array, null given 2 вот этот параметр $cat->id у меня находится в view и получается при обработке foreach в третьих это зачем $product->save(false);?

[Антон Натаров]

1)Model::loadMultiple - это лишь вид загрузки моделей и тут он принимает его как массив моделей. Тоже самое и с валидатором. Ошибка означает, что он сбросил операцию поскольку ожидает массив, а получает Null.

2)Значит при отправке формы он у вас уже есть, нужно тогда вар дампом глянуть какая у него вложенность и вытащить.

3)Метод $model-save() Делает вставку если объект был создан через new, если нет и объект заполнен, он его обновляет. Установка в ложь, означает, что валидация сохраняемых данных не будет проводится. т.к. ранее валидация уже была.

[Сергей]

1 так у меня же не массив моделей зачем мне ?
2) мне легче в виде его получать понимаю что не комильфо но всеже 3) по save() согласен но почему же он тогда не обновляется а добавляется?

[Антон Натаров]

Сергей: Я понял.
1) Этот метод и делает из объектов массив. Ладно это не важно.

вместо сейва в своем коде попробуйте напрямую использовать так
$model->update();

[Сергей]

Антон Натаров: Я пробовал вывалилась ошибка пришлось просто обнулить $model->tied и в контролерре и в моделе