Ошибка при выводе многомерного массива в цикле. Yii2?

Question

[MaikMain]

Всем, привет. Пытаюсь построить вывод в дропдауне название статусов с название груп, к которым они относятся.
Должно получится вот так:


У меня есть три таблицы: Orders. Здесь связь такая:

public function getOrderStatus(): ActiveQuery
    {
        return $this->hasOne(OrdersStatuses::class, ['id' => 'status_id']);
    }

В OrdersStatuses:

public function getGroup(): ActiveQuery
    {
        return $this->hasOne(OrderStatusGroups::class, ['id' => 'group_id']);
    }

    public function getOrders(): ActiveQuery
    {
        return $this->hasMany(Orders::class, ['status_id' => 'id']);
    }

В OrderStatusGroups:

public function getStatuses(): ActiveQuery
    {
        return $this->hasMany(OrdersStatuses::class, ['group_id' => 'id']);
    }

В экшине контроллера пишу так:

$orderStatuses = ShopOrdersStatuses::find()->all();
$statusList = [];

foreach ($orderStatuses as $orderStatus) {
   $statusList[$orderStatus->group->name][$orderStatus->id] = $orderStatus->name ;
}

$statusList = ArrayHelper::merge(['' => 'Все'], ArrayHelper::map($orderStatuses, 'id', 'name'));

В return:

return $this->render('update', [
            'model' => $model,
            'orderStatuses' => $statusList,
        ]);

Но выскакивает ошибка: Attempt to read property "name" on null
B ссылается на строчку в цикле.


В БД все данные есть, все записано, нигде null не стоит.
PHP 8.1

Подскажите пожалуйста где у меня ошибка и в какую сторону вообще копать?
За ранее большое спасибо =)

Comments

[Дмитрий]

Добрый вечер.
Ошибка говорит не о том, что где-то в базе null записан, а о том, что Вы в запросе не получили name, но пытаетесь его прочитать. Вот как-то так.

Сделайте print_r($orderStatuses) и Вы увидите, что group->name там нет.

В Вашем запросе надо указать, с какой связью надо выполнять запрос.

$orderStatuses = ShopOrdersStatuses::find()->with('имя связи')->all();

Возможно нужно будет создать связующую таблицу и оттуда получать нужные данные

https://www.yiiframework.com/doc/guide/2.0/ru/db-a...

[MaikMain]

Дмитрий, Спасибо за ответ. У меня имя связи getGroup, в with подставляю "group", Но ошибка вся та же самая

[Дмитрий]

MaikMain, где же это в запросе?

$orderStatuses = ShopOrdersStatuses::find()->all();

[MaikMain]

Дмитрий, уже после вашего ответа, я у себя попробовал вот так, и все равно ошибка сохраняется

$orderStatuses = ShopOrdersStatuses::find()->with('group')->all();

[Дмитрий]

MaikMain, ну а Вы смотрели, что приходит в $orderStatuses?
Есть там group->name?

Его там, скорей всего, нет. Значит надо изменить сам запрос.

with() это один из вариантов.

Вот видео, где рассказывается, как работать со связанными данными.

В документации, ссылку я давал выше, говорится ещё и о joinWith() и т.п.

[MaikMain]

Дмитрий, благодарю Вас за ответ. Буду разбираться)
А в print_r($orderStatuses) все данные есть, все что нужно туда попадает

Вот скрин того, что там находиться

[Дмитрий]

MaikMain, тогда такой вариант

foreach ($orderStatuses as $orderStatus) {
   $statusList[$orderStatuses->group->name][$orderStatus->id] = $orderStatus->name ;
}

$orderStatuses->group->name

[MaikMain]

Дмитрий, Ошибка на той же строчке, только теперь - Attempt to read property "group" on array.
Если через смотрю так:

foreach ($orderStatuses as $orderStatus) {
            print_r($statusList[$orderStatus->group->name]); 
            $statusList[$orderStatus->group->name][$orderStatus->id] = $orderStatus->name ;
        }

То ошибка Undefined array key "Новый"

А если так:

foreach ($orderStatuses as $orderStatus) {
            print_r($orderStatus->group->name); //или print_r($orderStatus->id); или $orderStatus->name
            $statusList[$orderStatus->group->name][$orderStatus->id] = $orderStatus->name ;
        }

То выводит только одну группу или один статус, который выбран.

А вот такое "аля дерево" не строит

[Дмитрий]

MaikMain, если у Вас в ответе все нужные данные, то попробуйте вложенный цикл. Обходите сначала группы, и для каждой группы делайте цикл для статусов

[MaikMain]

Дмитрий, Если Вам не сложно то можете помочь с написанием, а то я что-то сильно туплю и не понимаю как в моем случае это сделать. За ранее большое спасибо)

[Дмитрий]

MaikMain, проще xdebug поставить и смотреть в нём, что и в какой переменной находится без всяких print_r() )))

Что у Вас в $orderStatuses->group?

print_r($orderStatuses->group)

Если группы, то так

foreach ($orderStatuses->group as $name) {
      foreach () {
          // здесь из массива получаете статусы для группы, если они есть в ответе базы
      }
}

Если использовать print_r(), то делайте это пошагово

$orderStatuses = ShopOrdersStatuses::find()->all();
$statusList = [];

print_r($orderStatuses); exit(); // Если есть нужные данные, то комментируем этот код и идём дальше. Если нет - меняем запрос.

foreach ($orderStatuses as $orderStatus) {
//print_r($orderStatus); exit(); // Если есть нужные данные, то комментируем этот код и идём дальше. Если нет - меняем запрос.
   $statusList[$orderStatus->group->name][$orderStatus->id] = $orderStatus->name ;
}

Ну вот как-то так.

[MaikMain]

Дмитрий, Спасибо за помощь. В итоге нужно дерево построилось. Я удалил все таблицы, весь код, написал все заново и все заработало.

Но походу либо я дебил, либо уже голова не соображает.
Есть весь код в контроллере:

spoiler

public function actionUpdate($id)
    {
        $model = $this->findModel($id);

        $confirm = ArrayHelper::merge(['' => 'Все'], Orders::getConfirmStatuses());

        $orderStatuses = ShopOrdersStatuses::find()->all();

        $statusList = [];

        foreach ($orderStatuses as $orderStatus) {
            $statusList[$orderStatus->group->name][$orderStatus->id] = $orderStatus->name;
        }
        $statusList = ArrayHelper::merge(['' => 'Все'], ArrayHelper::map($orderStatuses, 'id', 'name'));

        $payments = ShopPaymentTypes::find()->all();

        if ($this->request->isPost && $model->load($this->request->post()) && $model->save()) {
            return $this->redirect(['view', 'id' => $model->id]);
        }
        return $this->render('update', [
            'model' => $model,
            'orderStatuses' => $statusList,
            'statuses' => $confirm,
            'payments' => $payments,
        ]);

    }

Во вьюхе _form вывожу вот так: <?= $form->field($model, 'status_id')->dropDownList($orderStatuses) ?>

Но выдается ошибка Undefined variable $orderStatuses
Я вроде сделал все правильно, все как всегда, но она чот не фурычит.

Можете пожалуйста крайний раз подсказать)

[Дмитрий]

MaikMain, дерево построилось, но код неверен, половина того, что сейчас в действии не должно быть в контроллере. Всё надо одним запросом делать, а сейчас у Вас костыльный костыль. И что-то не вижу валидации данных.
Смотрите видео, которое я указал, читайте блог автора видео, там много полезного по yii2.

По поводу undefined переменной ничего не могу сказать, по идее должна работать нормально. Можно прописать doc-block для переменной, инициализировать её в view.

Answer 1

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

У вас связь group для одного из статусов вернула null, а вы из этого null тяните свойство name.
Делайте проверку на null перед этим

foreach ($orderStatuses as $orderStatus) {
   $name = $orderStatus->group ? $orderStatus->group->name : 'TamaraGoniGusei';
   $statusList[$name][$orderStatus->id] = $orderStatus->name ;
}

Comments

[MaikMain]

Спасибо за ответ)

Answer 2

[pantsarny]

Вы в цикле идете по всем имеющимся в базе статусам, но не факт что для всех статусов есть хотя бы один заказ

https://www.yiiframework.com/doc/api/2.0/yii-widge...
А еще посмотрите как items можно группировать