Как передавать в функции модели id?

Question

[hollanditkzn]

Можно ли как то в моделе передавать id? или нет. Например в контроллере передавать id в геттер допустим и соответственно происходило без ошибок. У меня выходит такая ошибка Calling unknown method: app\models\Notification::byIdUser()
Вот моделе

public function getByIdUser($id)
    {
        $zakaz = new Zakaz();
        $notification->id_user = $id;
        $notification->name = 'Доставка '.$model->description;
        $notification->id_zakaz = $zakaz->id_zakaz;
    }

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

$notification->byIdUser(7);
$notification->byIdUser(7);

Answer 1

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

Если это id текущего юзера, то есть:
Yii::$app->user->id
Но Ваша функция ничего не возвращает. Она в принципе бессмысленная.

Comments

[hollanditkzn]

да id, только не данного пользователя, а для другого. Это для уведомление делаю.
то есть указывает что-то, уведомление приходит другому пользователю

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

hollanditkzn: $notification->byIdUser(7); это обращение как к атрибуту, а не как к функции, надо так $notification->getByIdUser(7);

[hollanditkzn]

Максим Тимофеев: Выходит что неизвестная метод
Calling unknown method: app\models\Notification::getByIdUser()

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

hollanditkzn: Значит нет в app\models\Notification метода getByIdUser()

[hollanditkzn]

Максим Тимофеев: А да, точно, я все делал в другой моделе делал

[hollanditkzn]

Только в моделе я делал, только выходит Creating default object from empty value. Только я не пойму, почему пустое значение, если я передаю туда 7
То есть выходит на эту строку $notification->id_user = $id;

public function getByIdUser($id)
    {
        $zakaz = new Zakaz();
        $notification->id_user = $id;
        $notification->name = 'Доставка '.$zakaz->description;
        $notification->id_zakaz = $zakaz->id_zakaz;
        $notification->category = 0;
        // $notification->saveNotification;
    }

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

hollanditkzn: Вы создали новый заказ, а потом дергаете из него значения и удивляетесь что они пустые?
В Вашей функции $zakaz->description и $zakaz->id_zakaz пустые. Соответственно $notification->id_zakaz = $zakaz->id_zakaz; лишено смысла. Но не думаю что

Creating default object from empty value

с этим связано. Вы при дали код того места на которое ошибка ссылается.

[hollanditkzn]

Максим Тимофеев: В контроллере все работает прекрасно, но слишком много кода. Новый заказ не создан, создано только уведомление. Да с этим не связано. А как лучше определить. Я же в него ввожу 7, соотвтественно $id = 7, я правильно мыслю?

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

Вы так пишите, как будто я рядом сижу и в код Вас смотрю

Я же в него ввожу 7

В кого в него? Как вводите?

соотвтественно $id = 7, я правильно мыслю?

хрен его знает. Вы что-то куда-то вводите и спрашиваете равен ли $id непонятно где, семерке, которую Вы как-то вставили в код.

[hollanditkzn]

Максим Тимофеев контроллер

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

            $zakaz = $model->id_zakaz;

            $notification->getByIdUser(7, $zakaz);
            $notification->saveNotification;
            
            return $this->redirect(['view', 'id' => $model->id_zakaz]);
        }
 }
        $this->view->params['notifications'] = Notification::find()->where(['id_user' => Yii::$app->user->getId(), 'active' => true])->all();

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

В моделе

public function getByIdUser($id, $zakaz)
    {
        $notification->id_user = $id;
        $notification->name = 'Доставка '.$zakaz;
        $notification->id_zakaz = $zakaz;
        $notification->category = 0;
        // $notification->saveNotification;
    }

Вот весь код.

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

$shipping = new Courier(); //тут создали
$shipping->id; //тут все пусто и id в том числе. id появится после сохранения
$shipping->save(); //сохранили
$shipping->id; //теперь там есть id

[hollanditkzn]

Максим Тимофеев: Мне нужно его привязать к $model->id_zakaz, не к доставке

$shipping->save();//сохранили доставку
$model->id_shipping = $shipping->id;//поместили id доставки в заказ 
            $model->save();//сохранили
            $zakaz = $model->id_zakaz;//указали №заказа
            $notification->getByIdUser(7, $zakaz);//передали в уведомление id юзеру, на данный момент 7 это курьер, передали вторым аргументом № закаказа

[hollanditkzn]

Максим Тимофеев: Правильная логика тут же?

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

У Вас $model уже существует в базе, ее id можно получить и без сохранения.
Но я бы Вам рекомендовал почитать про события в yii2 (afterSave, beforeSave). Именно там должна быть часть этой логики. И в afterSave соответственно и id уже есть.

[hollanditkzn]

Максим Тимофеев: model да, только это переходит в другую модель

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

hollanditkzn: Вот опять. Что это? В какую другую? На мой взгляд модель переходит в другую модель - это ересь какая-то. Вы видимо не можете адекватно сформулировать.

[hollanditkzn]

Максим Тимофеев: Там такая ситуация когда заказ переходит по статусам, то выводит уведомление другому пользователю, что у него пришел новый заказ, для курьера это создана новая доставка. 1 юзер может иметь несколько уведомлений. Я сделал так что уведомление связаны с юзером и заказом. А контроллер данный идет из заказа. С уведомлением она не связана.

[hollanditkzn]

Максим Тимофеев: Надо было добавить $this а не $notification. Мне сейчас подсказали. А то бился с контроллером, из головы ушло что надо было обращаться к классу в моделе

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

hollanditkzn: Сорри по такому объяснению помочь не смогу. На конкретные вопросы отвечу, а разбираться в Вашей структуре, времени нет.