Laravel notifications не присваивается?

Question

[NikSIk31]

почему такой код не добавляет в конечный notification поле parsed_created_at ? .
Ретурнится все что получено с БД кроме моих изменений (добавления поля parsed_created_at)

namespace App\Events;

use App\Notification;
use Jenssegers\Date\Date;
use Illuminate\Queue\SerializesModels;
use Illuminate\Broadcasting\PrivateChannel;
use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Broadcasting\InteractsWithSockets;
use Illuminate\Contracts\Broadcasting\ShouldBroadcast;

class NewNotification implements ShouldBroadcast
{
    use Dispatchable, InteractsWithSockets, SerializesModels;

    public $user_id;
    public $notification;

    public function __construct($user_id, $notification_id)
    {
        $this->user_id = $user_id;
        $notification = Notification::find($notification_id);
        $notification->parsed_created_at = Date::parse($notification['created_at'])->format('d F в H:i');
        $this->notification = $notification;
    }

    public function broadcastOn()
    {
        return new PrivateChannel('notification.' . $this->user_id);
    }

    public function broadcastWith()
    {
        return [
            'success' => true,
            'notification' => $this->notification
        ];
    }
}

Answer 1

[Юрий Кулаксыз]

$notification->save()
Забыли

Comments

[NikSIk31]

Но я ведь не сохранить в БД хочу. А лишь получить это уведомление и немного изменить, а дальше отдать на бродкаст. Эти изменения не нужно сохранять

[Юрий Кулаксыз]

NikSIk31, ну драйвер notifications же у вас стоит database?

[NikSIk31]

Юрий Кулаксыз, хм, должно быть так. Я просто пока не у компа, но должно быть что я не менял драйвер

[Юрий Кулаксыз]

NikSIk31, по умолчанию стоит email.
Плаваете в этом плане, почитайте документацию по уведомлениям, особенно database notifications, это именно то, что вам нужно.

[NikSIk31]

Юрий Кулаксыз, но это же events получается, если я save сделаю, оно сохранит это в бд? Я сейчас посмотрю конечно доку

[NikSIk31]

Юрий Кулаксыз, Я вообще не вижу чтобы был notifications_driver , это где такое?

[Юрий Кулаксыз]

NikSIk31,

public function via($notifiable)
    {
        return ['database'];
    }

Этот метод должен быть в классе уведомления. И toArray() метод ещё.
Почитайте, как настраивать уведомления, а то вы броадкастить хотите, не создавая и не сохраняя уведомления.

Answer 2

[Alex Wells]

Зачем отдавать в broadcastWith() обьект нотификации? Отдавайте массив примитивок с нужными данными.

Comments

[NikSIk31]

Так ведь больше будет, сейчас пока это расписать все. Ну не знаю может быть это решит проблему, но я же там генерю несколько новых переменных, мне тогда нужно создать эти переменные типа parsed_created_at и т.д.? Ведь они не хотят записываться в объект нотификаций

[Alex Wells]

NikSIk31, а почему ты, собственно, не используешь нотификации ларки? toArray в них тебе поможет.

[NikSIk31]

Alex Wells, но мне не надо сохранять в БД. Просто один юзер делает действие, а второму идет по бродкасту это с инфой

[Alex Wells]

NikSIk31, в смысле "не надо сохранять в бд"? А тут ты откуда данные вытягиваешь то

$notification = Notification::find($notification_id);

?

[NikSIk31]

Alex Wells, так я уже вытягиваю сохраненные данные и их надо мне отдать. Сейчас не важно в каком месте происходит сохранение

[Alex Wells]

NikSIk31, как это перечит нотификациям ларки?