Неопределенная переменная Laravel в уведомлении

Question

Я экспериментирую с Laravel, и я наткнулся на эту статью. Я следовал этому точно, но по какой-то причине при отправке почты в классе Notification он не может найти переменную $ user, которую я объявил в конструкторе. При печати в конструкторе он работает так, что пользовательский объект передается корректно, но когда я хочу получить к нему доступ в методе toMail, по какой-то причине он отсутствует. Кто-нибудь знает, почему и как это исправить?

<?php

namespace App\Notifications;

use App\User;
use Illuminate\Bus\Queueable;
use Illuminate\Notifications\Notification;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;

class UserRegisteredSuccessfully extends Notification
{
    use Queueable;

    /**
     * @var User
     */
    protected $user;

    /**
     * Create a new notification instance.
     *
     * @param User $user
     */
    public function __construct(User $user)
    {
        $this->$user = $user;
        // printing here works
    }

    /**
     * Get the notification's delivery channels.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function via($notifiable)
    {
        return ['mail'];
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)
    {
        // ERROR HERE (Undefined variable: user)
        $user = $this->$user;

        return (new MailMessage)
                    ->subject('Succesfully created new account')
                    ->greeting(sprintf('Hello %s', $user->username))
                    ->line('You have successfully registered to our system. Please activate your account.')
                    ->action('Click here', route('activate.user', $user->activation_code))
                    ->line('Thank you for using our application!');
    }

    /**
     * Get the array representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function toArray($notifiable)
    {
        return [
            //
        ];
    }
}

Метод регистрации:

/**
 * Register new user
 * 
 * @param Request $request
 * @return User
 */
protected function register(Request $request) 
{
    $validatedData = $request->validate([
        'username' => 'required|string|max:255',
        'email' => 'required|string|email|max:255|unique:users',
        'password' => 'required|string|min:6|confirmed',
    ]);

    try {
        $validatedData['password'] = Hash::make(array_get($validatedData, 'password'));
        $validatedData['activation_code'] = str_random(30).time();
        $user = app(User::class)->create($validatedData);

    } catch(\Exception $ex) {
        logger()->error($ex);
        return redirect()->back()->with('message', 'Unable to create new user.');
    }

    $user->notify(new UserRegisteredSuccessfully($user));
    return redirect()->back()->with('message', 'Successfully created a new account. Please check your email and activate your account.');
}

Заранее спасибо!

Answer 1

Вы сделали 2 опечатки:

В вашем конструкторе:

$this->$user = $user;

Должно быть:

$this->user = $user;

А в методе toMail():

$user = $this->$user;

Должно быть:

$user = $this->user;

---

Причина, по которой он не работает, заключается в том, что в настоящее время вы используете значение из $user в качестве имени переменной и не присваиваете значение объекта User для $this->user.