Как избавиться от ошибки foreach?

Question

[hollanditkzn]

У меня такая проблема в шаблоне main.php у меня возникает проблема и выводит вот такая запись Invalid argument supplied for foreach(). Просто не понятно почему foreach является инвалидом. Скорее всего как понимаю это я использую в main. Но как реализовывать систему уведомление тогда стоит вопрос. Сам никогда не делал данные моменты и не могу понять как это сделать.

<div class="wrap">
    <?php
    NavBar::begin([
        'brandLabel' => 'Holland',
        'brandUrl' => ['/zakaz/index'],
        'options' => [
            'class' => 'navbar-inverse navbar-fixed-top',
        ],
    ]);
    if (!Yii::$app->user->isGuest) {
        $menuItems[] = ['encode' => false, 'label' => '<span class="glyphicon glyphicon-bell"></span>', 'options' => ['id' => 'notification']];
    }
    if (Yii::$app->user->isGuest) {
        // $menuItems[] = ['label' => 'Signup', 'url' => ['/site/signup']];
        $menuItems[] = ['label' => 'Войти', 'url' => ['/site/login']];
    } else {
        $menuItems[] = '<li>'
            . Html::beginForm(['/site/logout'], 'post')
            . Html::submitButton(
                'Выйти (' . Yii::$app->user->identity->username . ')',
                ['class' => 'btn btn-link logout']
            )
            . Html::endForm()
            . '</li>';
    }
    echo Nav::widget([
        'options' => ['class' => 'navbar-nav navbar-right'],
        'items' => $menuItems,
    ]);
    NavBar::end();
    ?>
    <div class="notification-container">
                                                                                                                                                                   
        <?php foreach ($notification as $notifications): ?>
            <?php echo $notifications->name; ?>
        <?php endforeach ?>

        <?php Html::a('Прочитать все напоминание', ['#']) ?>
    </div>

Answer 1

[Screamie Di]

В foreach вы передаете переменную $notification, а где ее инициализация? В этой переменной должен содержаться массив объектов. А приходит, видимо, null.

Comments

[hollanditkzn]

да я ее в контроллере вывел, как понимаю во view надо брать данные как понимаю?

[Screamie Di]

Скиньте код контроллера и будет понятней. А так да, в контроллере вы используете метод render() или renderPartial() в котором первым параметром указываете view, а вторым идет именованный массив переменных для вьюшки.

Проверьте, что передаете правильный первый аргумент в foreach. Он должен быть массивом

[hollanditkzn]

Screamie Di: Как понимаю весь не стоит, а только action или нет нужен весь. Просто проще дать гид

use app\models\Notification;
class ZakazController extends Controller
{
public function actionView($id)
    {
        $model = $this->findModel($id);
        $notification = new Notification();
        $shipping = $model->idShipping;
        $statusDisain = $model->statusDisain;
        $status = $model->status;

        $shipping = new Courier();
        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            $model->id_shipping = $shipping->id;//Оформление доставки

            $notification->id_user = 7;//оформление уведомление
            $notification->name = 'Доставка '.$model->description;
            $notification->active = 0;

            $notification->save();
            $model->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;
            $notification->id_user = 5;//оформление уведомление
            $notification->name = 'Дизайнер выполнил работу №'.$model->id_zakaz.' '.$model->description;
            $notification->active = 0;
            }            
            $model->save();
            if ($model->status == 3) {
                $notification->id_user = 3;//оформление уведомление дизайнеру
                $notification->name = 'Новый заказ №'.$model->id_zakaz;
                $notification->active = 0;
                $notification->save();
            } elseif ($model->status == 6) {
                $notification->id_user = 6;//оформление уведомление мастеру
                $notification->name = 'Новый заказ №'.$model->id_zakaz;
                $notification->active = 0;
                $notification->save();
            }

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

        return $this->render('view', [
            'model' => $this->findModel($id),
            'notification' => $notification,
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }
}

[Screamie Di]

Понятно. Вы передаете переменную в обычный view файл. А поймать ее хотите в layot шаблоне main.php

Она недоступна в main.php, для того, что бы она была там доступна, вы можете создать публичное свойство у класса ZakazController, например так:

use app\models\Notification;
class ZakazController extends Controller
{

public function actionView($id)
    {
        $model = $this->findModel($id);
        $notification = new Notification();
        $shipping = $model->idShipping;
        $statusDisain = $model->statusDisain;
        $status = $model->status;

        $shipping = new Courier();
        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            $model->id_shipping = $shipping->id;//Оформление доставки

            $notification->id_user = 7;//оформление уведомление
            $notification->name = 'Доставка '.$model->description;
            $notification->active = 0;

            $notification->save();
            $model->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;
            $notification->id_user = 5;//оформление уведомление
            $notification->name = 'Дизайнер выполнил работу №'.$model->id_zakaz.' '.$model->description;
            $notification->active = 0;
            }            
            $model->save();
            if ($model->status == 3) {
                $notification->id_user = 3;//оформление уведомление дизайнеру
                $notification->name = 'Новый заказ №'.$model->id_zakaz;
                $notification->active = 0;
                $notification->save();
            } elseif ($model->status == 6) {
                $notification->id_user = 6;//оформление уведомление мастеру
                $notification->name = 'Новый заказ №'.$model->id_zakaz;
                $notification->active = 0;
                $notification->save();
            }
           
            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

       // Здесь ищем те уведомления которые нам нужны и присваиваем их в параметр вьюшки
       $this->view->params['notification'] = Notification::find()->all();        

        return $this->render('view', [
            'model' => $this->findModel($id),
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }
}

Затем присваиваете ей нужное значение и она будет доступна в main.php, как $this->notification. Только обратите внимание, что ваш $notification будет виден только когда вы используете ZakazController. Поправил ответ, теперь присваивание происходит перед рендерингом.

[Screamie Di]

И такие толстые контроллеры - не айс. Вынесите бизнес логику в модель и сделайте контроллер по тоньше

[hollanditkzn]

Screamie Di: Ну я немного его уменьшил где выставляет статус и сохранение. Или еще вынести описание и указание к какому юзеру уведомление относится. Но там выходит в описание нужно еще подключать другую модель

public function actionView($id)
    {
        $model = $this->findModel($id);
        $notification = new Notification();
        $shipping = $model->idShipping;
        // $this->view->params['notification'] = $notification;

        $shipping = new Courier();
        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            $model->id_shipping = $shipping->id;//Оформление доставки

            $notification->id_user = 7;//оформление уведомление доставки
            $notification->name = 'Доставка '.$model->description;
            $notification->saveNotification;
            $model->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;
                $notification->id_user = 5;//оформление уведомление выполение работы дизайнера
                $notification->name = 'Дизайнер выполнил работу №'.$model->id_zakaz.' '.$model->description;
                $notification->saveNotification;
            }            
            $model->save();
            if ($model->status == 3) {
                $notification->id_user = 3;//оформление уведомление дизайнеру
                $notification->name = 'Новый заказ №'.$model->id_zakaz;
                $notification->saveNotification;
            } elseif ($model->status == 6) {
                $notification->id_user = 6;//оформление уведомление мастеру
                $notification->name = 'Новый заказ №'.$model->id_zakaz;
                $notification->saveNotification;
            }

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

        return $this->render('view', [
            'model' => $this->findModel($id),
            'notification' => $notification,
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }

[Screamie Di]

Можно еще сократить. И вынести функции сохранения уведомления в модель Notification. Например так:

public function actionView($id)
    {
        $model = $this->findModel($id);
        $notification = new Notification();
        $shipping = $model->idShipping;  // Здесь в присваиваете в $shipping id
        $shipping = new Courier(); // А здесь сразу переопределяете $shipping, то есть строчка выше не нужна
        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            $model->id_shipping = $shipping->id;//Оформление доставки
            
            // В модели Notification пишем функцию saveByIdUser($id), которая будет сохранять
           // уведомление в зависимости от id, что бы не писать бизнес логику в контроллере
            $notification->saveByIdUser(7);

            $model->save();
            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

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

            // Для загрузки файлов лучше написать отдельный метод в модели например uploadFile()
            // который будет проверять размер файла, наличие каталога, генерировать случайное имя и т.д.
            // во избежании проблем в будущем.

            /* Тогда это можно будет убрать из контроллера
            $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;
                $notification->saveByIdUser(5);
            } */ 

          
            $model->uploadFile();
            $model->save();

            if ($model->status == 3 || $model->status == 6)
                $notification->saveByIdUser($model->status)

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

        return $this->render('view', [
            'model' => $this->findModel($id),
            'notification' => $notification,
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }

И вам нужно внимательно проверить содержание этого контроллера. Т.к. там довольно много спорных моментов, но описывать все сейчас нет времени. В итоге ваш контроллер будет выглядеть примерно так

public function actionView($id)
    {
        $model = $this->findModel($id);
        $notification = new Notification();
        $shipping = new Courier(); 

        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            $model->id_shipping = $shipping->id;
            $model->save();
            $notification->saveByIdUser(7);

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

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

            if ($model->status == 3 || $model->status == 6)
                $notification->saveByIdUser($model->status)

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

        return $this->render('view', [
            'model' => $this->findModel($id),
            'notification' => $notification,
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }

Еще нужно добавить проверки при сохранении моделей и валидацию.

[hollanditkzn]

Screamie Di: А так точно будет работать?
$notification->saveByIdUser(7);
Просто у меня пишет Calling unknown method: app\models\Notification::saveByIdUser()
И второй момент просто по основному вопросу. Как передавать массив во foreach

[Screamie Di]

Нужна написать метод saveByuserId() самому. Его у вас нет.

Массив передается так, в контроллере:

public function actionView($id)
    {
        $model = $this->findModel($id);
        $notification = new Notification();
        $shipping = new Courier(); 

        if ($shipping->load(Yii::$app->request->post()) && $shipping->save()) {
            $model->id_shipping = $shipping->id;
            $model->save();
            $notification->saveByIdUser(7);

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

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

            if ($model->status == 3 || $model->status == 6)
                $notification->saveByIdUser($model->status)

            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }

        // Создаем параметр notifications в модели view, где будем хранить массив
       // И передаем туда массив объектов Notification
        $this->view->params['notifications'] = Notification::find()->all();

        return $this->render('view', [
            'model' => $this->findModel($id),
            'user_name' => $user_name,
            'shipping' => $shipping,
            'file' => $file,
        ]);
    }

В шаблоне main.php:

// Ваш код шаблона
// ...
//

foreach($this->params['notifications'] as $notification){

    // Действия с экземпляром модели Notification, например
    echo $notification->name;
}

[hollanditkzn]

Screamie Di: У меня почему-то опять инфвалид опять. Скорее всего с $this как понимаю. Или нет?
Точнее при выводде этой конструкции массива Getting unknown property: yii\web\View::view

[hollanditkzn]

Все догнал, я по другому выводил. Спасибо за помощь

[Screamie Di]

Не за что! Удачного кодинга